Componentize the bad clock blocking page

chrome/browser/interstitials/chrome_controller_client.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_INTERSTITIALS_CHROME_CONTROLLER_CLIENT_H_
 #define CHROME_BROWSER_INTERSTITIALS_CHROME_CONTROLLER_CLIENT_H_
 
 #include "base/macros.h"
 #include "components/security_interstitials/core/controller_client.h"
-
-namespace content {
-class WebContents;
-}
+#include "content/public/browser/interstitial_page.h"
+#include "content/public/browser/web_contents.h"

[estark, 2015/12/01 18:40:17]

can these be left as forward-declarations?

[felt, 2015/12/01 19:13:44]

Gah, I was just working in google3 where forward-declarations aren't allowed,
and I got the style guides mixed up. Will fix here and elsewhere.

 
 // Provides embedder-specific logic for the security error page controller.
 class ChromeControllerClient : public security_interstitials::ControllerClient {
  public:
   explicit ChromeControllerClient(content::WebContents* web_contents);
   ~ChromeControllerClient() override;
 
+  void set_interstitial_page(content::InterstitialPage* interstitial_page);
+
+  // security_interstitials::ControllerClient overrides
+  bool CanLaunchDateAndTimeSettings() override;
+  void LaunchDateAndTimeSettings() override;
+  void GoBack() override;
+
  protected:
   // security_interstitials::ControllerClient overrides
   void OpenUrlInCurrentTab(const GURL& url) override;
   const std::string& GetApplicationLocale() override;
   PrefService* GetPrefService() override;
   const std::string GetExtendedReportingPrefName() override;
 
  private:
   content::WebContents* web_contents_;
+  content::InterstitialPage* interstitial_page_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeControllerClient);
 };
 
 #endif  // CHROME_BROWSER_INTERSTITIALS_CHROME_CONTROLLER_CLIENT_H_