Don't show an offline preview on a reloaded page |
|||
Issue descriptionIf a user reloads a page (via refresh button, typing in the URL again, swiping, etc.). The navigation should not be eligible for showing an offline preview.
,
Dec 7 2016
Thanks. I was thinking the same thing: https://codereview.chromium.org/2563493002/
,
Dec 7 2016
,
Dec 14 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/bc5da354fd37ccda92436a438f3f41ff0d1b940e commit bc5da354fd37ccda92436a438f3f41ff0d1b940e Author: ryansturm <ryansturm@chromium.org> Date: Wed Dec 14 18:43:03 2016 Disallow showing offline previews on a reload. This change prevents reloads being shown an offline preview. This does not prevent other previews from being shown. A reload on an offline preview is still considered to be non-opt out for the blacklist unless the user hit "show original". BUG= 672237 Review-Url: https://codereview.chromium.org/2563493002 Cr-Commit-Position: refs/heads/master@{#438549} [modify] https://crrev.com/bc5da354fd37ccda92436a438f3f41ff0d1b940e/components/previews/core/previews_black_list.h [modify] https://crrev.com/bc5da354fd37ccda92436a438f3f41ff0d1b940e/components/previews/core/previews_io_data.cc [modify] https://crrev.com/bc5da354fd37ccda92436a438f3f41ff0d1b940e/components/previews/core/previews_io_data_unittest.cc [modify] https://crrev.com/bc5da354fd37ccda92436a438f3f41ff0d1b940e/tools/metrics/histograms/histograms.xml
,
Dec 20 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by mmenke@chromium.org
, Dec 7 2016