Componentize sad_tab

components/sad_tab/sad_tab.h

 // Copyright 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_UI_SAD_TAB_H_
-#define CHROME_BROWSER_UI_SAD_TAB_H_
+#ifndef COMPONENTS_SAD_TAB_SAD_TAB_H_
+#define COMPONENTS_SAD_TAB_SAD_TAB_H_
 
 #include "base/process/kill.h"
-#include "chrome/browser/ui/sad_tab_types.h"
+#include "components/sad_tab/sad_tab_types.h"
 
 namespace content {
 class WebContents;
 }
 
-namespace chrome {
+namespace sad_tab {
+
+class SadTabClient;
 
 // Cross-platform interface to show the Sad tab UI.
 class SadTab {
  public:
   // Factory function to create the platform specific implementations.
-  static SadTab* Create(content::WebContents* web_contents, SadTabKind kind);
+  static SadTab* Create(content::WebContents* web_contents,
+                        SadTabKind kind,
+                        SadTabClient* client);
 

[sadrul, 2014/09/22 16:48:22]

This doesn't seem to exist/be necessary in this patchset, since SadTab is
created through the SadTabClient? (see other comments about this)

   // Returns true if the sad tab should be shown.
   static bool ShouldShow(base::TerminationStatus status);
 
   virtual ~SadTab() {}
 
   // Shows the Sad tab.
   virtual void Show() = 0;
 
   // Closes the Sad tab.
   virtual void Close() = 0;
 };
 
-}  // namespace chrome
+}  // namespace sad_tab
 
-#endif  // CHROME_BROWSER_UI_SAD_TAB_H_
+#endif  // COMPONENTS_SAD_TAB_SAD_TAB_H_