Domain Reliability: Don't use error code unless request failed

components/domain_reliability/monitor.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 "base/command_line.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/single_thread_task_runner.h"
@@ skipping 203 matching lines @@
 void DomainReliabilityMonitor::OnRequestLegComplete(
     const RequestInfo& request) {
   int response_code;
   if (request.response_info.headers)
     response_code = request.response_info.headers->response_code();
   else
     response_code = -1;
   ContextMap::iterator context_it;
   std::string beacon_status;
 
+  int error_code = net::OK;
+  if (request.status.status() == net::URLRequestStatus::FAILED)
+    error_code = request.status.error();
+
   // Ignore requests where:
   // 1. There is no context for the request host.
   // 2. The request did not access the network.
   // 3. The request is not supposed to send cookies (to avoid associating the
   //    request with any potentially unique data in the config).
   // 4. The request was itself a Domain Reliability upload (to avoid loops).
   // 5. There is no defined beacon status for the error or HTTP response code
   //    (to avoid leaking network-local errors).
   if ((context_it = contexts_.find(request.url.host())) == contexts_.end() ||
       !request.AccessedNetwork() ||
       (request.load_flags & net::LOAD_DO_NOT_SEND_COOKIES) ||
       request.is_upload ||
       !GetDomainReliabilityBeaconStatus(
-          request.status.error(), response_code, &beacon_status)) {
+          error_code, response_code, &beacon_status)) {
     return;
   }
 
   DomainReliabilityBeacon beacon;
   beacon.status = beacon_status;
-  beacon.chrome_error = request.status.error();
+  beacon.chrome_error = error_code;
   if (!request.response_info.was_fetched_via_proxy)
     beacon.server_ip = request.response_info.socket_address.host();
   else
     beacon.server_ip.clear();
   beacon.http_response_code = response_code;
   beacon.start_time = request.load_timing_info.request_start;
   beacon.elapsed = time_->NowTicks() - beacon.start_time;
   context_it->second->OnBeacon(request.url, beacon);
 }
 
 }  // namespace domain_reliability