Add checkbox for reporting invalid TLS/SSL cert chains

chrome/browser/safe_browsing/ping_manager.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 CHROME_BROWSER_SAFE_BROWSING_PING_MANAGER_H_
 #define CHROME_BROWSER_SAFE_BROWSING_PING_MANAGER_H_
 
 // A class that reports safebrowsing statistics to Google's SafeBrowsing
 // servers.
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/gtest_prod_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/safe_browsing/protocol_manager_helper.h"
 #include "chrome/browser/safe_browsing/safe_browsing_util.h"
 #include "net/url_request/url_fetcher_delegate.h"
 #include "url/gurl.h"
 
+namespace chrome_browser_net {
+class CertificateErrorReporter;
+}
+
 namespace net {
+class SSLInfo;
 class URLRequestContextGetter;
 }  // namespace net
 
 
 class SafeBrowsingPingManager : public net::URLFetcherDelegate {
  public:
   ~SafeBrowsingPingManager() override;
 
   // Create an instance of the safe browsing ping manager.
   static SafeBrowsingPingManager* Create(
@@ skipping 10 matching lines @@
                              const GURL& page_url,
                              const GURL& referrer_url,
                              bool is_subresource,
                              SBThreatType threat_type,
                              const std::string& post_data);
 
   // Users can opt-in on the SafeBrowsing interstitial to send detailed
   // malware reports. |report| is the serialized report.
   void ReportMalwareDetails(const std::string& report);
 
+  // Users can opt-in on the SSL interstitial to send reports of invalid
+  // certificate chains.
+  void ReportInvalidCertificateChain(const std::string& hostname,
+                                     const net::SSLInfo& ssl_info);
+
+  void SetCertificateErrorReporterForTesting(
+      chrome_browser_net::CertificateErrorReporter* certificate_error_reporter);

[mattm, 2015/03/23 05:31:17]

should be passed as a scoped_ptr since it takes ownership

[estark, 2015/03/23 16:42:13]

Done.

+
  private:
   FRIEND_TEST_ALL_PREFIXES(SafeBrowsingPingManagerTest,
                            TestSafeBrowsingHitUrl);
   FRIEND_TEST_ALL_PREFIXES(SafeBrowsingPingManagerTest,
                            TestMalwareDetailsUrl);
 
   typedef std::set<const net::URLFetcher*> Reports;
 
   // Constructs a SafeBrowsingPingManager that issues network requests
   // using |request_context_getter|.
@@ skipping 19 matching lines @@
   scoped_refptr<net::URLRequestContextGetter> request_context_getter_;
 
   // URL prefix where browser reports hits to the safebrowsing list and
   // sends detaild malware reports for UMA users.
   std::string url_prefix_;
 
   // Track outstanding SafeBrowsing report fetchers for clean up.
   // We add both "hit" and "detail" fetchers in this set.
   Reports safebrowsing_reports_;
 
+  // Sends reports of invalid SSL certificate chains.
+  scoped_ptr<chrome_browser_net::CertificateErrorReporter>
+      certificate_error_reporter_;
+
   DISALLOW_COPY_AND_ASSIGN(SafeBrowsingPingManager);
 };
 
 #endif  // CHROME_BROWSER_SAFE_BROWSING_PING_MANAGER_H_