Plumbs URLRequestContext and CacheBlobStorageContext down to cache.

content/browser/service_worker/service_worker_cache_storage_manager.h

Index: content/browser/service_worker/service_worker_cache_storage_manager.h
diff --git a/content/browser/service_worker/service_worker_cache_storage_manager.h b/content/browser/service_worker/service_worker_cache_storage_manager.h
index d70f08f221067a6c1ff36c9de4f764ef723a4e51..c1fbbb6541fd43e71253c217a3af25267641aaef 100644
--- a/content/browser/service_worker/service_worker_cache_storage_manager.h
+++ b/content/browser/service_worker/service_worker_cache_storage_manager.h
@@ -18,7 +18,12 @@ namespace base {
 class SequencedTaskRunner;
 }  // namespace base
 
+namespace net {
+class URLRequestContext;
+}
+
 namespace content {
+class ChromeBlobStorageContext;
 
 // Keeps track of a ServiceWorkerCacheStorage per origin. There is one
 // ServiceWorkerCacheStorageManager per ServiceWorkerContextCore.
@@ -28,7 +33,9 @@ class CONTENT_EXPORT ServiceWorkerCacheStorageManager {
  public:
   static scoped_ptr<ServiceWorkerCacheStorageManager> Create(
       const base::FilePath& path,
-      base::SequencedTaskRunner* cache_task_runner);
+      base::SequencedTaskRunner* cache_task_runner,
+      net::URLRequestContext* request_context,
+      ChromeBlobStorageContext* blob_storage_context);
 
   static scoped_ptr<ServiceWorkerCacheStorageManager> Create(
       ServiceWorkerCacheStorageManager* old_manager);
@@ -63,13 +70,22 @@ class CONTENT_EXPORT ServiceWorkerCacheStorageManager {
     return cache_task_runner_;
   }
 
+  net::URLRequestContext* url_request_context() const {
+    return request_context_;
+  }
+  ChromeBlobStorageContext* blob_storage_context() const {

[michaeln, 2014/08/12 03:48:52]

until these getters need to be public, can you keep them as private. these are
definitely needed by the impl of this cache backend stuff but i think it should
be more of an imple detail than something with public accessors

[jkarlin, 2014/08/12 18:45:18]

Done.

+    return blob_context_;
+  }
+
  private:
   typedef std::map<GURL, ServiceWorkerCacheStorage*>
       ServiceWorkerCacheStorageMap;
 
   ServiceWorkerCacheStorageManager(
       const base::FilePath& path,
-      base::SequencedTaskRunner* cache_task_runner);
+      base::SequencedTaskRunner* cache_task_runner,
+      net::URLRequestContext* url_request_context,
+      ChromeBlobStorageContext* blob_storage_context);
 
   // The returned ServiceWorkerCacheStorage* is owned by
   // service_worker_cache_storages_.
@@ -83,6 +99,9 @@ class CONTENT_EXPORT ServiceWorkerCacheStorageManager {
   // |cache_task_runner_|.
   ServiceWorkerCacheStorageMap cache_storage_map_;
 
+  net::URLRequestContext* request_context_;
+  ChromeBlobStorageContext* blob_context_;

[michaeln, 2014/08/12 03:48:52]

ditto base::WeakPtr<webkit_blob::BlobStorageContext>

[jkarlin, 2014/08/12 18:45:18]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerCacheStorageManager);
 };