LoFi should not trigger on image documents |
|||
Issue descriptionIf the whole page is an image, it doesn't make sense to replace it with an image placeholder. We should fix that. The options are either to tell the data reduction proxy when the request is a main frame, or not add q=low to mainframe requests.
,
Jun 17 2016
,
Jun 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0fed1fbe1bc0ea5fa0a98795bd203f7a929ef2e9 commit 0fed1fbe1bc0ea5fa0a98795bd203f7a929ef2e9 Author: tbansal <tbansal@chromium.org> Date: Wed Jun 22 23:35:58 2016 Do not trigger Lo-Fi on main frame requests BUG= 619513 Review-Url: https://codereview.chromium.org/2073233002 Cr-Commit-Position: refs/heads/master@{#401460} [modify] https://crrev.com/0fed1fbe1bc0ea5fa0a98795bd203f7a929ef2e9/components/data_reduction_proxy/content/browser/content_lofi_decider.cc [modify] https://crrev.com/0fed1fbe1bc0ea5fa0a98795bd203f7a929ef2e9/components/data_reduction_proxy/content/browser/content_lofi_decider_unittest.cc
,
Jun 22 2016
,
Jun 22 2016
Issue 540928 has been merged into this issue.
,
Jun 22 2016
Thanks for marking as duplicate. I had a feeling that I saw this bug before. |
|||
►
Sign in to add a comment |
|||
Comment 1 by bengr@chromium.org
, Jun 13 2016