Besides UX improvements, determine other improvements to faster service background load requests for downloads requested in foreground.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8bc69612087b58e7a679996a2f3e2f60f4f52519 commit 8bc69612087b58e7a679996a2f3e2f60f4f52519 Author: dougarnett <dougarnett@chromium.org> Date: Fri Oct 14 19:25:28 2016 [Offline Pages] Define separate watchdog timeout for concurrent bg loads BUG= 655341 Review-Url: https://codereview.chromium.org/2420503002 Cr-Commit-Position: refs/heads/master@{#425426} [modify] https://crrev.com/8bc69612087b58e7a679996a2f3e2f60f4f52519/components/offline_pages/background/offliner_policy.h [modify] https://crrev.com/8bc69612087b58e7a679996a2f3e2f60f4f52519/components/offline_pages/background/request_coordinator.cc [modify] https://crrev.com/8bc69612087b58e7a679996a2f3e2f60f4f52519/components/offline_pages/background/request_coordinator.h [modify] https://crrev.com/8bc69612087b58e7a679996a2f3e2f60f4f52519/components/offline_pages/background/request_coordinator_unittest.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/48475f48458467f0f196847a6d007dc444294be7 commit 48475f48458467f0f196847a6d007dc444294be7 Author: dougarnett <dougarnett@chromium.org> Date: Mon Oct 24 18:08:48 2016 [Offline Pages] Defines longer processing budget for immediate bg loads. BUG= 655341 Review-Url: https://codereview.chromium.org/2431193003 Cr-Commit-Position: refs/heads/master@{#427098} [modify] https://crrev.com/48475f48458467f0f196847a6d007dc444294be7/components/offline_pages/background/offliner_policy.h [modify] https://crrev.com/48475f48458467f0f196847a6d007dc444294be7/components/offline_pages/background/request_coordinator.cc [modify] https://crrev.com/48475f48458467f0f196847a6d007dc444294be7/components/offline_pages/background/request_coordinator.h [modify] https://crrev.com/48475f48458467f0f196847a6d007dc444294be7/components/offline_pages/background/request_coordinator_unittest.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/75e325cb30270ec359ee79055e825f0e3cfb334a commit 75e325cb30270ec359ee79055e825f0e3cfb334a Author: dougarnett <dougarnett@chromium.org> Date: Wed Nov 02 15:43:11 2016 [Offline Pages] Adds UMA for effective network connection on api calls. Captures the NetworkQualityEstimator connection type on SavePageLater, RemoveRequests, PauseRequest, and ResumeRequests to shed some insight on the conditions under which the user is triggering these apis. BUG= 655341 Review-Url: https://codereview.chromium.org/2466343003 Cr-Commit-Position: refs/heads/master@{#429283} [modify] https://crrev.com/75e325cb30270ec359ee79055e825f0e3cfb334a/components/offline_pages/background/request_coordinator.cc [modify] https://crrev.com/75e325cb30270ec359ee79055e825f0e3cfb334a/tools/metrics/histograms/histograms.xml
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/12aa9df385dcd9e1b85ead71827358bc32648873 commit 12aa9df385dcd9e1b85ead71827358bc32648873 Author: dougarnett <dougarnett@chromium.org> Date: Fri Nov 04 22:05:58 2016 [Offline Pages] Coordinator checks for net connection before starting next req As part of improving the servicing of background load requests, we should avoid scheduling the next request if we have lost the network connection (to save resources and attempt count). When the network comes back then we want scheduler to start us again. BUG= 655341 Review-Url: https://codereview.chromium.org/2467333003 Cr-Commit-Position: refs/heads/master@{#430027} [modify] https://crrev.com/12aa9df385dcd9e1b85ead71827358bc32648873/components/offline_pages/background/request_coordinator.cc [modify] https://crrev.com/12aa9df385dcd9e1b85ead71827358bc32648873/components/offline_pages/background/request_coordinator_unittest.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fa0aebf37854698a45180ac8886b78d22f8b5de4 commit fa0aebf37854698a45180ac8886b78d22f8b5de4 Author: dougarnett <dougarnett@chromium.org> Date: Thu Nov 10 00:20:54 2016 [Offline Pages] Feature flag to allow concurrent background loading on svelte Adds flag (default off) to allow immediate processing of SavePageLater requests on low-end devices. This will allow us to evaluate concurrent loading on our development devices and allows for prospect of a future Finch trial. BUG= 655341 Review-Url: https://codereview.chromium.org/2483463002 Cr-Commit-Position: refs/heads/master@{#431094} [modify] https://crrev.com/fa0aebf37854698a45180ac8886b78d22f8b5de4/chrome/app/generated_resources.grd [modify] https://crrev.com/fa0aebf37854698a45180ac8886b78d22f8b5de4/chrome/browser/about_flags.cc [modify] https://crrev.com/fa0aebf37854698a45180ac8886b78d22f8b5de4/components/offline_pages/background/BUILD.gn [modify] https://crrev.com/fa0aebf37854698a45180ac8886b78d22f8b5de4/components/offline_pages/background/request_coordinator.cc [modify] https://crrev.com/fa0aebf37854698a45180ac8886b78d22f8b5de4/components/offline_pages/background/request_coordinator_unittest.cc [modify] https://crrev.com/fa0aebf37854698a45180ac8886b78d22f8b5de4/components/offline_pages/offline_page_feature.cc [modify] https://crrev.com/fa0aebf37854698a45180ac8886b78d22f8b5de4/components/offline_pages/offline_page_feature.h [modify] https://crrev.com/fa0aebf37854698a45180ac8886b78d22f8b5de4/components/offline_pages/offline_page_model_impl_unittest.cc [modify] https://crrev.com/fa0aebf37854698a45180ac8886b78d22f8b5de4/tools/metrics/histograms/histograms.xml
Closing this now for the M-56 work. Opened 670119 for leveraging network observer for M-57.
Comment 1 by bugdroid1@chromium.org
, Oct 14 2016