Move content/child/background_sync to Blink.

third_party/WebKit/Source/modules/background_sync/SyncManager.cpp

 // 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 "modules/background_sync/SyncManager.h"
 
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "core/dom/DOMException.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/dom/ExecutionContext.h"
 #include "modules/background_sync/SyncCallbacks.h"
 #include "modules/serviceworkers/ServiceWorkerRegistration.h"
 #include "public/platform/Platform.h"
-#include "public/platform/modules/background_sync/WebSyncProvider.h"
-#include "public/platform/modules/background_sync/WebSyncRegistration.h"
-#include "wtf/RefPtr.h"
+#include "wtf/ThreadSpecific.h"
 
 namespace blink {
-namespace {
 
-WebSyncProvider* backgroundSyncProvider() {
-  WebSyncProvider* webSyncProvider =
-      Platform::current()->backgroundSyncProvider();
-  ASSERT(webSyncProvider);
-  return webSyncProvider;
-}
+// static
+BackgroundSyncProvider* SyncManager::backgroundSyncProvider() {
+  DEFINE_THREAD_SAFE_STATIC_LOCAL(ThreadSpecific<BackgroundSyncProvider>,

[haraken, 2016/11/08 07:11:36]

nhiroki@: We might want to consider introducing a PerThreadData class to avoid
defining per-thread classes like this with DEFINE_THREAD_SAFE_STATIC_LOCAL.

Ideally, the caller should be able to get the BackgroundSyncProvider by
PerThreadData::current()->backgroundSyncProvider().

[nhiroki, 2016/11/09 05:26:55]

Filed an issue: https://bugs.chromium.org/p/chromium/issues/detail?id=663632
Let's discuss there :)

+                                  syncProvider,
+                                  new ThreadSpecific<BackgroundSyncProvider>);
+  DCHECK(syncProvider);
+  return syncProvider;
 }
 
 SyncManager::SyncManager(ServiceWorkerRegistration* registration)
     : m_registration(registration) {
-  ASSERT(registration);
+  DCHECK(registration);
 }
 
 ScriptPromise SyncManager::registerFunction(ScriptState* scriptState,
                                             ExecutionContext* context,
                                             const String& tag) {
   // TODO(jkarlin): Wait for the registration to become active instead of
   // rejecting. See crbug.com/542437.
   if (!m_registration->active())
     return ScriptPromise::rejectWithDOMException(
         scriptState,
         DOMException::create(AbortError,
                              "Registration failed - no active Service Worker"));
 
   ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
   ScriptPromise promise = resolver->promise();
 
-  WebSyncRegistration* webSyncRegistration = new WebSyncRegistration(
-      WebSyncRegistration::UNREGISTERED_SYNC_ID /* id */, tag,
-      WebSyncRegistration::NetworkStateOnline /* networkState */
-      );
+  mojom::blink::SyncRegistrationPtr syncRegistration =
+      mojom::blink::SyncRegistration::New();
+  syncRegistration->id = SyncManager::kUnregisteredSyncID;
+  syncRegistration->tag = tag;
+  syncRegistration->network_state =
+      blink::mojom::BackgroundSyncNetworkState::ONLINE;
+
   backgroundSyncProvider()->registerBackgroundSync(
-      webSyncRegistration, m_registration->webRegistration(),
-      new SyncRegistrationCallbacks(resolver, m_registration));
+      std::move(syncRegistration), m_registration->webRegistration(),
+      std::unique_ptr<SyncRegistrationCallbacks>(

[jbroman, 2016/11/07 19:48:10]

super-nit: You might consider using makeUnique (from wtf/PtrUtil.h) for cases
like this to avoid having to write the type name twice. I don't feel strongly
one way or the other, though.

[adithyas, 2016/11/08 16:47:47]

Done! I like makeUnique better in this case :)

+          new SyncRegistrationCallbacks(resolver, m_registration)));
 
   return promise;
 }
 
 ScriptPromise SyncManager::getTags(ScriptState* scriptState) {
   ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
   ScriptPromise promise = resolver->promise();
 
   backgroundSyncProvider()->getRegistrations(
       m_registration->webRegistration(),
-      new SyncGetRegistrationsCallbacks(resolver, m_registration));
+      std::unique_ptr<SyncGetRegistrationsCallbacks>(
+          new SyncGetRegistrationsCallbacks(resolver, m_registration)));
 
   return promise;
 }
 
 DEFINE_TRACE(SyncManager) {
   visitor->trace(m_registration);
 }
 
 }  // namespace blink