URLRequestJob: change ReadRawData contract

content/browser/service_worker/service_worker_write_to_cache_job.h

Index: content/browser/service_worker/service_worker_write_to_cache_job.h
diff --git a/content/browser/service_worker/service_worker_write_to_cache_job.h b/content/browser/service_worker/service_worker_write_to_cache_job.h
index deee1070fd9ce8a3c85077ba86fb01a989b3429f..b41fe389c4b61416ad38fe71ad2257906433c7c3 100644
--- a/content/browser/service_worker/service_worker_write_to_cache_job.h
+++ b/content/browser/service_worker/service_worker_write_to_cache_job.h
@@ -71,7 +71,8 @@ class CONTENT_EXPORT ServiceWorkerWriteToCacheJob
   void GetResponseInfo(net::HttpResponseInfo* info) override;
   int GetResponseCode() const override;
   void SetExtraRequestHeaders(const net::HttpRequestHeaders& headers) override;
-  bool ReadRawData(net::IOBuffer* buf, int buf_size, int* bytes_read) override;
+  int ReadRawData(net::IOBuffer* buf, int buf_size) override;
+  void NotifyReadComplete(int bytes_read);

[Randy Smith (Not in Mondays), 2015/10/22 20:38:45]

I can't find this function anywhere in the .cc file; maybe a leftover bit of
pre-merge cruft?  (If it is used, I'm probably going to want a different name,
but I'm hoping that's moot.)

[xunjieli, 2015/10/23 13:43:08]

Done.

 
   const net::HttpResponseInfo* http_info() const;
 
@@ -114,10 +115,10 @@ class CONTENT_EXPORT ServiceWorkerWriteToCacheJob
   // status to IO_PENDING. After this function returns, if the URLRequestJob
   // isn't IO_PENDING, all of the data in |io_buffer_| has been written back to
   // the script cache if necessary.
-  void HandleNetData(int bytes_read);
+  int HandleNetData(int bytes_read);
 
-  void NotifyDoneHelper(const net::URLRequestStatus& status,
-                        const std::string& status_message);
+  void NotifyStartErrorHelper(const net::URLRequestStatus& status,
+                              const std::string& status_message);
 
   void NotifyFinishedCaching(net::URLRequestStatus status,
                              const std::string& status_message);
@@ -140,6 +141,7 @@ class CONTENT_EXPORT ServiceWorkerWriteToCacheJob
   bool has_been_killed_;
   bool did_notify_started_;
   bool did_notify_finished_;
+
   base::WeakPtrFactory<ServiceWorkerWriteToCacheJob> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerWriteToCacheJob);