Split cert reporter class into report building/serializing and sending

chrome/browser/ssl/ssl_cert_reporter.h

Index: chrome/browser/ssl/ssl_cert_reporter.h
diff --git a/chrome/browser/ssl/ssl_cert_reporter.h b/chrome/browser/ssl/ssl_cert_reporter.h
index d31c08018cbc19f8ba11feabb4081421cb303fd9..03864aa80aaa1999d8426b08e80c79bbd22efc2e 100644
--- a/chrome/browser/ssl/ssl_cert_reporter.h
+++ b/chrome/browser/ssl/ssl_cert_reporter.h
@@ -17,11 +17,10 @@ class SSLCertReporter {
  public:
   virtual ~SSLCertReporter() {}
 
-  // Send a report for |hostname| with the given |ssl_info| to the
-  // report collection endpoint. |callback| will be run when the report has
-  // been sent off (not necessarily after a response has been received).

[eroman, 2015/05/12 00:27:52]

I assume the reference to |callback| was obsolete? And completion is best-effort

[estark, 2015/05/12 20:42:16]

Yes, that's right.

-  virtual void ReportInvalidCertificateChain(const std::string& hostname,
-                                             const net::SSLInfo& ssl_info) = 0;
+  // Send a serialized certificate report to the report collection

[eroman, 2015/05/12 00:27:52]

Sends

+  // endpoint.
+  virtual void ReportInvalidCertificateChain(
+      const std::string& serialized_report) = 0;
 };
 
 #endif  // CHROME_BROWSER_SSL_SSL_CERT_REPORTER_H_