Ignore Content-Length mismatches when Content-Length is too large.

net/url_request/url_request_unittest.cc

 // Copyright (c) 2012 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 "build/build_config.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shlobj.h>
 #endif
@@ skipping 1153 matching lines @@
   ASSERT_TRUE(test_server_.Start());
 
   // Parameter that specifies the Content-Length field in the response:
   // C - Compressed length.
   // U - Uncompressed length.
   // L - Large length (larger than both C & U).
   // M - Medium length (between C & U).
   // S - Small length (smaller than both C & U).
   const char test_parameters[] = "CULMS";
   const int num_tests = arraysize(test_parameters)- 1;  // Skip NULL.
-  // C & U should be OK.
-  // L & M are larger than the data sent, and show an error.
-  // S has too little data, but we seem to accept it.
-  const bool test_expect_success[num_tests] =
-      { true, true, false, false, true };
 
   for (int i = 0; i < num_tests ; i++) {
     TestDelegate d;
     {
       std::string test_file =
           base::StringPrintf("compressedfiles/BullRunSpeech.txt?%c",
                              test_parameters[i]);
 
       TestNetworkDelegate network_delegate;  // must outlive URLRequest
       scoped_refptr<TestURLRequestContext> context(
           new TestURLRequestContext(true));
       context->set_network_delegate(&network_delegate);
       context->Init();
 
       TestURLRequest r(test_server_.GetURL(test_file), &d);
       r.set_context(context);
       r.Start();
       EXPECT_TRUE(r.is_pending());
 
       MessageLoop::current()->Run();
 
       EXPECT_EQ(1, d.response_started_count());
       EXPECT_FALSE(d.received_data_before_response());
       VLOG(1) << " Received " << d.bytes_received() << " bytes"
               << " status = " << r.status().status()
               << " error = " << r.status().error();
-      if (test_expect_success[i]) {
-        EXPECT_EQ(URLRequestStatus::SUCCESS, r.status().status())
-            << " Parameter = \"" << test_file << "\"";
-      } else {
-        EXPECT_EQ(URLRequestStatus::FAILED, r.status().status());
-        EXPECT_EQ(-100, r.status().error())
-            << " Parameter = \"" << test_file << "\"";
-      }
+      EXPECT_EQ(URLRequestStatus::SUCCESS, r.status().status())
+          << " Parameter = \"" << test_file << "\"";
     }
   }
 }
 
 // This test was disabled because it made chrome_frame_net_tests hang
 // (see bug 102991).
 TEST_F(URLRequestTestHTTP, DISABLED_HTTPSToHTTPRedirectNoRefererTest) {
   ASSERT_TRUE(test_server_.Start());
 
   TestServer https_test_server(TestServer::TYPE_HTTPS,
@@ skipping 3292 matching lines @@
   req.SetExtraRequestHeaders(headers);
   req.Start();
   MessageLoop::current()->Run();
   // If the net tests are being run with ChromeFrame then we need to allow for
   // the 'chromeframe' suffix which is added to the user agent before the
   // closing parentheses.
   EXPECT_TRUE(StartsWithASCII(d.data_received(), "Lynx (textmode", true));
 }
 
 }  // namespace net