ServiceWorkerWriteToCacheJob: refactor

content/browser/appcache/appcache_response.h

Index: content/browser/appcache/appcache_response.h
diff --git a/content/browser/appcache/appcache_response.h b/content/browser/appcache/appcache_response.h
index 8d6c79060bdcf2eb76ac5c4870d90fcdc36e089d..b5595209bcff4665761a94265d485c949aa5da23 100644
--- a/content/browser/appcache/appcache_response.h
+++ b/content/browser/appcache/appcache_response.h
@@ -209,8 +209,8 @@ class CONTENT_EXPORT AppCacheResponseWriter
   // negative error code or the number of bytes written. The 'callback' is a
   // required parameter. The contents of 'info_buf' are not modified.
   // Should only be called where there is no Write operation in progress.

[michaeln, 2015/09/10 20:50:27]

please put a comment here that says
// (virtual for testing)
similar to those on ReadInfo and ReadData

-  void WriteInfo(HttpResponseInfoIOBuffer* info_buf,
-                 const net::CompletionCallback& callback);
+  virtual void WriteInfo(HttpResponseInfoIOBuffer* info_buf,
+                         const net::CompletionCallback& callback);
 
   // Writes data to storage. Always returns the result of the write
   // asynchronously through the 'callback'. Returns the number of bytes written
@@ -220,8 +220,9 @@ class CONTENT_EXPORT AppCacheResponseWriter
   // the number of bytes written. The 'callback' is a required parameter.
   // The contents of 'buf' are not modified.
   // Should only be called where there is no Write operation in progress.
-  void WriteData(net::IOBuffer* buf, int buf_len,
-                 const net::CompletionCallback& callback);
+  virtual void WriteData(net::IOBuffer* buf,
+                         int buf_len,
+                         const net::CompletionCallback& callback);
 
   // Returns true if there is a write pending.
   bool IsWritePending() { return IsIOPending(); }