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

Unified Diff: content/browser/background_sync/background_sync_service_impl.h

Issue 1344843003: [BackgroundSync] Add browser side support for SyncRegistration.done (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ncn_max
Patch Set: Address comments from PS7 Created 5 years, 3 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_sync/background_sync_service_impl.h
diff --git a/content/browser/background_sync/background_sync_service_impl.h b/content/browser/background_sync/background_sync_service_impl.h
index 7e17b7f0b68f7b193184aac0c449ae420aa5372e..f00c668d4260e38138ddf5eda18f70a3cea92df9 100644
--- a/content/browser/background_sync/background_sync_service_impl.h
+++ b/content/browser/background_sync/background_sync_service_impl.h
@@ -53,6 +53,8 @@ class CONTENT_EXPORT BackgroundSyncServiceImpl
const DuplicateRegistrationHandleCallback& callback) override;
void ReleaseRegistration(
BackgroundSyncRegistrationHandle::HandleId handle_id) override;
+ void NotifyWhenDone(BackgroundSyncRegistrationHandle::HandleId handle_id,
+ const NotifyWhenDoneCallback& callback) override;
void OnRegisterResult(const RegisterCallback& callback,
BackgroundSyncStatus status,
@@ -63,6 +65,9 @@ class CONTENT_EXPORT BackgroundSyncServiceImpl
const GetRegistrationsCallback& callback,
BackgroundSyncStatus status,
scoped_ptr<ScopedVector<BackgroundSyncRegistrationHandle>> result);
+ void OnNotifyWhenDoneResult(const NotifyWhenDoneCallback& callback,
+ BackgroundSyncStatus status,
+ BackgroundSyncState sync_state);
// Called when an error is detected on binding_.
void OnConnectionError();

Powered by Google App Engine
This is Rietveld 408576698