Move Sad Tab implementation out of the TabContentsViews.

content/browser/tab_contents/tab_contents_view_win.cc

Index: content/browser/tab_contents/tab_contents_view_win.cc
diff --git a/content/browser/tab_contents/tab_contents_view_win.cc b/content/browser/tab_contents/tab_contents_view_win.cc
index 45d4c0f6ffd7f0075fcec0acf893f2a9482a035f..0dc9826fb10b17ff6ccd3c204df217e7a7c5d900 100644
--- a/content/browser/tab_contents/tab_contents_view_win.cc
+++ b/content/browser/tab_contents/tab_contents_view_win.cc
@@ -74,6 +74,8 @@ void TabContentsViewWin::SetPageTitle(const string16& title) {
 
 void TabContentsViewWin::OnTabCrashed(base::TerminationStatus status,
                                       int error_code) {
+  // TODO(avi): No other TCV implementation does anything in this callback. Can
+  // this be moved elsewhere so that |OnTabCrashed| can be removed everywhere?

[jam, 2011/11/14 19:56:28]

perhaps :) I had to add this to avoid dereferencing a null views_ after a tab
crashed. I can look into moving this elsewhere

   view_ = NULL;
 }
 
@@ -147,6 +149,14 @@ void TabContentsViewWin::GetViewBounds(gfx::Rect* out) const {
   *out = gfx::Rect(r);
 }
 
+void TabContentsViewWin::InstallOverlayView(gfx::NativeView view) {
+  NOTREACHED();
+}
+
+void TabContentsViewWin::RemoveOverlayView() {
+  NOTREACHED();
+}
+
 void TabContentsViewWin::CreateNewWindow(
     int route_id,
     const ViewHostMsg_CreateWindow_Params& params) {