Fix all failed and canceled URLRequestStatuses without errors.

components/domain_reliability/monitor_unittest.cc

 // Copyright 2014 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 "components/domain_reliability/monitor.h"
 
 #include <map>
 #include <string>
 #include <vector>
 
@@ skipping 321 matching lines @@
 TEST_F(DomainReliabilityMonitorTest, ClearContexts) {
   // Initially the monitor should have just the test context.
   EXPECT_EQ(1u, monitor_.contexts_size_for_testing());
 
   monitor_.ClearBrowsingData(CLEAR_CONTEXTS);
 
   // Clearing contexts should leave the monitor with none.
   EXPECT_EQ(0u, monitor_.contexts_size_for_testing());
 }
 
-// TODO(davidben): When https://crbug.com/490311 is resolved, this test can be
-// removed.
-TEST_F(DomainReliabilityMonitorTest, IgnoreSuccessError) {
-  RequestInfo request = MakeRequestInfo();
-  request.url = GURL("http://example/always_report");
-  request.status = net::URLRequestStatus(net::URLRequestStatus::SUCCESS,
-                                         net::ERR_QUIC_PROTOCOL_ERROR);
-  OnRequestLegComplete(request);
-
-  BeaconVector beacons;
-  context_->GetQueuedBeaconsForTesting(&beacons);
-  EXPECT_EQ(1u, beacons.size());
-  EXPECT_EQ(net::OK, beacons[0].chrome_error);
-
-  EXPECT_TRUE(CheckRequestCounts(kAlwaysReportIndex, 1u, 0u));
-}
-
 TEST_F(DomainReliabilityMonitorTest, WildcardMatchesSelf) {
   DomainReliabilityContext* context = CreateAndAddContext("*.wildcard");
 
   RequestInfo request = MakeRequestInfo();
   request.url = GURL("http://wildcard/always_report");
   OnRequestLegComplete(request);
   EXPECT_TRUE(CheckRequestCounts(context, kAlwaysReportIndex, 1u, 0u));
 }
 
 TEST_F(DomainReliabilityMonitorTest, WildcardMatchesSubdomain) {
@@ skipping 47 matching lines @@
   request.url = GURL("http://test.wildcard/always_report");
   OnRequestLegComplete(request);
 
   EXPECT_TRUE(CheckRequestCounts(context1, kAlwaysReportIndex, 1u, 0u));
   EXPECT_TRUE(CheckRequestCounts(context2, kAlwaysReportIndex, 0u, 0u));
 }
 
 }  // namespace
 
 }  // namespace domain_reliability