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

Unified Diff: content/child/service_worker/web_service_worker_impl.cc

Issue 309503014: Make ServiceWorkerDispatcher reuse existing WebServiceWorkerImpls. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Correct comment. Created 6 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: content/child/service_worker/web_service_worker_impl.cc
diff --git a/content/child/service_worker/web_service_worker_impl.cc b/content/child/service_worker/web_service_worker_impl.cc
index d996dd6e52a11c1a30c5967177c89b96c3cf8742..685c68cb8ab82dd501640d707edc3b2fbfe0899a 100644
--- a/content/child/service_worker/web_service_worker_impl.cc
+++ b/content/child/service_worker/web_service_worker_impl.cc
@@ -20,21 +20,6 @@ using blink::WebString;
namespace content {
WebServiceWorkerImpl::WebServiceWorkerImpl(
- const ServiceWorkerObjectInfo& info,
- ThreadSafeSender* thread_safe_sender)
- : handle_ref_(
- ServiceWorkerHandleReference::CreateForDeleter(info,
- thread_safe_sender)),
- state_(handle_ref_->state()),
- thread_safe_sender_(thread_safe_sender),
- proxy_(NULL) {
- ServiceWorkerDispatcher* dispatcher =
- ServiceWorkerDispatcher::GetThreadSpecificInstance();
- DCHECK(dispatcher);
- dispatcher->AddServiceWorker(handle_ref_->handle_id(), this);
-}
-
-WebServiceWorkerImpl::WebServiceWorkerImpl(
scoped_ptr<ServiceWorkerHandleReference> handle_ref,
ThreadSafeSender* thread_safe_sender)
: handle_ref_(handle_ref.Pass()),
@@ -69,6 +54,10 @@ void WebServiceWorkerImpl::setProxy(blink::WebServiceWorkerProxy* proxy) {
proxy_ = proxy;
}
+blink::WebServiceWorkerProxy* WebServiceWorkerImpl::proxy() {
+ return proxy_;
+}
+
void WebServiceWorkerImpl::proxyReadyChanged() {
if (!proxy_->isReady())
return;

Powered by Google App Engine
This is Rietveld 408576698