More scaffolding, add class ServiceWorkerProviderHost.

content/browser/service_worker/service_worker_context_core.h

Index: content/browser/service_worker/service_worker_context_core.h
diff --git a/content/browser/service_worker/service_worker_context_core.h b/content/browser/service_worker/service_worker_context_core.h
index 3a629770dc95fedb1770929363b825237a5cb571..f428642bb4a4d9a95ffd892b207c30b7d99e5f66 100644
--- a/content/browser/service_worker/service_worker_context_core.h
+++ b/content/browser/service_worker/service_worker_context_core.h
@@ -5,8 +5,11 @@
 #ifndef CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_CONTEXT_CORE_H_
 #define CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_CONTEXT_CORE_H_
 
+#include <map>
+
 #include "base/files/file_path.h"
 #include "base/memory/weak_ptr.h"
+#include "content/browser/service_worker/service_worker_provider_host.h"
 #include "content/common/content_export.h"
 
 namespace base {
@@ -33,9 +36,19 @@ class CONTENT_EXPORT ServiceWorkerContextCore
                            quota::QuotaManagerProxy* quota_manager_proxy);
   ~ServiceWorkerContextCore();
 
+  // ServiceWorkerProviderHosts are reachable thru the context core class.
+  ServiceWorkerProviderHost* GetProviderHost(int process_id, int provider_id);
+  void AttachProviderHostMap(int process_id,
+                             ServiceWorkerProviderHostMap* map);

[alecflett, 2013/11/18 23:40:22]

It seems strange that the dispatcher host is providing (and owning) the map - it
seems like this should all be done within the Context, and perhaps the context
just needs to know of the arrival/destruction of processes.

[michaeln, 2013/11/19 01:14:26]

Seems like 6 vs half-a-dozen?

+  void DetachProviderHostMap(int process_id);
+
+  // Checks the cmdline flag.
   bool IsEnabled();
 
  private:
+  typedef std::map<int, ServiceWorkerProviderHostMap*> ProcessToProvidersMap;
+
+  ProcessToProvidersMap providers_map_;
   scoped_refptr<quota::QuotaManagerProxy> quota_manager_proxy_;
   base::FilePath path_;
 };