Refactor ChromeFraudulentCertReporter for code reuse by SSL reporting

chrome/browser/ssl/ssl_interstitial_certificate_reporter.h

+// 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.
+
+#ifndef CHROME_BROWSER_SSL_SSL_INTERSTITIAL_CERTIFICATE_REPORTER_H_
+#define CHROME_BROWSER_SSL_SSL_INTERSTITIAL_CERTIFICATE_REPORTER_H_
+
+#include <string>
+
+#include "base/logging.h"
+#include "chrome/browser/net/chrome_certificate_reporter.h"
+
+namespace net {
+class SSLInfo;
+class URLRequestContext;
+}
+
+// This class builds and sends certiifcate reports from SSL blocking
+// page interstitials.
+class SSLInterstitialCertificateReporter

[felt, 2015/03/04 19:15:47]

Small detail, but I'm not sure if this is the best name for it. The thing that
really distinguishes this from the pinning violation report is the opt-in (and
therefore the expanded nature of the reports), not where it's invoked from. It
might be more appropriate to name it ExtendedCertificateReporter,
OptionalCertificateReporter, or something else.

[estark, 2015/03/04 22:11:57]

This class is gone now (see previous comment).

+    : public chrome_browser_net::ChromeCertificateReporter {
+ public:
+  explicit SSLInterstitialCertificateReporter(
+      net::URLRequestContext* request_context);
+  ~SSLInterstitialCertificateReporter() {}
+
+  // chrome_browser_net::ChromeCertificateReporter implementation
+  virtual void SendReport(const std::string& hostname,
+                          const net::SSLInfo& ssl_info);
+
+  DISALLOW_COPY_AND_ASSIGN(SSLInterstitialCertificateReporter);
+};
+
+#endif  // CHROME_BROWSER_SSL_SSL_INTERSTITIAL_CERTIFICATE_REPORTER_H_