Revert of Send encoded_body_length to renderer when response completed (1/3)

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 185 matching lines @@
       break;
     if (r == MOJO_RESULT_SHOULD_WAIT)
       continue;
     ASSERT_EQ(MOJO_RESULT_OK, r);
     contents += std::string(buffer, read_size);
   }
   std::string expected;
   base::ReadFileToString(
       root.Append(base::FilePath(FILE_PATH_LITERAL("hello.html"))), &expected);
   EXPECT_EQ(expected, contents);
-  EXPECT_EQ(static_cast<int64_t>(expected.size()),
-            client.completion_status().encoded_data_length);
-  EXPECT_EQ(static_cast<int64_t>(expected.size()),
-            client.completion_status().encoded_body_length);
 }
 
 TEST_P(URLLoaderFactoryImplTest, GetFailedResponse) {
   NavigationResourceThrottle::set_ui_checks_always_succeed_for_testing(true);
   mojom::URLLoaderAssociatedPtr loader;
   ResourceRequest request;
   TestURLLoaderClient client;
   net::URLRequestFailedJob::AddUrlHandler();
   request.url = net::URLRequestFailedJob::GetMockHttpUrlWithFailurePhase(
       net::URLRequestFailedJob::START, net::ERR_TIMED_OUT);
   request.method = "GET";
   factory_->CreateLoaderAndStart(
       mojo::GetProxy(&loader, factory_.associated_group()), 2, 1, request,
       client.CreateRemoteAssociatedPtrInfo(factory_.associated_group()));
 
   client.RunUntilComplete();
   ASSERT_FALSE(client.has_received_response());
   ASSERT_FALSE(client.response_body().is_valid());
 
   EXPECT_EQ(net::ERR_TIMED_OUT, client.completion_status().error_code);
-  EXPECT_EQ(0, client.completion_status().encoded_data_length);
-  EXPECT_EQ(0, client.completion_status().encoded_body_length);
-}
-
-// In this case, the loading fails after receiving a response.
-TEST_P(URLLoaderFactoryImplTest, GetFailedResponse2) {
-  NavigationResourceThrottle::set_ui_checks_always_succeed_for_testing(true);
-  mojom::URLLoaderAssociatedPtr loader;
-  ResourceRequest request;
-  TestURLLoaderClient client;
-  net::URLRequestFailedJob::AddUrlHandler();
-  request.url = net::URLRequestFailedJob::GetMockHttpUrlWithFailurePhase(
-      net::URLRequestFailedJob::READ_ASYNC, net::ERR_TIMED_OUT);
-  request.method = "GET";
-  request.is_main_frame = true;
-  factory_->CreateLoaderAndStart(
-      mojo::GetProxy(&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());
-
-  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;
   TestURLLoaderClient client;
   request.url = GURL();
   request.method = "GET";
   ASSERT_FALSE(request.url.is_valid());
@@ skipping 85 matching lines @@
 
   EXPECT_EQ(static_cast<int64_t>(contents.size()),
             client.download_data_length());
   EXPECT_EQ(static_cast<int64_t>(contents.size()),
             client.encoded_download_data_length());
 
   std::string expected;
   base::ReadFileToString(
       root.Append(base::FilePath(FILE_PATH_LITERAL("hello.html"))), &expected);
   EXPECT_EQ(expected, contents);
-  EXPECT_EQ(static_cast<int64_t>(expected.size()),
-            client.completion_status().encoded_data_length);
-  EXPECT_EQ(static_cast<int64_t>(expected.size()),
-            client.completion_status().encoded_body_length);
 }
 
 TEST_P(URLLoaderFactoryImplTest, DownloadToFileFailure) {
   constexpr int32_t kRoutingId = 1;
   constexpr int32_t kRequestId = 2;
 
   mojom::URLLoaderAssociatedPtr loader;
   base::FilePath root;
   PathService::Get(DIR_TEST_DATA, &root);
   net::URLRequestSlowDownloadJob::AddUrlHandler();
@@ skipping 79 matching lines @@
   ASSERT_FALSE(rdh_.GetURLRequest(GlobalRequestID(kChildId, kRequestId)));
 }
 
 INSTANTIATE_TEST_CASE_P(URLLoaderFactoryImplTest,
                         URLLoaderFactoryImplTest,
                         ::testing::Values(128, 32 * 1024));
 
 }  // namespace
 
 }  // namespace content