Add response code to the success callback of ReportSender

net/http/transport_security_state.h

 // 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.
 
 #ifndef NET_HTTP_TRANSPORT_SECURITY_STATE_H_
 #define NET_HTTP_TRANSPORT_SECURITY_STATE_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 266 matching lines @@
     // True if subdomains are subject to this policy
     bool include_subdomains;
   };
 
   // An interface for asynchronously sending HPKP violation reports.
   class NET_EXPORT ReportSenderInterface {
    public:
     // Sends the given serialized |report| to |report_uri| with
     // Content-Type header as specified in
     // |content_type|. |content_type| should be non-empty.
-    // |report_id| could be any non-negative integer. It's passed back to the
-    // error or success callbacks.
-    virtual void Send(
-        const GURL& report_uri,
-        base::StringPiece content_type,
-        base::StringPiece report,
-        const base::Callback<void()>& success_callback,
-        const base::Callback<void(const GURL&, int)>& error_callback) = 0;
+    // |success_callback| is called iff an HTTP 200 response is received.
+    // |error_callback| is called in all other cases. Error callback's
+    // |net_error| can be net::OK if the upload was successful but the server
+    // returned a non-HTTP 200 |http_response_code|. In all other cases,
+    // error callback's |http_response_code| is -1.
+    virtual void Send(const GURL& report_uri,
+                      base::StringPiece content_type,
+                      base::StringPiece report,
+                      const base::Callback<void()>& success_callback,
+                      const base::Callback<void(const GURL&,
+                                                int /* net_error */,
+                                                int /* http_response_code */)>&
+                          error_callback) = 0;
 
    protected:
     virtual ~ReportSenderInterface() {}
   };
 
   // An interface for building and asynchronously sending reports when a
   // site expects valid Certificate Transparency information but it
   // wasn't supplied.
   class NET_EXPORT ExpectCTReporter {
    public:
@@ skipping 343 matching lines @@
   // rate-limiting.
   ExpiringCache<std::string, bool, base::TimeTicks, std::less<base::TimeTicks>>
       sent_reports_cache_;
 
   DISALLOW_COPY_AND_ASSIGN(TransportSecurityState);
 };
 
 }  // namespace net
 
 #endif  // NET_HTTP_TRANSPORT_SECURITY_STATE_H_