[Mojo-Loading] OnStartLoadingResponseBody should be called after OnReceiveResponse

content/browser/loader/url_loader_factory_impl_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/loader/url_loader_factory_impl.h"
 
 #include <stdint.h>
 
 #include <memory>
 #include <string>
@@ skipping 164 matching lines @@
   ASSERT_TRUE(url_request);
   ResourceRequestInfoImpl* request_info =
       ResourceRequestInfoImpl::ForRequest(url_request);
   ASSERT_TRUE(request_info);
   EXPECT_EQ(kChildId, request_info->GetChildID());
   EXPECT_EQ(kRoutingId, request_info->GetRouteID());
   EXPECT_EQ(kRequestId, request_info->GetRequestID());
 
   ASSERT_FALSE(client.has_received_completion());
 
-  client.RunUntilResponseBodyArrived();
+  client.RunUntilComplete();
   ASSERT_TRUE(client.response_body().is_valid());
-  ASSERT_FALSE(client.has_received_completion());
-
-  client.RunUntilComplete();
+  ASSERT_TRUE(client.has_received_completion());
 
   EXPECT_EQ(200, client.response_head().headers->response_code());
   std::string content_type;
   client.response_head().headers->GetNormalizedHeader("content-type",
                                                       &content_type);
   EXPECT_EQ("text/html", content_type);
   EXPECT_EQ(0, client.completion_status().error_code);
 
   std::string contents;
   while (true) {
@@ skipping 66 matching lines @@
   // must be blob scheme.
   request.resource_type = RESOURCE_TYPE_XHR;
   // Need to set |request_initiator| for non main frame type request.
   request.request_initiator = url::Origin();
   factory_->CreateLoaderAndStart(
       mojo::MakeRequest(&loader, factory_.associated_group()), 2, 1, request,
       client.CreateRemoteAssociatedPtrInfo(factory_.associated_group()));
 
   client.RunUntilComplete();
   ASSERT_FALSE(client.has_received_response());
-  ASSERT_TRUE(client.response_body().is_valid());
+  ASSERT_FALSE(client.response_body().is_valid());
 
   EXPECT_EQ(net::ERR_TIMED_OUT, client.completion_status().error_code);
   EXPECT_GT(client.completion_status().encoded_data_length, 0);
   EXPECT_EQ(0, client.completion_status().encoded_body_length);
 }
 
 // This test tests a case where resource loading is cancelled before started.
 TEST_P(URLLoaderFactoryImplTest, InvalidURL) {
   mojom::URLLoaderAssociatedPtr loader;
   ResourceRequest request;
@@ skipping 272 matching lines @@
   ASSERT_FALSE(rdh_.GetURLRequest(GlobalRequestID(kChildId, kRequestId)));
 }
 
 INSTANTIATE_TEST_CASE_P(URLLoaderFactoryImplTest,
                         URLLoaderFactoryImplTest,
                         ::testing::Values(128, 32 * 1024));
 
 }  // namespace
 
 }  // namespace content