* Fix seg fault in deleting view from RemoveTabAtIndex...

views/controls/tabbed_pane/native_tabbed_pane_gtk.cc

 // Copyright (c) 2009 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.
 
 #include "views/controls/tabbed_pane/native_tabbed_pane_gtk.h"
 
 #include <gtk/gtk.h>
 
 #include "app/gfx/canvas.h"
 #include "app/gfx/font.h"
@@ skipping 42 matching lines @@
     // Select the next tab.
     SelectTabAt(index + 1);
   } else {
     // We are the last tab, select the previous one.
     if (index > 0) {
       SelectTabAt(index - 1);
     } else {
       // last tab. nothing to select.
     }
   }
+
+  GtkWidget* page =
+      gtk_notebook_get_nth_page(GTK_NOTEBOOK(native_view()), index);
+  WidgetGtk* widget = WidgetGtk::GetViewForNative(page);
+
+  // detach the content view from widget so that we can delete widget
+  // without destroying the content view.
   View* removed_tab = GetTabViewAt(index);
+  widget->GetRootView()->RemoveChildView(removed_tab);
 
+  // widget delete itself when native_view is deleted.

[sky, 2009/09/09 03:23:15]

nit: delete -> deletes

   gtk_notebook_remove_page(GTK_NOTEBOOK(native_view()), index);
 
   return removed_tab;
 }
 
 void NativeTabbedPaneGtk::SelectTabAt(int index) {
   DCHECK((index >= 0) && (index < GetTabCount()));
   gtk_notebook_set_current_page(GTK_NOTEBOOK(native_view()), index);
 }
 
@@ skipping 91 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 // NativeTabbedPaneWrapper, public:
 
 // static
 NativeTabbedPaneWrapper* NativeTabbedPaneWrapper::CreateNativeWrapper(
     TabbedPane* tabbed_pane) {
   return new NativeTabbedPaneGtk(tabbed_pane);
 }
 
 }  // namespace views