Plumbs URLRequestContext and CacheBlobStorageContext down to cache.

content/browser/service_worker/service_worker_cache.h

 // Copyright 2014 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_BROWSER_SERVICE_WORKER_SERVICE_WORKER_CACHE_H_
 #define CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_CACHE_H_
 
 #include "base/callback.h"
 #include "base/files/file_path.h"
 
+namespace net {
+class URLRequestContext;
+}
+
 namespace content {
 
+class ChromeBlobStorageContext;
+
 // TODO(jkarlin): Fill this in with a real Cache implementation as
 // specified in
 // https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html.
 // TODO(jkarlin): Unload cache backend from memory once the cache object is no
 // longer referenced in javascript.
 
 // Represents a ServiceWorker Cache as seen in
 // https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html.
 // InitializeIfNeeded must be called before calling the other public members.
 class ServiceWorkerCache {
  public:
-  static ServiceWorkerCache* CreateMemoryCache(const std::string& name);
-  static ServiceWorkerCache* CreatePersistentCache(const base::FilePath& path,
-                                                   const std::string& name);
+  static ServiceWorkerCache* CreateMemoryCache(
+      const std::string& name,
+      net::URLRequestContext* request_context,
+      ChromeBlobStorageContext* blob_context);

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

Since these classes are used on the IO thread, it might make more sense to hang
on to a WeakPtr<webkit_blob::BlobStorageContext> instead of a raw ptr to the
refcounted 'ChromeXxx' class. So you can test for whether its still valid w/o
influencing its lifetime.

+  static ServiceWorkerCache* CreatePersistentCache(
+      const base::FilePath& path,
+      const std::string& name,
+      net::URLRequestContext* request_context,
+      ChromeBlobStorageContext* blob_context);
   virtual ~ServiceWorkerCache();
 
   // Loads the backend and calls the callback with the result (true for
   // success). This must be called before member functions that require a
   // backend are called.
   void CreateBackend(const base::Callback<void(bool)>& callback);
 
   void set_name(const std::string& name) { name_ = name; }
   const std::string& name() const { return name_; }
   int32 id() const { return id_; }
   void set_id(int32 id) { id_ = id; }
 
  private:
-  ServiceWorkerCache(const base::FilePath& path, const std::string& name);
+  ServiceWorkerCache(const base::FilePath& path,
+                     const std::string& name,
+                     net::URLRequestContext* request_context,
+                     ChromeBlobStorageContext* blob_context);
 
   base::FilePath path_;
   std::string name_;
+  net::URLRequestContext* request_context_;

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

always makes me queasy to see the rawptrs, but what's a programmer to do with
this class other than this?

+  ChromeBlobStorageContext* blob_storage_context_;

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

ditto base::WeakPtr<webkit_blob::BlobStorageContext>

   int32 id_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerCache);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_CACHE_H_