Wire provider_id into scriptable API provider to start listening events

content/child/service_worker/web_service_worker_provider_impl.cc

 // Copyright 2013 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/child/service_worker/web_service_worker_provider_impl.h"
 
 #include "base/atomic_sequence_num.h"
 #include "base/logging.h"
 #include "content/child/child_thread.h"
 #include "content/child/service_worker/service_worker_dispatcher.h"
 #include "content/child/thread_safe_sender.h"
-#include "content/common/service_worker/service_worker_messages.h"
-#include "third_party/WebKit/public/platform/WebServiceWorkerProviderClient.h"
 #include "third_party/WebKit/public/platform/WebURL.h"
 
 using blink::WebURL;
 
 namespace content {
 
 WebServiceWorkerProviderImpl::WebServiceWorkerProviderImpl(
-    ThreadSafeSender* thread_safe_sender)
+    ThreadSafeSender* thread_safe_sender,
+    int provider_id)
     : thread_safe_sender_(thread_safe_sender),
-      client_(NULL) {
+      provider_id_(provider_id) {
 }
 
 WebServiceWorkerProviderImpl::~WebServiceWorkerProviderImpl() {

[michaeln, 2014/03/07 01:26:23]

Do we need to send the reset message here to get the backend to forget about
this thread_id or are we guaranteed to have seen a setClient(NULL) before we get
here?

[kinuko, 2014/03/07 02:02:17]

Done.  Would be better to ensure that.

 }
 
 void WebServiceWorkerProviderImpl::setClient(
     blink::WebServiceWorkerProviderClient* client) {
-  // TODO(kinuko): We should register the client with provider_id
-  // so that the client can start listening events for the provider.
-  client_ = client;
+  GetDispatcher()->SetScriptClient(provider_id_, client);
 }
 
 void WebServiceWorkerProviderImpl::registerServiceWorker(
     const WebURL& pattern,
     const WebURL& script_url,
     WebServiceWorkerCallbacks* callbacks) {
-  ServiceWorkerDispatcher::ThreadSpecificInstance(thread_safe_sender_)
-      ->RegisterServiceWorker(pattern, script_url, callbacks);
+  GetDispatcher()->RegisterServiceWorker(pattern, script_url, callbacks);
 }
 
 void WebServiceWorkerProviderImpl::unregisterServiceWorker(
     const WebURL& pattern,
     WebServiceWorkerCallbacks* callbacks) {
-  ServiceWorkerDispatcher::ThreadSpecificInstance(thread_safe_sender_)
-      ->UnregisterServiceWorker(pattern, callbacks);
+  GetDispatcher()->UnregisterServiceWorker(pattern, callbacks);
+}
+
+ServiceWorkerDispatcher* WebServiceWorkerProviderImpl::GetDispatcher() {
+  return ServiceWorkerDispatcher::ThreadSpecificInstance(thread_safe_sender_);
 }
 
 }  // namespace content