Add unit tests for ChromeResourceDispatcherHostDelegate::GetPreviewsState |
||||||||
Issue descriptionRecently server previews broke due to bug introduced in ChromeResourceDispatcherHostDelegate::GetPreviewsState There are currently no unit tests for this method.
,
Jul 13 2017
This should be somewhat easier to add since I added the unittest file for this class for something else. I think we initially skipped coverage here because there was no unit test file. Since this method should act as a static method in a lot of ways, it should just require adding tests that call it with various combinations.
,
Aug 15 2017
Making available
,
Aug 15 2017
Doug, mind taking this one?
,
Dec 1 2017
Refreshed during triage.
,
Dec 1 2017
,
Dec 5 2017
,
Dec 5 2017
,
Jan 24 2018
Refreshed during triage.
,
Mar 21 2018
,
May 15 2018
The code has been refactored significantly since this was opened and refactored logic has more test coverage. |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by bengr@chromium.org
, Jul 13 2017Labels: M-62
Owner: megjab...@chromium.org
Status: Assigned (was: Untriaged)