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

Unified Diff: content/browser/background_fetch/background_fetch_context.h

Issue 2776353004: Give the BackgroundFetchJobController a simple state. (Closed)
Patch Set: Created 3 years, 9 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: content/browser/background_fetch/background_fetch_context.h
diff --git a/content/browser/background_fetch/background_fetch_context.h b/content/browser/background_fetch/background_fetch_context.h
index b1e5924fe988b331ef57e671d902d1b708d10bc5..75df6ecd687f54c6139de914defb225aa98048b5 100644
--- a/content/browser/background_fetch/background_fetch_context.h
+++ b/content/browser/background_fetch/background_fetch_context.h
@@ -91,11 +91,8 @@ class CONTENT_EXPORT BackgroundFetchContext
const blink::mojom::BackgroundFetchService::FetchCallback& callback,
blink::mojom::BackgroundFetchError error);
- // Called when a Job Controller has finished processing a Background Fetch
- // registration, as identified by |registration_id|. |aborted_by_developer|
- // will be set to true when it finished at the developer's request.
- void DidFinishFetch(const BackgroundFetchRegistrationId& registration_id,
- bool aborted_by_developer);
+ // Called when a the given |controller| has finished processing its job.
+ void DidCompleteJob(BackgroundFetchJobController* controller);
// |this| is owned by the BrowserContext via the StoragePartitionImpl.
BrowserContext* browser_context_;

Powered by Google App Engine
This is Rietveld 408576698