Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(376)

Unified Diff: components/offline_pages/background/request_coordinator.cc

Issue 2472593002: Add retries for completed background loading attempts (Closed)
Patch Set: CR feedback Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/offline_pages/background/request_coordinator.cc
diff --git a/components/offline_pages/background/request_coordinator.cc b/components/offline_pages/background/request_coordinator.cc
index 120c08db9b92122170187545e8727f4c00faff31..9376177bd251d131f3421fd233b6fffbed824f11 100644
--- a/components/offline_pages/background/request_coordinator.cc
+++ b/components/offline_pages/background/request_coordinator.cc
@@ -218,8 +218,7 @@ void RequestCoordinator::StopPrerendering(Offliner::RequestStatus stop_status) {
offliner_->Cancel();
// If we timed out, let the offliner done callback handle it.
- if (processing_state_ == ProcessingWindowState::IMMEDIATE_WINDOW &&
- stop_status == Offliner::RequestStatus::REQUEST_COORDINATOR_TIMED_OUT)
+ if (stop_status == Offliner::RequestStatus::REQUEST_COORDINATOR_TIMED_OUT)
return;
// Otherwise, this attempt never really had a chance to run, mark it

Powered by Google App Engine
This is Rietveld 408576698