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

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

Issue 2416083002: [Offline pages] Introduction of MarkAttemptStartedTask with tests (Closed)
Patch Set: Addressing code review feedback Created 4 years, 2 months 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.h
diff --git a/components/offline_pages/background/request_coordinator.h b/components/offline_pages/background/request_coordinator.h
index 371fd23c319dac2200575e2fdbc700c98867a499..780fd4032f763ea0a87b02404aacbdd75dcda94d 100644
--- a/components/offline_pages/background/request_coordinator.h
+++ b/components/offline_pages/background/request_coordinator.h
@@ -283,8 +283,15 @@ class RequestCoordinator : public KeyedService,
// Cancels an in progress pre-rendering, and updates state appropriately.
void StopPrerendering(Offliner::RequestStatus stop_status);
+ // Marks attempt on the request and sends it to offliner in continuation.
void SendRequestToOffliner(const SavePageRequest& request);
+ // Continuation of |SendRequestToOffliner| after the request is marked as
+ // started.
+ void StartOffliner(int64_t request_id,
+ const std::string& client_namespace,
+ std::unique_ptr<UpdateRequestsResult> update_result);
+
// Called by the offliner when an offlining request is completed. (and by
// tests).
void OfflinerDoneCallback(const SavePageRequest& request,

Powered by Google App Engine
This is Rietveld 408576698