Add response code to the success callback of ReportSender

chrome/browser/safe_browsing/certificate_reporting_service.h

 // 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.
 
 #ifndef CHROME_BROWSER_SAFE_BROWSING_CERTIFICATE_REPORTING_SERVICE_H_
 #define CHROME_BROWSER_SAFE_BROWSING_CERTIFICATE_REPORTING_SERVICE_H_
 
 #include <map>
 #include <memory>
 #include <string>
@@ skipping 108 matching lines @@
     // provided in the constructor. Failed reports will be added to the retry
     // list.
     void SendPending();
 
     // Getter and setters for testing:
     size_t inflight_report_count_for_testing() const;
     BoundedReportList* GetQueueForTesting() const;
 
    private:
     void SendInternal(const Report& report);
-    void ErrorCallback(int report_id, const GURL& url, int error);
+    void ErrorCallback(int report_id,
+                       const GURL& url,
+                       int error,

[eroman, 2017/04/26 22:52:28]

I suggested updating this to "net_error" to make it clear what the domain is.

[meacer, 2017/04/27 00:56:41]

Done.

+                       int http_response_code);
     void SuccessCallback(int report_id);
 
     std::unique_ptr<certificate_reporting::ErrorReporter> error_reporter_;
     std::unique_ptr<BoundedReportList> retry_list_;
     base::Clock* const clock_;
     // Maximum age of a queued report. Reports older than this are discarded in
     // the next SendPending() call.
     const base::TimeDelta report_ttl_;
     const bool retries_enabled_;
     // Current report id, starting from zero and monotonically incrementing.
@@ skipping 93 matching lines @@
   base::Callback<void()> reset_callback_;
 
   // Encryption parameters.
   uint8_t* server_public_key_;
   uint32_t server_public_key_version_;
 
   DISALLOW_COPY_AND_ASSIGN(CertificateReportingService);
 };
 
 #endif  // CHROME_BROWSER_SAFE_BROWSING_CERTIFICATE_REPORTING_SERVICE_H_