Add checkbox for reporting invalid TLS/SSL cert chains

chrome/browser/interstitials/security_interstitial_page.h

 // Copyright 2014 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_INTERSTITIALS_SECURITY_INTERSTITIAL_PAGE_H_
 #define CHROME_BROWSER_INTERSTITIALS_SECURITY_INTERSTITIAL_PAGE_H_
 
+#include "base/callback.h"
 #include "base/strings/string16.h"
 #include "content/public/browser/interstitial_page_delegate.h"
 #include "url/gurl.h"
 
 namespace base {
 class DictionaryValue;
 }
 
 namespace content {
 class InterstitialPage;
 class WebContents;
 }
 
+namespace interstitials {
+// Constants used to communicate with the JavaScript.
+extern const char kBoxChecked[];
+extern const char kDisplayCheckBox[];
+extern const char kOptInLink[];
+extern const char kPrivacyLinkHtml[];
+}
+
 class SecurityInterstitialPage : public content::InterstitialPageDelegate {
  public:
   SecurityInterstitialPage(content::WebContents* web_contents,
                            const GURL& url);
   ~SecurityInterstitialPage() override;
 
   // Creates an interstitial and shows it.
   virtual void Show();
 
   // Returns interstitial type for testing.
   virtual const void* GetTypeForTesting() const = 0;
 
   // Prevents creating the actual interstitial view for testing.
   void DontCreateViewForTesting();
 
+  // Allows tests to be notified when an invalid cert chain report has
+  // been sent (or not sent).
+  void SetCertificateReportCallbackForTesting(base::Callback<void()>& callback);

[mattm, 2015/03/04 02:58:41]

Shouldn't this (and the member) be on SSLBlockingPage?

[estark, 2015/03/12 22:22:21]

I put them here because in a follow-up CL I'm going to add a similar feature to
captive portal interstitials, which are a subclass of SecurityInterstitialPage.
(In other words, they're going to end up here eventually.)

+
  protected:
   // Returns true if the interstitial should create a new navigation entry.
   virtual bool ShouldCreateNewNavigation() const = 0;
 
   // Populates the strings used to generate the HTML from the template.
   virtual void PopulateInterstitialStrings(
       base::DictionaryValue* load_time_data) = 0;
 
   // InterstitialPageDelegate method:
   std::string GetHTMLContents() override;
 
   // Returns the formatted host name for the request url.
   base::string16 GetFormattedHostName() const;
 
   content::InterstitialPage* interstitial_page() const;
   content::WebContents* web_contents() const;
   GURL request_url() const;
 
+  // Record the user's preference for reporting information about
+  // malware and SSL errors.
+  void SetReportingPreference(bool report);
+
+  // Fills the passed dictionary with the values to be passed to the template
+  // when creating the HTML.
+  virtual void PopulateExtendedReportingOption(
+      base::DictionaryValue* load_time_data) = 0;
+
+  // Returns the boolean value of the given |pref| from the PrefService of the
+  // Profile associated with |web_contents_|.
+  bool IsPrefEnabled(const char* pref);
+
+  // This callback is run when an extended reporting certificate chain
+  // report has been sent, or when it is decided that it should not be
+  // sent (for example, when in incognito mode).
+  base::Callback<void()> certificate_report_callback_for_testing_;

[Bernhard Bauer, 2015/03/03 21:38:06]

This is typedef'd as base::Closure.

[estark, 2015/03/12 22:22:21]

Done.

+
  private:
   content::WebContents* web_contents_;
   const GURL request_url_;
   // Once shown, |interstitial_page| takes ownership of this
   // SecurityInterstitialPage instance.
   content::InterstitialPage* interstitial_page_;
   // Whether the interstitial should create a view.
   bool create_view_;
 
   DISALLOW_COPY_AND_ASSIGN(SecurityInterstitialPage);
 };
 
 #endif  // CHROME_BROWSER_INTERSTITIALS_SECURITY_INTERSTITIAL_PAGE_H_