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

Side by Side Diff: content/child/service_worker/web_service_worker_provider_impl.cc

Issue 182863004: Wire provider_id into scriptable API provider to start listening events (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: nit fix Created 6 years, 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/child/service_worker/web_service_worker_provider_impl.h" 5 #include "content/child/service_worker/web_service_worker_provider_impl.h"
6 6
7 #include "base/atomic_sequence_num.h" 7 #include "base/atomic_sequence_num.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "content/child/child_thread.h" 9 #include "content/child/child_thread.h"
10 #include "content/child/service_worker/service_worker_dispatcher.h" 10 #include "content/child/service_worker/service_worker_dispatcher.h"
11 #include "content/child/thread_safe_sender.h" 11 #include "content/child/thread_safe_sender.h"
12 #include "content/common/service_worker/service_worker_messages.h"
13 #include "third_party/WebKit/public/platform/WebServiceWorkerProviderClient.h"
14 #include "third_party/WebKit/public/platform/WebURL.h" 12 #include "third_party/WebKit/public/platform/WebURL.h"
15 13
16 using blink::WebURL; 14 using blink::WebURL;
17 15
18 namespace content { 16 namespace content {
19 17
20 WebServiceWorkerProviderImpl::WebServiceWorkerProviderImpl( 18 WebServiceWorkerProviderImpl::WebServiceWorkerProviderImpl(
21 ThreadSafeSender* thread_safe_sender) 19 ThreadSafeSender* thread_safe_sender,
20 int provider_id)
22 : thread_safe_sender_(thread_safe_sender), 21 : thread_safe_sender_(thread_safe_sender),
23 client_(NULL) { 22 provider_id_(provider_id) {
24 } 23 }
25 24
26 WebServiceWorkerProviderImpl::~WebServiceWorkerProviderImpl() { 25 WebServiceWorkerProviderImpl::~WebServiceWorkerProviderImpl() {
26 // Make sure the script client is removed.
27 GetDispatcher()->RemoveScriptClient(provider_id_);
27 } 28 }
28 29
29 void WebServiceWorkerProviderImpl::setClient( 30 void WebServiceWorkerProviderImpl::setClient(
30 blink::WebServiceWorkerProviderClient* client) { 31 blink::WebServiceWorkerProviderClient* client) {
31 // TODO(kinuko): We should register the client with provider_id 32 if (client)
32 // so that the client can start listening events for the provider. 33 GetDispatcher()->AddScriptClient(provider_id_, client);
33 client_ = client; 34 else
35 GetDispatcher()->RemoveScriptClient(provider_id_);
34 } 36 }
35 37
36 void WebServiceWorkerProviderImpl::registerServiceWorker( 38 void WebServiceWorkerProviderImpl::registerServiceWorker(
37 const WebURL& pattern, 39 const WebURL& pattern,
38 const WebURL& script_url, 40 const WebURL& script_url,
39 WebServiceWorkerCallbacks* callbacks) { 41 WebServiceWorkerCallbacks* callbacks) {
40 ServiceWorkerDispatcher::ThreadSpecificInstance(thread_safe_sender_) 42 GetDispatcher()->RegisterServiceWorker(pattern, script_url, callbacks);
41 ->RegisterServiceWorker(pattern, script_url, callbacks);
42 } 43 }
43 44
44 void WebServiceWorkerProviderImpl::unregisterServiceWorker( 45 void WebServiceWorkerProviderImpl::unregisterServiceWorker(
45 const WebURL& pattern, 46 const WebURL& pattern,
46 WebServiceWorkerCallbacks* callbacks) { 47 WebServiceWorkerCallbacks* callbacks) {
47 ServiceWorkerDispatcher::ThreadSpecificInstance(thread_safe_sender_) 48 GetDispatcher()->UnregisterServiceWorker(pattern, callbacks);
48 ->UnregisterServiceWorker(pattern, callbacks); 49 }
50
51 ServiceWorkerDispatcher* WebServiceWorkerProviderImpl::GetDispatcher() {
52 return ServiceWorkerDispatcher::ThreadSpecificInstance(thread_safe_sender_);
49 } 53 }
50 54
51 } // namespace content 55 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698