Issue metadata
Sign in to add a comment
|
[gFeedback Prestable] Images not loading while data saver ON |
||||||||||||||||||||||
Issue descriptionImages don't load in data saver mode UI language: en-US Version: 65.0.3295.0 Device Model: Pixel Device Release: 8.1.0 App Version: 65.0.3295.0 Product Specific Data (whitelisted): Chrome Home State: Enabled Chrome Variations: 3300132 3310967 3313323 3313956 3314106 3314176 Connection check elapsed (ms): 214 Connection type: WiFi Data Reduction Proxy Enabled: true Effective WebView package name: com.android.chrome Effective WebView package version: VersionCode:323911152;VersionName:63.0.3239.111 HTTP connection check (Android network stack): CONNECTED HTTP connection check (Chrome network stack): CONNECTED HTTPS connection check (Android network stack): CONNECTED HTTPS connection check (Chrome network stack): CONNECTED URL: http://hexclad.com/product/8-hexclad/ com.google.android.webview: VersionCode:316409800;VersionName:61.0.3163.98 lowmem: false
,
Dec 18 2017
I expect this is works-as-intended. This page was a Lo-Fi preview that is enabled via Data Saver and when the network is slow. In the video, when the page is loaded notice the info bar at the bottom that says "Saved data. Show original" You can click on "Show original" to get the full page with images; or you can long press on the image placeholder to get an option to load that single image. You will get different results depending on the effective-connection-type. That is if the network quality is currently rate a 2G or slower, you will get the Lo-Fi preview; if rated 3G or faster then you will get the full page. At least, that is the intended behavior. You can check the effective-connection-type on 65 version by visiting the chrome://interventions-internals page and going to the Network Quality tab. Can you try to repro and report the Network Quality value?
,
Dec 18 2017
FWIW, there has been a ~3x spike in the number of LoFi infobars shown: https://uma.googleplex.com/p/chrome/timeline_v2/?sid=4d61628edb5ffc625a55fd72e36c48eb
,
Dec 18 2017
Thanks Tarun, will look further.
,
Dec 18 2017
I was able to repro on local debug build and hit related DCHECK.
,
Dec 18 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9542a805b31d65f8fb144e4f016b90b85edb964c commit 9542a805b31d65f8fb144e4f016b90b85edb964c Author: Doug Arnett <dougarnett@chromium.org> Date: Mon Dec 18 22:57:54 2017 Fix issue with Lo-Fi occurring for http without slow network Bug: 795557 Change-Id: I61c3a0dc01851d3d057a948e0d0be8dd65628e9d Reviewed-on: https://chromium-review.googlesource.com/832521 Reviewed-by: Ryan Sturm <ryansturm@chromium.org> Commit-Queue: Doug Arnett <dougarnett@chromium.org> Cr-Commit-Position: refs/heads/master@{#524844} [modify] https://crrev.com/9542a805b31d65f8fb144e4f016b90b85edb964c/components/data_reduction_proxy/content/browser/content_lofi_decider.cc [modify] https://crrev.com/9542a805b31d65f8fb144e4f016b90b85edb964c/components/data_reduction_proxy/content/browser/content_lofi_decider_unittest.cc [modify] https://crrev.com/9542a805b31d65f8fb144e4f016b90b85edb964c/components/data_reduction_proxy/core/browser/data_reduction_proxy_data.cc [modify] https://crrev.com/9542a805b31d65f8fb144e4f016b90b85edb964c/components/data_reduction_proxy/core/browser/data_reduction_proxy_data.h [modify] https://crrev.com/9542a805b31d65f8fb144e4f016b90b85edb964c/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.cc [modify] https://crrev.com/9542a805b31d65f8fb144e4f016b90b85edb964c/tools/chrome_proxy/webdriver/lofi.py
,
Dec 18 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by pnangunoori@chromium.org
, Dec 18 2017Labels: hasbisect-per-revision Needs-triage-Mobile Triaged-Mobile
Owner: dougarnett@chromium.org
Status: Assigned (was: Untriaged)