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

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

Issue 2608553002: [OfflinePages] Improve visiblity/handling of "Loading not started" case (Closed)
Patch Set: Fixed test Created 4 years 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/core/background/request_coordinator.cc
diff --git a/components/offline_pages/core/background/request_coordinator.cc b/components/offline_pages/core/background/request_coordinator.cc
index f098edfaf85f348f69e0cda5fd4c0201d09d77da..6b6d4d2811dc0b1ab2f8af118a1cebdc6d568af2 100644
--- a/components/offline_pages/core/background/request_coordinator.cc
+++ b/components/offline_pages/core/background/request_coordinator.cc
@@ -846,7 +846,7 @@ void RequestCoordinator::StartOffliner(
} else {
is_busy_ = false;
DVLOG(0) << "Unable to start LoadAndSave";
- StopProcessing(Offliner::LOADING_NOT_STARTED);
+ StopProcessing(Offliner::LOADING_NOT_ACCEPTED);
// We need to undo the MarkAttemptStarted that brought us to this
// method since we didn't success in starting after all.

Powered by Google App Engine
This is Rietveld 408576698