Discuss and decide if we should have a similar functionality for taking a snapshot of whatever we have loaded if it's the last retry per policy and it's a timeout. Counterpart for prerenderer: 680363
TODO link: https://cs.chromium.org/chromium/src/chrome/browser/android/offline_pages/background_loader_offliner.cc?q=TODO.romax.+lang:c%2B%2B&l=196&ct=rc&cd=1&dr=C
Taking this over because some UMA we should add too. Let me know if you are already working on this.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3bcd8469054bf76be2008630904f4e2f9ba146e1 commit 3bcd8469054bf76be2008630904f4e2f9ba146e1 Author: chili <chili@chromium.org> Date: Tue Apr 18 01:04:36 2017 [Offline pages]: Implement background loader to save on last retry, and record last retry success UMA separately BUG= 705090 Review-Url: https://codereview.chromium.org/2818783002 Cr-Commit-Position: refs/heads/master@{#465088} [modify] https://crrev.com/3bcd8469054bf76be2008630904f4e2f9ba146e1/chrome/browser/android/offline_pages/background_loader_offliner.cc [modify] https://crrev.com/3bcd8469054bf76be2008630904f4e2f9ba146e1/chrome/browser/android/offline_pages/background_loader_offliner.h [modify] https://crrev.com/3bcd8469054bf76be2008630904f4e2f9ba146e1/chrome/browser/android/offline_pages/background_loader_offliner_unittest.cc [modify] https://crrev.com/3bcd8469054bf76be2008630904f4e2f9ba146e1/chrome/browser/android/offline_pages/prerendering_offliner.cc [modify] https://crrev.com/3bcd8469054bf76be2008630904f4e2f9ba146e1/chrome/browser/android/offline_pages/prerendering_offliner.h [modify] https://crrev.com/3bcd8469054bf76be2008630904f4e2f9ba146e1/chrome/browser/android/offline_pages/prerendering_offliner_unittest.cc [modify] https://crrev.com/3bcd8469054bf76be2008630904f4e2f9ba146e1/components/offline_pages/core/background/offliner.h [modify] https://crrev.com/3bcd8469054bf76be2008630904f4e2f9ba146e1/components/offline_pages/core/background/request_coordinator.cc [modify] https://crrev.com/3bcd8469054bf76be2008630904f4e2f9ba146e1/components/offline_pages/core/background/request_coordinator_event_logger.cc [modify] https://crrev.com/3bcd8469054bf76be2008630904f4e2f9ba146e1/tools/metrics/histograms/histograms.xml
Comment 1 by romax@chromium.org
, Mar 24 2017