Make ServiceWorkerDispatcher reuse existing WebServiceWorkerImpls.

content/child/service_worker/service_worker_dispatcher.h

 // 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.
 
 #ifndef CONTENT_CHILD_SERVICE_WORKER_SERVICE_WORKER_DISPATCHER_H_
 #define CONTENT_CHILD_SERVICE_WORKER_SERVICE_WORKER_DISPATCHER_H_
 
 #include <map>
 #include <vector>
 
@@ skipping 52 matching lines @@
   // (This is attached only to the document thread's ServiceWorkerDispatcher)
   void AddProviderContext(ServiceWorkerProviderContext* provider_context);
   void RemoveProviderContext(ServiceWorkerProviderContext* provider_context);
 
   // Called when navigator.serviceWorker is instantiated or detached
   // for a document whose provider can be identified by |provider_id|.
   void AddScriptClient(int provider_id,
                        blink::WebServiceWorkerProviderClient* client);
   void RemoveScriptClient(int provider_id);
 
+  // If an existing WebServiceWorkerImpl exists for the Service

[michaeln, 2014/06/02 21:11:19]

fyi: eventually, i think we'll have consumers of this class on both the main
thread and on background thread.

+  // Worker, it is returned; otherwise a WebServiceWorkerImpl is
+  // created and its ownership is transferred caller. If

[falken, 2014/06/02 05:40:11]

transferred to the caller

+  // |adopt_handle| is true, a ServiceWorkerHandleReference will be
+  // adopted for the specified Service Worker.
+  //
+  // TODO(dominicc): The lifetime of WebServiceWorkerImpl is too tricky; this

[michaeln, 2014/06/02 21:11:19]

how does the caller know when ownership is transferred vs not?

+  // method can return an existing WebServiceWorkerImpl, in which case
+  // it is owned by a WebCore::ServiceWorker and the lifetime is not
+  // being transferred to the owner; or it can create a
+  // WebServiceWorkerImpl, in which case ownership is transferred to
+  // the caller who must bounce it to a method that will associate it
+  // with a WebCore::ServiceWorker.
+  WebServiceWorkerImpl* GetServiceWorker(const ServiceWorkerObjectInfo&,
+                                         bool adopt_handle);
+
   // |thread_safe_sender| needs to be passed in because if the call leads to
   // construction it will be needed.
   static ServiceWorkerDispatcher* GetOrCreateThreadSpecificInstance(
       ThreadSafeSender* thread_safe_sender);
 
   // Unlike GetOrCreateThreadSpecificInstance() this doesn't create a new
   // instance if thread-local instance doesn't exist.
   static ServiceWorkerDispatcher* GetThreadSpecificInstance();
 
  private:
@@ skipping 44 matching lines @@
   WorkerToProviderMap worker_to_provider_;
 
   scoped_refptr<ThreadSafeSender> thread_safe_sender_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerDispatcher);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_CHILD_SERVICE_WORKER_SERVICE_WORKER_DISPATCHER_H_