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);

[kinuko, 2013/11/18 05:57:24]

nit: could this be const ptr ?

[michaeln, 2013/11/18 20:18:17]

Could be? Is the contract your suggesting that this class never alter the
'providerhosts'? I'm not sure that's a good contract. I think having
GetProviderHost return a const ptr would not be helpful as i expect the caller
of that method will want/need to invoke non-const methods.

[kinuko, 2013/11/19 00:55:56]

Does this class alter the given map?  Probably you're talking about this class
(or this class's customers) going to call non-const methods of providerhost's
while I was concerned if this class alters the map?

+  void DetachProviderHostMap(int process_id);
+
+  // Checks the cmdline flag.
   bool IsEnabled();
 
  private:
+  typedef std::map<int, ServiceWorkerProviderHostMap*> ProcessToProvidersMap;

[kinuko, 2013/11/18 05:57:24]

ditto

+
+  ProcessToProvidersMap providers_map_;
   scoped_refptr<quota::QuotaManagerProxy> quota_manager_proxy_;
   base::FilePath path_;
 };