[Mojo-Loading] OnStartLoadingResponseBody should be called after OnReceiveResponse

content/browser/loader/mojo_async_resource_handler.h

Index: content/browser/loader/mojo_async_resource_handler.h
diff --git a/content/browser/loader/mojo_async_resource_handler.h b/content/browser/loader/mojo_async_resource_handler.h
index 894ff848c0c1faf04c7c4f1aa750f6de87ec31a2..52c56ab4381d95b1b538653bb21390c5f78cc9e5 100644
--- a/content/browser/loader/mojo_async_resource_handler.h
+++ b/content/browser/loader/mojo_async_resource_handler.h
@@ -18,6 +18,7 @@
 #include "content/common/content_export.h"
 #include "content/common/url_loader.mojom.h"
 #include "mojo/public/cpp/bindings/associated_binding.h"
+#include "mojo/public/cpp/system/data_pipe.h"
 #include "mojo/public/cpp/system/watcher.h"
 #include "net/base/io_buffer.h"
 #include "url/gurl.h"
@@ -119,6 +120,7 @@ class CONTENT_EXPORT MojoAsyncResourceHandler
   size_t buffer_offset_ = 0;
   size_t buffer_bytes_read_ = 0;
   scoped_refptr<SharedWriter> shared_writer_;
+  mojo::ScopedDataPipeConsumerHandle data_pipe_consumer_handle_;

[dcheng, 2017/01/17 23:20:18]

If this data pipe is just for transferring the response body, it might be nice
to give it a less generic name.

[yhirano, 2017/01/18 09:10:45]

Done.

 
   base::WeakPtrFactory<MojoAsyncResourceHandler> weak_factory_;
   DISALLOW_COPY_AND_ASSIGN(MojoAsyncResourceHandler);