[Background Sync] Allow sync manager access from uncontrolled clients

public/platform/modules/background_sync/WebSyncProvider.h

Index: public/platform/modules/background_sync/WebSyncProvider.h
diff --git a/public/platform/modules/background_sync/WebSyncProvider.h b/public/platform/modules/background_sync/WebSyncProvider.h
index f118589cf972ac0d03d1f4467cc7e21df602bc29..175b0b1b4b82a5490845ca0bd976830db26fef83 100644
--- a/public/platform/modules/background_sync/WebSyncProvider.h
+++ b/public/platform/modules/background_sync/WebSyncProvider.h
@@ -29,14 +29,7 @@ public:
 
     // Takes ownership of the WebSyncRegistrationCallbacks.
     // Does not take ownership of the WebServiceWorkerRegistration.
-    // TODO(iclelland): Remove the body of this method once it is implemented in all subclasses.
-    virtual void registerBackgroundSync(const WebSyncRegistration* options, WebServiceWorkerRegistration* serviceWorkerRegistration, bool requestedFromServiceWorker, WebSyncRegistrationCallbacks* callbacks)
-    {
-        registerBackgroundSync(options, serviceWorkerRegistration, callbacks);
-    }
-
-    // TODO(iclelland): Remove this deprecated method once the browser implements the 4-argument version.
-    virtual void registerBackgroundSync(const WebSyncRegistration*, WebServiceWorkerRegistration*, WebSyncRegistrationCallbacks*) { }
+    virtual void registerBackgroundSync(const WebSyncRegistration*, WebServiceWorkerRegistration*, bool, WebSyncRegistrationCallbacks*) = 0;
 
     // Takes ownership of the WebSyncUnregistrationCallbacks.
     // Does not take ownership of the WebServiceWorkerRegistration.