Add response code to the success callback of ReportSender

net/url_request/report_sender.cc

 // Copyright 2015 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 "net/url_request/report_sender.h"
 
 #include <utility>
 
 #include "net/base/elements_upload_data_stream.h"
 #include "net/base/load_flags.h"
 #include "net/base/request_priority.h"
 #include "net/base/upload_bytes_element_reader.h"
+#include "net/http/http_status_code.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_status.h"
 
 namespace {
 const void* const kUserDataKey = &kUserDataKey;
 
 class CallbackInfo : public base::SupportsUserData::Data {
  public:
   CallbackInfo(const net::ReportSender::SuccessCallback& success_callback,
                const net::ReportSender::ErrorCallback& error_callback)
@@ skipping 61 matching lines @@
 
 void ReportSender::OnResponseStarted(URLRequest* request, int net_error) {
   DCHECK_NE(ERR_IO_PENDING, net_error);
 
   CallbackInfo* callback_info =
       static_cast<CallbackInfo*>(request->GetUserData(&kUserDataKey));
   DCHECK(callback_info);
   if (net_error != OK) {
     DVLOG(1) << "Failed to send report for " << request->url().host();
     if (!callback_info->error_callback().is_null())
-      callback_info->error_callback().Run(request->url(), net_error);
-  } else if (!callback_info->success_callback().is_null()) {
-    callback_info->success_callback().Run();
+      callback_info->error_callback().Run(request->url(), net_error, -1);
+  } else if (request->GetResponseCode() != net::HTTP_OK) {
+    if (!callback_info->error_callback().is_null())
+      callback_info->error_callback().Run(request->url(), OK,
+                                          request->GetResponseCode());
+  } else {
+    if (!callback_info->success_callback().is_null())
+      callback_info->success_callback().Run();
   }
-
   CHECK_GT(inflight_requests_.erase(request), 0u);
 }
 
 void ReportSender::OnReadCompleted(URLRequest* request, int bytes_read) {
   NOTREACHED();
 }
 
 }  // namespace net