ServiceWorker: Use SimpleCache for the script cache

content/browser/appcache/appcache_disk_cache.h

Index: content/browser/appcache/appcache_disk_cache.h
diff --git a/content/browser/appcache/appcache_disk_cache.h b/content/browser/appcache/appcache_disk_cache.h
index 8995e83cf719db91571925f8437a078cbbcf88a1..612c68d8ad0515d93993a78f3f18aec2f7ee872a 100644
--- a/content/browser/appcache/appcache_disk_cache.h
+++ b/content/browser/appcache/appcache_disk_cache.h
@@ -52,6 +52,9 @@ class CONTENT_EXPORT AppCacheDiskCache
                 const net::CompletionCallback& callback) override;
   int DoomEntry(int64 key, const net::CompletionCallback& callback) override;
 
+ protected:
+  explicit AppCacheDiskCache(bool use_simple_cache);
+
  private:
   class CreateBackendCallbackShim;
   class EntryImpl;
@@ -101,7 +104,8 @@ class CONTENT_EXPORT AppCacheDiskCache
   void AddOpenEntry(EntryImpl* entry) { open_entries_.insert(entry); }
   void RemoveOpenEntry(EntryImpl* entry) { open_entries_.erase(entry); }
 
-  bool is_disabled_;
+  bool use_simple_cache_ = false;

[kinuko, 2015/05/15 02:28:48]

nit: we initialize this differently in ctors, it's probably not the best use of
in-class member initialization.

nit: and if that's the case probably we can just leave is_disabled_
initialization in ctors too (it's optional, but having all initialization in one
place is preferred in general)

[nhiroki, 2015/05/15 03:02:47]

Done.

+  bool is_disabled_ = false;
   net::CompletionCallback init_callback_;
   scoped_refptr<CreateBackendCallbackShim> create_backend_callback_;
   PendingCalls pending_calls_;