net: Move write flush responsiblity from URLFetcherResponseWriter to URLFetcherCore

net/url_request/url_fetcher_core.h

Index: net/url_request/url_fetcher_core.h
diff --git a/net/url_request/url_fetcher_core.h b/net/url_request/url_fetcher_core.h
index 5ccd3ecda1c809738bc702d5c512050381a40214..7d81475b4114194eab8f906891a8976e206e6db0 100644
--- a/net/url_request/url_fetcher_core.h
+++ b/net/url_request/url_fetcher_core.h
@@ -28,6 +28,7 @@ class SingleThreadTaskRunner;
 }  // namespace base
 
 namespace net {
+class DrainableIOBuffer;
 class HttpResponseHeaders;
 class IOBuffer;
 class URLFetcherDelegate;
@@ -193,8 +194,12 @@ class URLFetcherCore
   void CompleteAddingUploadDataChunk(const std::string& data,
                                      bool is_last_chunk);
 
-  // Handles the result of WriteBuffer.
-  void DidWriteBuffer(int result);
+  // Writes all bytes stored in |data| with |response_writer_|.
+  // Returns OK when all bytes in |data| gets written successfully.

[wtc, 2013/05/23 17:22:53]

Nit: when all bytes in |data| gets written successfully =>
       if all bytes in |data| get written synchronously

I think it's also helpful to add:

  Otherwise, returns ERR_IO_PENDING or a network error code.

[hashimoto, 2013/05/24 02:29:28]

Thanks, done.

+  int WriteBuffer(scoped_refptr<DrainableIOBuffer> data);
+
+  // Used to implement WriteBuffer().
+  void DidWriteBuffer(scoped_refptr<DrainableIOBuffer> data, int result);
 
   // Read response bytes from the request.
   void ReadResponse();