Componentize the bad clock blocking page

chrome/browser/ssl/bad_clock_blocking_page.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_BAD_CLOCK_BLOCKING_PAGE_H_
 #define CHROME_BROWSER_SSL_BAD_CLOCK_BLOCKING_PAGE_H_
 
 #include <string>
 
 #include "base/callback.h"
 #include "base/time/time.h"
 #include "chrome/browser/interstitials/security_interstitial_page.h"
 #include "chrome/browser/ssl/ssl_cert_reporter.h"
+#include "components/security_interstitials/core/bad_clock_ui.h"

[estark, 2015/12/01 21:41:51]

forward declare?

[felt, 2015/12/01 22:59:23]

Done.

 #include "net/ssl/ssl_info.h"
 
 class CertReportHelper;
 class GURL;
 
-// This class is responsible for showing/hiding the interstitial page that is
-// shown when a certificate error happens. This error is not overridable.
-// It deletes itself when the interstitial page is closed.
+// This class is responsible for showing/hiding the interstitial page that
+// occurs when an SSL error is triggered by a clock misconfiguration. It
+// creates the UI using security_interstitials::BadClockUI and then
+// displays it. It deletes itself when the interstitial page is closed.
 class BadClockBlockingPage : public SecurityInterstitialPage {
  public:
   // Interstitial type, used in tests.
   static InterstitialPageDelegate::TypeID kTypeForTesting;
 
-  // Creates a bad clock interstitial. If the blocking page isn't shown, the
-  // caller is responsible for cleaning up the blocking page, otherwise the
-  // interstitial takes ownership when shown.
+  // If the blocking page isn't shown, the caller is responsible for cleaning
+  // up the blocking page. Otherwise, the interstitial takes ownership when
+  // shown.
   BadClockBlockingPage(content::WebContents* web_contents,
                        int cert_error,
                        const net::SSLInfo& ssl_info,
                        const GURL& request_url,
                        const base::Time& time_triggered,
                        scoped_ptr<SSLCertReporter> ssl_cert_reporter,
                        const base::Callback<void(bool)>& callback);
 
   ~BadClockBlockingPage() override;
 
   // InterstitialPageDelegate method:
   InterstitialPageDelegate::TypeID GetTypeForTesting() const override;
 
   void SetSSLCertReporterForTesting(
       scoped_ptr<SSLCertReporter> ssl_cert_reporter);
 
  protected:
-  // InterstitialPageDelegate implementation.
+  // InterstitialPageDelegate implementation:
   void CommandReceived(const std::string& command) override;
   void OverrideEntry(content::NavigationEntry* entry) override;
   void OverrideRendererPrefs(content::RendererPreferences* prefs) override;
   void OnDontProceed() override;
 
   // SecurityInterstitialPage implementation:
   bool ShouldCreateNewNavigation() const override;
   void PopulateInterstitialStrings(
       base::DictionaryValue* load_time_data) override;
+  void AfterShow() override;
 
  private:
   void NotifyDenyCertificate();
 
   base::Callback<void(bool)> callback_;
   const int cert_error_;
   const net::SSLInfo ssl_info_;
+  const base::Time time_triggered_;
+  scoped_ptr<ChromeControllerClient> controller_;
 
-  // The time at which the interstitial was triggered. The interstitial
-  // calculates all times relative to this.
-  const base::Time time_triggered_;
-
+  scoped_ptr<security_interstitials::BadClockUI> bad_clock_ui_;
   scoped_ptr<CertReportHelper> cert_report_helper_;
 
   DISALLOW_COPY_AND_ASSIGN(BadClockBlockingPage);
 };
 
 #endif  // CHROME_BROWSER_SSL_BAD_CLOCK_BLOCKING_PAGE_H_