Cleanup: Refactor ManagedModeInterstitial to not "delete this" in the ctor.

chrome/browser/managed_mode/managed_mode_interstitial.h

Index: chrome/browser/managed_mode/managed_mode_interstitial.h
diff --git a/chrome/browser/managed_mode/managed_mode_interstitial.h b/chrome/browser/managed_mode/managed_mode_interstitial.h
index 06027d6df424968e0c8374b4df343c10682dfeb1..5a1ead16721a98034f9ede81177a3a1af432a215 100644
--- a/chrome/browser/managed_mode/managed_mode_interstitial.h
+++ b/chrome/browser/managed_mode/managed_mode_interstitial.h
@@ -23,12 +23,19 @@ class WebContents;
 // blocked page, to decide later whether to manually allow it.
 class ManagedModeInterstitial : public content::InterstitialPageDelegate {
  public:
+  virtual ~ManagedModeInterstitial();

[Bernhard Bauer, 2014/04/25 11:10:18]

The destructor doesn't need to be public either. No one should ever get a
pointer to this class :)

[Marc Treib, 2014/04/25 12:02:19]

Done. Good point!

+
+  static void Show(content::WebContents* web_contents,
+                   const GURL& url,
+                   const base::Callback<void(bool)>& callback);
+
+ private:
   ManagedModeInterstitial(content::WebContents* web_contents,
                           const GURL& url,
                           const base::Callback<void(bool)>& callback);
-  virtual ~ManagedModeInterstitial();
 
- private:
+  bool Init();
+
   // InterstitialPageDelegate implementation.
   virtual std::string GetHTMLContents() OVERRIDE;
   virtual void CommandReceived(const std::string& command) OVERRIDE;