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

Unified Diff: content/child/background_sync/background_sync_provider.cc

Issue 1316743002: Reland of [Background Sync] Allow sync manager access from uncontrolled clients (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 4 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/child/background_sync/background_sync_provider.cc
diff --git a/content/child/background_sync/background_sync_provider.cc b/content/child/background_sync/background_sync_provider.cc
index bbe67d505b2797bf8eb19abbd3ae0c74b28d689b..750059e2c13d2ed057162258077ba4a385b179f5 100644
--- a/content/child/background_sync/background_sync_provider.cc
+++ b/content/child/background_sync/background_sync_provider.cc
@@ -40,6 +40,7 @@ BackgroundSyncProvider::~BackgroundSyncProvider() {
void BackgroundSyncProvider::registerBackgroundSync(
const blink::WebSyncRegistration* options,
blink::WebServiceWorkerRegistration* service_worker_registration,
+ bool requested_from_service_worker,
blink::WebSyncRegistrationCallbacks* callbacks) {
DCHECK(options);
DCHECK(service_worker_registration);
@@ -53,7 +54,7 @@ void BackgroundSyncProvider::registerBackgroundSync(
// will wipe its callbacks) before 'this' is deleted.
GetBackgroundSyncServicePtr()->Register(
mojo::ConvertTo<SyncRegistrationPtr>(*(optionsPtr.get())),
- service_worker_registration_id,
+ service_worker_registration_id, requested_from_service_worker,
base::Bind(&BackgroundSyncProvider::RegisterCallback,
base::Unretained(this), base::Passed(callbacksPtr.Pass())));
}

Powered by Google App Engine
This is Rietveld 408576698