ServiceWorkerWriteToCacheJob: refactor

content/browser/service_worker/service_worker_disk_cache.h

Index: content/browser/service_worker/service_worker_disk_cache.h
diff --git a/content/browser/service_worker/service_worker_disk_cache.h b/content/browser/service_worker/service_worker_disk_cache.h
index 19cb531fd2b0d469d6cfc4f7aeb8b3197b0c30ee..3e6caec6ff225008c130bd1b4a5fbe44c1a98557 100644
--- a/content/browser/service_worker/service_worker_disk_cache.h
+++ b/content/browser/service_worker/service_worker_disk_cache.h
@@ -32,6 +32,7 @@ class CONTENT_EXPORT ServiceWorkerResponseReader
     : public AppCacheResponseReader {
  protected:
   // Should only be constructed by the storage class.
+  friend class MockServiceWorkerResponseReader;
   friend class ServiceWorkerDiskCacheMigrator;
   friend class ServiceWorkerDiskCacheMigratorTest;
   friend class ServiceWorkerStorage;
@@ -44,6 +45,7 @@ class CONTENT_EXPORT ServiceWorkerResponseWriter
     : public AppCacheResponseWriter {
  protected:
   // Should only be constructed by the storage class.
+  friend class MockServiceWorkerResponseWriter;

[Randy Smith (Not in Mondays), 2015/09/13 21:32:07]

I'm confused.  What are the friend decls for?  There doesn't seem to be anything
private about this class that the friend decl gives "Mock*" access to.

[Elly Fong-Jones, 2015/09/15 15:08:24]

These are left over from earlier. Deleted.

   friend class ServiceWorkerDiskCacheMigrator;
   friend class ServiceWorkerDiskCacheMigratorTest;
   friend class ServiceWorkerStorage;