BackgroundSyncManager tracks client registrations

content/renderer/background_sync/background_sync_client_impl.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/background_sync/background_sync_client_impl.h"
 
 #include "content/child/background_sync/background_sync_type_converters.h"
 #include "content/renderer/service_worker/service_worker_context_client.h"
+#include "third_party/WebKit/public/platform/Platform.h"
+#include "third_party/WebKit/public/platform/modules/background_sync/WebSyncProvider.h"
 #include "third_party/WebKit/public/platform/modules/background_sync/WebSyncRegistration.h"
 
 namespace content {
 
 // static
 void BackgroundSyncClientImpl::Create(
     mojo::InterfaceRequest<BackgroundSyncServiceClient> request) {
   new BackgroundSyncClientImpl(request.Pass());
 }
 
 BackgroundSyncClientImpl::~BackgroundSyncClientImpl() {}
 
 BackgroundSyncClientImpl::BackgroundSyncClientImpl(
     mojo::InterfaceRequest<BackgroundSyncServiceClient> request)
     : binding_(this, request.Pass()) {}
 
 void BackgroundSyncClientImpl::Sync(content::SyncRegistrationPtr registration,
                                     const SyncCallback& callback) {
   ServiceWorkerContextClient* client =
       ServiceWorkerContextClient::ThreadSpecificInstance();
   if (!client) {
     callback.Run(SERVICE_WORKER_EVENT_STATUS_ABORTED);
     return;
   }
-  scoped_ptr<blink::WebSyncRegistration> reg =
+
+  scoped_ptr<blink::WebSyncRegistration> web_registration =
       mojo::ConvertTo<scoped_ptr<blink::WebSyncRegistration>>(registration);
-  client->DispatchSyncEvent(*reg, callback);
+
+  blink::WebSyncRegistration registration_copy = *web_registration;
+
+  // The BackgroundSyncServiceImpl doesn't know about this registration yet,
+  // inform it via trackRegistration.
+  blink::WebSyncProvider* web_sync_provider =
+      blink::Platform::current()->backgroundSyncProvider();
+  DCHECK(web_sync_provider);
+
+  // trackRegistration takes ownership of web_registration.
+  web_sync_provider->trackRegistration(web_registration.release());

[michaeln, 2015/08/21 02:39:25]

track is an odd verb? take adopt acquire grab?

[jkarlin, 2015/08/25 17:32:59]

went with DuplicateRegistrationHandle

+
+  client->DispatchSyncEvent(registration_copy, callback);
 }
 
 }  // namespace content