Manage ServiceWorkerDispatcherHost in ServiceWorkerContextCore

content/browser/service_worker/embedded_worker_registry.h

Index: content/browser/service_worker/embedded_worker_registry.h
diff --git a/content/browser/service_worker/embedded_worker_registry.h b/content/browser/service_worker/embedded_worker_registry.h
index 09b925b918007e85c5071d24b25ae609c6098e08..f7631e9b3739e4ce9e1e516c0b76f2afa562dd51 100644
--- a/content/browser/service_worker/embedded_worker_registry.h
+++ b/content/browser/service_worker/embedded_worker_registry.h
@@ -22,7 +22,6 @@ class GURL;
 
 namespace IPC {
 class Message;
-class Sender;
 }
 
 namespace content {
@@ -88,12 +87,8 @@ class CONTENT_EXPORT EmbeddedWorkerRegistry
                               int line_number,
                               const GURL& source_url);
 
-  // Keeps a map from process_id to sender information.
-  void AddChildProcessSender(
-      int process_id,
-      IPC::Sender* sender,
-      MessagePortMessageFilter* message_port_message_filter);
-  void RemoveChildProcessSender(int process_id);
+  // Detaches the process and removes all workers living on the process.
+  void DetachProcess(int process_id);

[falken, 2017/02/08 06:53:51]

Can we have a better name than "Detach". Historically, "Detach" just meant an
emergency measure when "Stop" wasn't working. Should just be "RemoveProcess"?
The comment also doesn't explain well what "detach" means.

[shimazu, 2017/02/13 03:25:56]

Done.

 
   // Returns an embedded worker instance for given |embedded_worker_id|.
   EmbeddedWorkerInstance* GetWorker(int embedded_worker_id);
@@ -112,9 +107,6 @@ class CONTENT_EXPORT EmbeddedWorkerRegistry
                            RemoveWorkerInSharedProcess);
 
   using WorkerInstanceMap = std::map<int, EmbeddedWorkerInstance*>;
-  using ProcessToSenderMap = std::map<int, IPC::Sender*>;
-  using ProcessToMessagePortMessageFilterMap =
-      std::map<int, MessagePortMessageFilter*>;
 
   EmbeddedWorkerRegistry(
       const base::WeakPtr<ServiceWorkerContextCore>& context,
@@ -143,8 +135,6 @@ class CONTENT_EXPORT EmbeddedWorkerRegistry
   base::WeakPtr<ServiceWorkerContextCore> context_;
 
   WorkerInstanceMap worker_map_;
-  ProcessToSenderMap process_sender_map_;
-  ProcessToMessagePortMessageFilterMap process_message_port_message_filter_map_;
 
   // Map from process_id to embedded_worker_id.
   // This map only contains starting and running workers.