New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 759632 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Data-saver layout test not working with PlzNavigate.

Project Member Reported by arthurso...@chromium.org, Aug 28 2017

Issue description

The blink layout test:
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/http/tests/inspector/network/network-datasaver-warning.html
doesn't work with PlzNavigate(--enable-browser-side-navigation)

The test checks that a console log warning is displayed on reload when the
Data-Saver is enabled.

The reason the test fails is that ContentBrowserClient::IsDataSaverEnabled() is
implemented only by ChromeContentBrowserClient. The blink layout tests are
using ShellContentBrowserClient instead. So from the browser point of view
Data-Saver is always disabled in layout tests.

The solution would be to replace this layout test with a chrome browser test.

+CC: FYI
 
Owner: allada@chromium.org
Status: Assigned (was: Available)
Status: WontFix (was: Assigned)
Triaging network issues, closing the ones we won't be able to address.

Sign in to add a comment