Componentize sad_tab

components/sad_tab/sad_tab.h

Index: components/sad_tab/sad_tab.h
diff --git a/chrome/browser/ui/sad_tab.h b/components/sad_tab/sad_tab.h
similarity index 62%
rename from chrome/browser/ui/sad_tab.h
rename to components/sad_tab/sad_tab.h
index e22b14948072cf051583412bce85faf415b8bf81..3de321ed183da5bd2da7c0d08fe483fae0dd5673 100644
--- a/chrome/browser/ui/sad_tab.h
+++ b/components/sad_tab/sad_tab.h
@@ -2,23 +2,27 @@
 // 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);
@@ -32,6 +36,6 @@ class SadTab {
   virtual void Close() = 0;
 };
 
-}  // namespace chrome
+}  // namespace sad_tab
 
-#endif  // CHROME_BROWSER_UI_SAD_TAB_H_
+#endif  // COMPONENTS_SAD_TAB_SAD_TAB_H_