Revert r51526 (Auto-size the views version of the options dialog.)

Unexpected unit_tests crash on chromeos.

TBR: mnissler

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51529 0039d316-1c4b-4281-b951-d872f2087c98

23 files changed, 81 insertions, 361 deletions

diff --git a/views/controls/button/native_button_win.cc b/views/controls/button/native_button_win.cc
index 9e24fc3..c485c71 100644
--- a/views/controls/button/native_button_win.cc
+++ b/views/controls/button/native_button_win.cc
@@ -19,8 +19,7 @@ namespace views {
 // NativeButtonWin, public:
 
 NativeButtonWin::NativeButtonWin(NativeButton* native_button)
-    : native_button_(native_button),
-      button_size_valid_(false) {
+    : native_button_(native_button) {
   // Associates the actual HWND with the native_button so the native_button is
   // the one considered as having the focus (not the wrapper) when the HWND is
   // focused directly (with a click for example).
@@ -44,14 +43,12 @@ void NativeButtonWin::UpdateLabel() {
   }
 
   SetWindowText(native_view(), native_button_->label().c_str());
-  button_size_valid_ = false;
 }
 
 void NativeButtonWin::UpdateFont() {
   SendMessage(native_view(), WM_SETFONT,
               reinterpret_cast<WPARAM>(native_button_->font().hfont()),
               FALSE);
-  button_size_valid_ = false;
 }
 
 void NativeButtonWin::UpdateEnabled() {
@@ -63,7 +60,6 @@ void NativeButtonWin::UpdateDefault() {
     SendMessage(native_view(), BM_SETSTYLE,
                 native_button_->is_default() ? BS_DEFPUSHBUTTON : BS_PUSHBUTTON,
                 true);
-    button_size_valid_ = false;
   }
 }
 
@@ -92,13 +88,10 @@ gfx::NativeView NativeButtonWin::GetTestingHandle() const {
 // NativeButtonWin, View overrides:
 
 gfx::Size NativeButtonWin::GetPreferredSize() {
-  if (!button_size_valid_) {
-    SIZE sz = {0};
-    Button_GetIdealSize(native_view(), reinterpret_cast<LPARAM>(&sz));
-    button_size_.SetSize(sz.cx, sz.cy);
-    button_size_valid_ = true;
-  }
-  return button_size_;
+  SIZE sz = {0};
+  SendMessage(native_view(), BCM_GETIDEALSIZE, 0, reinterpret_cast<LPARAM>(&sz));
+
+  return gfx::Size(sz.cx, sz.cy);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
diff --git a/views/controls/button/native_button_win.h b/views/controls/button/native_button_win.h
index 95b8a5ac..20ce575 100644
--- a/views/controls/button/native_button_win.h
+++ b/views/controls/button/native_button_win.h
@@ -1,6 +1,6 @@
-// Copyright (c) 2010 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.
+// 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.
 
 #ifndef VIEWS_CONTROLS_BUTTON_NATIVE_BUTTON_WIN_H_
 #define VIEWS_CONTROLS_BUTTON_NATIVE_BUTTON_WIN_H_
@@ -49,10 +49,6 @@ class NativeButtonWin : public NativeControlWin,
   // The NativeButton we are bound to.
   NativeButton* native_button_;
 
-  // It's expensive to find the size of a button on windows, so we cache it.
-  mutable gfx::Size button_size_;
-  mutable bool button_size_valid_;
-
   DISALLOW_COPY_AND_ASSIGN(NativeButtonWin);
 };
 
diff --git a/views/controls/menu/menu_scroll_view_container.cc b/views/controls/menu/menu_scroll_view_container.cc
index 532366c..cb92284 100644
--- a/views/controls/menu/menu_scroll_view_container.cc
+++ b/views/controls/menu/menu_scroll_view_container.cc
@@ -257,7 +257,6 @@ void MenuScrollViewContainer::DidChangeBounds(const gfx::Rect& previous,
   gfx::Size content_pref = scroll_view_->GetContents()->GetPreferredSize();
   scroll_up_button_->SetVisible(content_pref.height() > height());
   scroll_down_button_->SetVisible(content_pref.height() > height());
-  Layout();
 }
 
 gfx::Size MenuScrollViewContainer::GetPreferredSize() {
diff --git a/views/controls/native_control.cc b/views/controls/native_control.cc
index 0792b29..f714382 100644
--- a/views/controls/native_control.cc
+++ b/views/controls/native_control.cc
@@ -201,7 +201,7 @@ void NativeControl::ValidateNativeControl() {
 
 void NativeControl::ViewHierarchyChanged(bool is_add, View *parent,
                                          View *child) {
-  if (is_add && parent != this && !container_ && GetWidget()) {
+  if (is_add && child == this && GetWidget()) {
     ValidateNativeControl();
     Layout();
   }
diff --git a/views/controls/scrollbar/native_scroll_bar_win.cc b/views/controls/scrollbar/native_scroll_bar_win.cc
index b289823..b3018e3 100644
--- a/views/controls/scrollbar/native_scroll_bar_win.cc
+++ b/views/controls/scrollbar/native_scroll_bar_win.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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.
 
@@ -200,15 +200,15 @@ NativeScrollBarWin::NativeScrollBarWin(NativeScrollBar* scroll_bar)
     : native_scroll_bar_(scroll_bar),
       sb_container_(NULL) {
   set_focus_view(scroll_bar);
-  memset(&scroll_info_, 0, sizeof(scroll_info_));
 }
 
 NativeScrollBarWin::~NativeScrollBarWin() {
-  if (sb_container_.get()) {
+  if (sb_container_) {
     // We always destroy the scrollbar container explicitly to cover all
     // cases including when the container is no longer connected to a
     // widget tree.
     DestroyWindow(sb_container_->hwnd());
+    delete sb_container_;
   }
 }
 
@@ -227,7 +227,7 @@ gfx::Size NativeScrollBarWin::GetPreferredSize() {
 }
 
 bool NativeScrollBarWin::OnKeyPressed(const KeyEvent& event) {
-  if (!sb_container_.get())
+  if (!sb_container_)
     return false;
   int code = -1;
   switch (event.GetKeyCode()) {
@@ -270,7 +270,7 @@ bool NativeScrollBarWin::OnKeyPressed(const KeyEvent& event) {
 }
 
 bool NativeScrollBarWin::OnMouseWheel(const MouseWheelEvent& e) {
-  if (!sb_container_.get())
+  if (!sb_container_)
     return false;
   sb_container_->ScrollWithOffset(e.GetOffset());
   return true;
@@ -280,12 +280,8 @@ bool NativeScrollBarWin::OnMouseWheel(const MouseWheelEvent& e) {
 // NativeScrollBarWin, NativeControlWin overrides:
 
 void NativeScrollBarWin::CreateNativeControl() {
-  sb_container_.reset(new ScrollBarContainer(native_scroll_bar_));
+  sb_container_ = new ScrollBarContainer(native_scroll_bar_);
   NativeControlCreated(sb_container_->hwnd());
-  // Reinstall scroll state if we have valid information.
-  if (scroll_info_.cbSize)
-    SetScrollInfo(sb_container_->GetScrollBarHWND(), SB_CTL, &scroll_info_,
-                  TRUE);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
@@ -306,7 +302,7 @@ View* NativeScrollBarWin::GetView() {
 void NativeScrollBarWin::Update(int viewport_size,
                                 int content_size,
                                 int current_pos) {
-  if (!sb_container_.get())
+  if (!sb_container_)
     return;
 
   if (content_size < 0)
@@ -318,13 +314,14 @@ void NativeScrollBarWin::Update(int viewport_size,
   if (current_pos > content_size)
     current_pos = content_size;
 
-  scroll_info_.cbSize = sizeof(scroll_info_);
-  scroll_info_.fMask = SIF_DISABLENOSCROLL | SIF_POS | SIF_RANGE | SIF_PAGE;
-  scroll_info_.nMin = 0;
-  scroll_info_.nMax = content_size;
-  scroll_info_.nPos = current_pos;
-  scroll_info_.nPage = viewport_size;
-  SetScrollInfo(sb_container_->GetScrollBarHWND(), SB_CTL, &scroll_info_, TRUE);
+  SCROLLINFO si;
+  si.cbSize = sizeof(si);
+  si.fMask = SIF_DISABLENOSCROLL | SIF_POS | SIF_RANGE | SIF_PAGE;
+  si.nMin = 0;
+  si.nMax = content_size;
+  si.nPos = current_pos;
+  si.nPage = viewport_size;
+  SetScrollInfo(sb_container_->GetScrollBarHWND(), SB_CTL, &si, TRUE);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
@@ -347,3 +344,4 @@ int NativeScrollBarWrapper::GetVerticalScrollBarWidth() {
 }
 
 }  // namespace views
+
diff --git a/views/controls/scrollbar/native_scroll_bar_win.h b/views/controls/scrollbar/native_scroll_bar_win.h
index ba7dc25..566f036 100644
--- a/views/controls/scrollbar/native_scroll_bar_win.h
+++ b/views/controls/scrollbar/native_scroll_bar_win.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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.
 
@@ -51,12 +51,7 @@ class NativeScrollBarWin : public NativeControlWin,
   // sb_container_ is a custom hwnd that we use to wrap the real
   // windows scrollbar. We need to do this to get the scroll events
   // without having to do anything special in the high level hwnd.
-  scoped_ptr<ScrollBarContainer> sb_container_;
-
-  // Last scrollbar state we wrote to the scrollbar. We keep it here, so we can
-  // reinitialize the scrollbar to its previous state in case it gets hidden and
-  // recreated.
-  SCROLLINFO scroll_info_;
+  ScrollBarContainer* sb_container_;
 
   DISALLOW_COPY_AND_ASSIGN(NativeScrollBarWin);
 };
@@ -64,3 +59,4 @@ class NativeScrollBarWin : public NativeControlWin,
 }  // namespace views
 
 #endif  // #ifndef VIEWS_CONTROLS_SCROLLBAR_NATIVE_SCROLL_BAR_WIN_H_
+
diff --git a/views/controls/tabbed_pane/native_tabbed_pane_gtk.cc b/views/controls/tabbed_pane/native_tabbed_pane_gtk.cc
index 25705fc..8900de3 100644
--- a/views/controls/tabbed_pane/native_tabbed_pane_gtk.cc
+++ b/views/controls/tabbed_pane/native_tabbed_pane_gtk.cc
@@ -75,11 +75,6 @@ View* NativeTabbedPaneGtk::RemoveTabAtIndex(int index) {
   // widget delete itself when native_view is deleted.
   gtk_notebook_remove_page(GTK_NOTEBOOK(native_view()), index);
 
-  // Removing a tab might change the size of the tabbed pane.
-  RootView* root_view = GetRootView();
-  if (root_view)
-    GetRootView()->Layout();
-
   return removed_tab;
 }
 
@@ -108,19 +103,6 @@ void NativeTabbedPaneGtk::SetFocus() {
   Focus();
 }
 
-gfx::Size NativeTabbedPaneGtk::GetPreferredSize() {
-  if (!native_view())
-    return gfx::Size();
-
-  // For some strange reason (or maybe it's a bug), the requisition is not
-  // returned in the passed requisition parameter, but instead written to the
-  // widget's requisition field.
-  GtkRequisition requisition = { 0, 0 };
-  gtk_widget_size_request(GTK_WIDGET(native_view()), &requisition);
-  GtkRequisition& size(GTK_WIDGET(native_view())->requisition);
-  return gfx::Size(size.width, size.height);
-}
-
 gfx::NativeView NativeTabbedPaneGtk::GetTestingHandle() const {
   return native_view();
 }
@@ -184,11 +166,6 @@ void NativeTabbedPaneGtk::DoAddTabAtIndex(int index, const std::wstring& title,
 
   if (tab_count == 0 && select_if_first_tab)
     gtk_notebook_set_current_page(GTK_NOTEBOOK(native_view()), 0);
-
-  // Relayout the hierarchy, since the added tab might require more space.
-  RootView* root_view = GetRootView();
-  if (root_view)
-    GetRootView()->Layout();
 }
 
 WidgetGtk* NativeTabbedPaneGtk::GetWidgetAt(int index) {
diff --git a/views/controls/tabbed_pane/native_tabbed_pane_gtk.h b/views/controls/tabbed_pane/native_tabbed_pane_gtk.h
index 229faa5..d586e74 100644
--- a/views/controls/tabbed_pane/native_tabbed_pane_gtk.h
+++ b/views/controls/tabbed_pane/native_tabbed_pane_gtk.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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.
 
@@ -31,7 +31,6 @@ class NativeTabbedPaneGtk : public NativeControlGtk,
   virtual View* GetSelectedTab();
   virtual View* GetView();
   virtual void SetFocus();
-  virtual gfx::Size GetPreferredSize();
   virtual gfx::NativeView GetTestingHandle() const;
 
   // NativeControlGtk overrides.
diff --git a/views/controls/tabbed_pane/native_tabbed_pane_win.cc b/views/controls/tabbed_pane/native_tabbed_pane_win.cc
index 838991f..388cb56 100644
--- a/views/controls/tabbed_pane/native_tabbed_pane_win.cc
+++ b/views/controls/tabbed_pane/native_tabbed_pane_win.cc
@@ -46,64 +46,12 @@ class TabBackground : public Background {
   DISALLOW_COPY_AND_ASSIGN(TabBackground);
 };
 
-// Custom layout manager that takes care of sizing and displaying the tab pages.
-class TabLayout : public LayoutManager {
- public:
-  TabLayout() {}
-
-  // Switches to the tab page identified by the given index.
-  void SwitchToPage(View* host, View* page) {
-    for (int i = 0; i < host->GetChildViewCount(); ++i) {
-      View* child = host->GetChildViewAt(i);
-      // The child might not have been laid out yet.
-      if (child == page)
-        child->SetBounds(gfx::Rect(host->size()));
-      child->SetVisible(child == page);
-    }
-  }
-
- private:
-  // LayoutManager overrides:
-  virtual void Layout(View* host) {
-    gfx::Rect bounds(host->size());
-    for (int i = 0; i < host->GetChildViewCount(); ++i) {
-      View* child = host->GetChildViewAt(i);
-      // We only layout visible children, since it may be expensive.
-      if (child->IsVisible() && child->bounds() != bounds)
-        child->SetBounds(bounds);
-    }
-  }
-
-  virtual gfx::Size GetPreferredSize(View* host) {
-    // First, query the preferred sizes to determine a good width.
-    int width = 0;
-    for (int i = 0; i < host->GetChildViewCount(); ++i) {
-      View* page = host->GetChildViewAt(i);
-      width = std::max(width, page->GetPreferredSize().width());
-    }
-    // After we know the width, decide on the height.
-    return gfx::Size(width, GetPreferredHeightForWidth(host, width));
-  }
-
-  virtual int GetPreferredHeightForWidth(View* host, int width) {
-    int height = 0;
-    for (int i = 0; i < host->GetChildViewCount(); ++i) {
-      View* page = host->GetChildViewAt(i);
-      height = std::max(height, page->GetHeightForWidth(width));
-    }
-    return height;
-  }
-
-  DISALLOW_COPY_AND_ASSIGN(TabLayout);
-};
-
 ////////////////////////////////////////////////////////////////////////////////
 // NativeTabbedPaneWin, public:
 
 NativeTabbedPaneWin::NativeTabbedPaneWin(TabbedPane* tabbed_pane)
     : NativeControlWin(),
       tabbed_pane_(tabbed_pane),
-      tab_layout_manager_(NULL),
       content_window_(NULL),
       selected_index_(-1) {
   // Associates the actual HWND with the tabbed-pane so the tabbed-pane is
@@ -129,32 +77,31 @@ void NativeTabbedPaneWin::AddTabAtIndex(int index, const std::wstring& title,
                                         bool select_if_first_tab) {
   DCHECK(index <= static_cast<int>(tab_views_.size()));
   contents->set_parent_owned(false);
-  contents->SetVisible(false);
   tab_views_.insert(tab_views_.begin() + index, contents);
   tab_titles_.insert(tab_titles_.begin() + index, title);
 
   if (!contents->background())
     contents->set_background(new TabBackground);
 
-  if (tab_views_.size() == 1 && select_if_first_tab)
+  if (tab_views_.size() == 1 && select_if_first_tab) {
+    // If this is the only tab displayed, make sure the contents is set.
     selected_index_ = 0;
+    if (content_window_)
+      content_window_->GetRootView()->AddChildView(contents);
+  }
 
   // Add native tab only if the native control is alreay created.
   if (content_window_) {
-    AddNativeTab(index, title);
+    AddNativeTab(index, title, contents);
+
     // The newly added tab may have made the contents window smaller.
     ResizeContents();
-
-    RootView* content_root = content_window_->GetRootView();
-    content_root->AddChildView(contents);
-    // Switch to the newly added tab if requested;
-    if (tab_views_.size() == 1 && select_if_first_tab)
-      tab_layout_manager_->SwitchToPage(content_root, contents);
   }
 }
 
 void NativeTabbedPaneWin::AddNativeTab(int index,
-                                       const std::wstring &title) {
+                                       const std::wstring &title,
+                                       views::View* contents) {
   TCITEM tcitem;
   tcitem.mask = TCIF_TEXT;
 
@@ -193,8 +140,6 @@ View* NativeTabbedPaneWin::RemoveTabAtIndex(int index) {
 
   std::vector<View*>::iterator iter = tab_views_.begin() + index;
   View* removed_tab = *iter;
-  if (content_window_)
-    content_window_->GetRootView()->RemoveChildView(removed_tab);
   tab_views_.erase(iter);
   tab_titles_.erase(tab_titles_.begin() + index);
 
@@ -231,17 +176,6 @@ void NativeTabbedPaneWin::SetFocus() {
   Focus();
 }
 
-gfx::Size NativeTabbedPaneWin::GetPreferredSize() {
-  if (!native_view())
-    return gfx::Size();
-
-  gfx::Rect contentSize(content_window_->GetRootView()->GetPreferredSize());
-  RECT paneSize = { 0, 0, contentSize.width(), contentSize.height() };
-  TabCtrl_AdjustRect(native_view(), TRUE, &paneSize);
-  return gfx::Size(paneSize.right - paneSize.left,
-                   paneSize.bottom - paneSize.top);
-}
-
 gfx::NativeView NativeTabbedPaneWin::GetTestingHandle() const {
   return native_view();
 }
@@ -291,8 +225,7 @@ void NativeTabbedPaneWin::CreateNativeControl() {
     l10n_util::HWNDSetRTLLayout(tab_control);
 
   RootView* root_view = content_window_->GetRootView();
-  tab_layout_manager_ = new TabLayout();
-  root_view->SetLayoutManager(tab_layout_manager_);
+  root_view->SetLayoutManager(new FillLayout());
   DWORD sys_color = ::GetSysColor(COLOR_3DHILIGHT);
   SkColor color = SkColorSetRGB(GetRValue(sys_color), GetGValue(sys_color),
                                 GetBValue(sys_color));
@@ -354,13 +287,9 @@ void NativeTabbedPaneWin::ViewHierarchyChanged(bool is_add,
 // NativeTabbedPaneWin, private:
 
 void NativeTabbedPaneWin::InitializeTabs() {
-  for (size_t i = 0; i < tab_titles_.size(); ++i)
-    AddNativeTab(i, tab_titles_[i]);
-
-  RootView* content_root = content_window_->GetRootView();
-  for (std::vector<View*>::iterator tab(tab_views_.begin());
-       tab != tab_views_.end(); ++tab)
-    content_root->AddChildView(*tab);
+  for (size_t i = 0; i < tab_views_.size(); ++i) {
+    AddNativeTab(i, tab_titles_[i], tab_views_[i]);
+  }
 }
 
 void NativeTabbedPaneWin::DoSelectTabAt(int index, boolean invoke_listener) {
@@ -374,7 +303,10 @@ void NativeTabbedPaneWin::DoSelectTabAt(int index, boolean invoke_listener) {
     View* focused_view = focus_manager->GetFocusedView();
     if (focused_view && content_root->IsParentOf(focused_view))
       focus_manager->ClearFocus();
-    tab_layout_manager_->SwitchToPage(content_root, tab_views_.at(index));
+
+    content_root->RemoveAllChildViews(false);
+    content_root->AddChildView(tab_views_[index]);
+    content_root->Layout();
   }
   if (invoke_listener && tabbed_pane_->listener())
     tabbed_pane_->listener()->TabSelectedAt(index);
diff --git a/views/controls/tabbed_pane/native_tabbed_pane_win.h b/views/controls/tabbed_pane/native_tabbed_pane_win.h
index 4e4cc2a..c3dcaed 100644
--- a/views/controls/tabbed_pane/native_tabbed_pane_win.h
+++ b/views/controls/tabbed_pane/native_tabbed_pane_win.h
@@ -13,7 +13,6 @@
 namespace views {
 
 class WidgetWin;
-class TabLayout;
 
 class NativeTabbedPaneWin : public NativeControlWin,
                             public NativeTabbedPaneWrapper {
@@ -34,7 +33,6 @@ class NativeTabbedPaneWin : public NativeControlWin,
   virtual View* GetSelectedTab();
   virtual View* GetView();
   virtual void SetFocus();
-  virtual gfx::Size GetPreferredSize();
   virtual gfx::NativeView GetTestingHandle() const;
 
   // NativeControlWin overrides.
@@ -59,7 +57,7 @@ class NativeTabbedPaneWin : public NativeControlWin,
   void InitializeTabs();
 
   // Adds a tab with the given content to native control at the given index.
-  void AddNativeTab(int index, const std::wstring& title);
+  void AddNativeTab(int index, const std::wstring& title, View* contents);
 
   // Changes the contents view to the view associated with the tab at |index|.
   // |invoke_listener| controls if this methold should invoke the
@@ -72,9 +70,6 @@ class NativeTabbedPaneWin : public NativeControlWin,
   // The tabbed-pane we are bound to.
   TabbedPane* tabbed_pane_;
 
-  // The layout manager we use for managing our tabs.
-  TabLayout* tab_layout_manager_;
-
   // The views associated with the different tabs.
   std::vector<View*> tab_views_;
 
diff --git a/views/controls/tabbed_pane/native_tabbed_pane_wrapper.h b/views/controls/tabbed_pane/native_tabbed_pane_wrapper.h
index 97c5fba..a051898 100644
--- a/views/controls/tabbed_pane/native_tabbed_pane_wrapper.h
+++ b/views/controls/tabbed_pane/native_tabbed_pane_wrapper.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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.
 
@@ -56,9 +56,6 @@ class NativeTabbedPaneWrapper {
   // Sets the focus to the tabbed pane native view.
   virtual void SetFocus() = 0;
 
-  // Gets the preferred size of the tabbed pane.
-  virtual gfx::Size GetPreferredSize() = 0;
-
   // Returns a handle to the underlying native view for testing.
   virtual gfx::NativeView GetTestingHandle() const = 0;
 
diff --git a/views/controls/tabbed_pane/tabbed_pane.cc b/views/controls/tabbed_pane/tabbed_pane.cc
index 1a7b0f0..5cb570a 100644
--- a/views/controls/tabbed_pane/tabbed_pane.cc
+++ b/views/controls/tabbed_pane/tabbed_pane.cc
@@ -102,8 +102,10 @@ void TabbedPane::LoadAccelerators() {
 }
 
 void TabbedPane::Layout() {
-  if (native_tabbed_pane_)
+  if (native_tabbed_pane_) {
     native_tabbed_pane_->GetView()->SetBounds(0, 0, width(), height());
+    native_tabbed_pane_->GetView()->Layout();
+  }
 }
 
 void TabbedPane::Focus() {
@@ -127,9 +129,4 @@ bool TabbedPane::GetAccessibleRole(AccessibilityTypes::Role* role) {
   return true;
 }
 
-gfx::Size TabbedPane::GetPreferredSize() {
-  return native_tabbed_pane_ ?
-      native_tabbed_pane_->GetPreferredSize() : gfx::Size();
-}
-
 }  // namespace views
diff --git a/views/controls/tabbed_pane/tabbed_pane.h b/views/controls/tabbed_pane/tabbed_pane.h
index 36026a0..2fe503e 100644
--- a/views/controls/tabbed_pane/tabbed_pane.h
+++ b/views/controls/tabbed_pane/tabbed_pane.h
@@ -13,6 +13,7 @@ class NativeTabbedPaneWrapper;
 
 // The TabbedPane class is a view that shows tabs.  When the user clicks on a
 // tab, the associated view is displayed.
+// TODO (jcampan): implement GetPreferredSize().
 
 class TabbedPane : public View {
  public:
@@ -71,7 +72,6 @@ class TabbedPane : public View {
   virtual void Focus();
   virtual void PaintFocusBorder(gfx::Canvas* canvas);
   virtual bool GetAccessibleRole(AccessibilityTypes::Role* role);
-  virtual gfx::Size GetPreferredSize();
 
   NativeTabbedPaneWrapper* native_wrapper() const {
     return native_tabbed_pane_;
diff --git a/views/controls/tabbed_pane/tabbed_pane_unittest.cc b/views/controls/tabbed_pane/tabbed_pane_unittest.cc
deleted file mode 100644
index e0428f2..0000000
--- a/views/controls/tabbed_pane/tabbed_pane_unittest.cc
+++ /dev/null
@@ -1,82 +0,0 @@
-// Copyright (c) 2010 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 "base/message_loop.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "views/controls/tabbed_pane/tabbed_pane.h"
-#include "views/window/window.h"
-#include "views/window/window_delegate.h"
-
-namespace views {
-
-// A view for testing that takes a fixed preferred size upon construction.
-class FixedSizeView : public View {
- public:
-  FixedSizeView(const gfx::Size& size)
-    : size_(size) {}
-
-  virtual gfx::Size GetPreferredSize() {
-    return size_;
-  }
-
- private:
-  const gfx::Size size_;
-
-  DISALLOW_COPY_AND_ASSIGN(FixedSizeView);
-};
-
-class TabbedPaneTest : public testing::Test, WindowDelegate {
- public:
-  TabbedPaneTest() {}
-
-  TabbedPane* tabbed_pane_;
-
- private:
-  virtual void SetUp() {
-    tabbed_pane_ = new TabbedPane();
-    window_ = Window::CreateChromeWindow(NULL, gfx::Rect(0, 0, 100, 100), this);
-    window_->Show();
-  }
-
-  virtual void TearDown() {
-    window_->Close();
-  }
-
-  virtual views::View* GetContentsView() {
-    return tabbed_pane_;
-  }
-
-  MessageLoopForUI message_loop_;
-  Window* window_;
-
-  DISALLOW_COPY_AND_ASSIGN(TabbedPaneTest);
-};
-
-// Tests that TabbedPane::GetPreferredSize() and TabbedPane::Layout().
-TEST_F(TabbedPaneTest, SizeAndLayout) {
-  View* child1 = new FixedSizeView(gfx::Size(20, 10));
-  tabbed_pane_->AddTab(L"tab1", child1);
-  View* child2 = new FixedSizeView(gfx::Size(5, 5));
-  tabbed_pane_->AddTab(L"tab2", child2);
-  tabbed_pane_->SelectTabAt(0);
-
-  // Check that the preferred size is larger than the largest child.
-  gfx::Size pref(tabbed_pane_->GetPreferredSize());
-  EXPECT_GT(pref.width(), 20);
-  EXPECT_GT(pref.height(), 10);
-
-  // The bounds of our children should be smaller than the tabbed pane's bounds.
-  tabbed_pane_->SetBounds(0, 0, 100, 200);
-  gfx::Rect bounds(child1->bounds());
-  EXPECT_GT(bounds.width(), 0);
-  EXPECT_LT(bounds.width(), 100);
-  EXPECT_GT(bounds.height(), 0);
-  EXPECT_LT(bounds.height(), 200);
-
-  // If we switch to the other tab, it should get assigned the same bounds.
-  tabbed_pane_->SelectTabAt(1);
-  EXPECT_EQ(bounds, child2->bounds());
-}
-
-} // namespace views
diff --git a/views/controls/table/table_view.cc b/views/controls/table/table_view.cc
index 7ab4c91..8b86a6e 100644
--- a/views/controls/table/table_view.cc
+++ b/views/controls/table/table_view.cc
@@ -64,6 +64,7 @@ TableView::TableView(TableModel* model,
       single_selection_(single_selection),
       ignore_listview_change_(false),
       custom_colors_enabled_(false),
+      sized_columns_(false),
       autosize_columns_(autosize_columns),
       resizable_columns_(resizable_columns),
       list_view_(NULL),
@@ -132,8 +133,10 @@ void TableView::DidChangeBounds(const gfx::Rect& previous,
     return;
   SendMessage(list_view_, WM_SETREDRAW, static_cast<WPARAM>(FALSE), 0);
   Layout();
-  if (autosize_columns_ && width() > 0)
+  if ((!sized_columns_ || autosize_columns_) && width() > 0) {
+    sized_columns_ = true;
     ResetColumnSizes();
+  }
   UpdateContentOffset();
   SendMessage(list_view_, WM_SETREDRAW, static_cast<WPARAM>(TRUE), 0);
 }
@@ -1548,15 +1551,6 @@ gfx::Font TableView::GetAltTextFont() {
   return ResourceBundle::GetSharedInstance().GetFont(ResourceBundle::BaseFont);
 }
 
-void TableView::VisibilityChanged(View* starting_from, bool is_visible) {
-  // GetClientRect as used by ResetColumnSize to obtain the total width
-  // available to the columns only works when the native control is visible, so
-  // update the column sizes in case we become visible. This depends on
-  // VisibilityChanged() being called in post order on the view tree.
-  if (is_visible && autosize_columns_)
-    ResetColumnSizes();
-}
-
 
 //
 // TableSelectionIterator
diff --git a/views/controls/table/table_view.h b/views/controls/table/table_view.h
index 28d877e..f237df2 100644
--- a/views/controls/table/table_view.h
+++ b/views/controls/table/table_view.h
@@ -407,10 +407,6 @@ class TableView : public NativeControl,
   // Returns the font used for alt text.
   gfx::Font GetAltTextFont();
 
-  // Overriden in order to update the column sizes, which can only be sized
-  // accurately when the native control is available.
-  virtual void VisibilityChanged(View* starting_from, bool is_visible);
-
   TableModel* model_;
   TableTypes table_type_;
   TableViewObserver* table_view_observer_;
@@ -436,6 +432,10 @@ class TableView : public NativeControl,
   // Reflects the value passed to SetCustomColorsEnabled.
   bool custom_colors_enabled_;
 
+  // Whether or not the columns have been sized in the ListView. This is
+  // set to true the first time Layout() is invoked and we have a valid size.
+  bool sized_columns_;
+
   // Whether or not columns should automatically be resized to fill the
   // the available width when the list view is resized.
   bool autosize_columns_;
diff --git a/views/examples/tabbed_pane_example.h b/views/examples/tabbed_pane_example.h
index 1b712a8..4b4bdfc 100644
--- a/views/examples/tabbed_pane_example.h
+++ b/views/examples/tabbed_pane_example.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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.
 
@@ -109,3 +109,4 @@ class TabbedPaneExample : public ExampleBase,
 }  // namespace examples
 
 #endif  // VIEWS_EXAMPLES_TABBED_PANE_EXAMPLE_H_
+
diff --git a/views/grid_layout.cc b/views/grid_layout.cc
index 5e685ac..b3f0625 100644
--- a/views/grid_layout.cc
+++ b/views/grid_layout.cc
@@ -844,26 +844,22 @@ void GridLayout::SizeRowsAndColumns(bool layout, int width, int height,
   if (rows_.empty())
     return;
 
-  // Calculate the preferred width of each of the columns. Some views'
-  // preferred heights are derived from their width, as such we need to
-  // calculate the size of the columns first.
+  // Calculate the size of each of the columns. Some views preferred heights are
+  // derived from their width, as such we need to calculate the size of the
+  // columns first.
   for (std::vector<ColumnSet*>::iterator i = column_sets_.begin();
        i != column_sets_.end(); ++i) {
     (*i)->CalculateSize();
+    if (layout || width > 0) {
+      // We're doing a layout, divy up any extra space.
+      (*i)->Resize(width - (*i)->LayoutWidth() - left_inset_ - right_inset_);
+      // And reset the x coordinates.
+      (*i)->ResetColumnXCoordinates();
+    }
     pref->set_width(std::max(pref->width(), (*i)->LayoutWidth()));
   }
   pref->set_width(pref->width() + left_inset_ + right_inset_);
 
-  // Go over the columns again and set them all to the size we settled for.
-  width = width ? width : pref->width();
-  for (std::vector<ColumnSet*>::iterator i = column_sets_.begin();
-       i != column_sets_.end(); ++i) {
-    // We're doing a layout, divy up any extra space.
-    (*i)->Resize(width - (*i)->LayoutWidth() - left_inset_ - right_inset_);
-    // And reset the x coordinates.
-    (*i)->ResetColumnXCoordinates();
-  }
-
   // Reset the height of each row.
   LayoutElement::ResetSizes(&rows_);
 
@@ -1067,3 +1063,4 @@ views::GridLayout* CreatePanelGridLayout(views::View* host) {
                     kPanelVertMargin, kPanelHorizMargin);
   return layout;
 }
+
diff --git a/views/grid_layout.h b/views/grid_layout.h
index bb882ad..6d6c61c 100644
--- a/views/grid_layout.h
+++ b/views/grid_layout.h
@@ -183,7 +183,8 @@ class GridLayout : public LayoutManager {
   // they both call into this method. This sizes the Columns/Rows as
   // appropriate. If layout is true, width/height give the width/height the
   // of the host, otherwise they are ignored.
-  void SizeRowsAndColumns(bool layout, int width, int height, gfx::Size* pref);
+     void SizeRowsAndColumns(bool layout, int width, int height,
+                             gfx::Size* pref);
 
   // Calculates the master columns of all the column sets. See Column for
   // a description of what a master column is.
diff --git a/views/grid_layout_unittest.cc b/views/grid_layout_unittest.cc
index a70ae5f..c5e56fd 100644
--- a/views/grid_layout_unittest.cc
+++ b/views/grid_layout_unittest.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2006-2008 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.
 
@@ -32,25 +32,6 @@ class SettableSizeView : public View {
    gfx::Size pref_;
 };
 
-// A view with fixed circumference that trades height for width.
-class FlexibleView : public View {
- public:
-  explicit FlexibleView(int circumference) {
-    circumference_ = circumference;
-  }
-
-  virtual gfx::Size GetPreferredSize() {
-    return gfx::Size(0, circumference_ / 2);
-  }
-
-  virtual int GetHeightForWidth(int width) {
-    return std::max(0, circumference_ / 2 - width);
-  }
-
- private:
-   int circumference_;
-};
-
 class GridLayoutTest : public testing::Test {
  public:
   virtual void SetUp() {
@@ -567,42 +548,3 @@ TEST_F(GridLayoutTest, ColumnSpanResizing) {
   // 4 + (6 * 4 / 6).
   ExpectViewBoundsEquals(4, 40, 8, 40, view2);
 }
-
-// Check that GetPreferredSize() takes resizing of columns into account when
-// there is additional space in the case we have column sets of different
-// preferred sizes.
-TEST_F(GridLayoutTest, ColumnResizingOnGetPreferredSize) {
-  views::ColumnSet* set = layout->AddColumnSet(0);
-  set->AddColumn(views::GridLayout::FILL, views::GridLayout::CENTER,
-                 1, views::GridLayout::USE_PREF, 0, 0);
-
-  set = layout->AddColumnSet(1);
-  set->AddColumn(views::GridLayout::FILL, views::GridLayout::CENTER,
-                 1, views::GridLayout::USE_PREF, 0, 0);
-
-  set = layout->AddColumnSet(2);
-  set->AddColumn(views::GridLayout::FILL, views::GridLayout::CENTER,
-                 1, views::GridLayout::USE_PREF, 0, 0);
-
-  // Make a row containing a flexible view that trades width for height.
-  layout->StartRow(0, 0);
-  View* view1 = new FlexibleView(100);
-  layout->AddView(view1, 1, 1, GridLayout::FILL, GridLayout::LEADING);
-
-  // The second row contains a view of fixed size that will enforce a column
-  // width of 20 pixels.
-  layout->StartRow(0, 1);
-  View* view2 = new SettableSizeView(gfx::Size(20, 20));
-  layout->AddView(view2, 1, 1, GridLayout::FILL, GridLayout::LEADING);
-
-  // Add another flexible view in row three in order to ensure column set
-  // ordering doesn't influence sizing behaviour.
-  layout->StartRow(0, 2);
-  View* view3 = new FlexibleView(40);
-  layout->AddView(view3, 1, 1, GridLayout::FILL, GridLayout::LEADING);
-
-  // We expect a height of 50: 30 from the variable width view in the first row
-  // plus 20 from the statically sized view in the second row. The flexible
-  // view in the third row should contribute no height.
-  EXPECT_EQ(gfx::Size(20, 50), layout->GetPreferredSize(&host));
-}
diff --git a/views/view.cc b/views/view.cc
index 22b048b..90fb2bc 100644
--- a/views/view.cc
+++ b/views/view.cc
@@ -218,14 +218,17 @@ void View::ScrollRectToVisible(const gfx::Rect& rect) {
 /////////////////////////////////////////////////////////////////////////////
 
 void View::Layout() {
-  // If we have a layout manager, let it handle the layout for us.
+  // Layout child Views
   if (layout_manager_.get()) {
     layout_manager_->Layout(this);
     SchedulePaint();
-    return;
+    // TODO(beng): We believe the right thing to do here is return since the
+    //             layout manager should be handling things, but it causes
+    //             regressions (missing options from Options dialog and a hang
+    //             in interactive_ui_tests).
   }
 
-  // Otherwise, just pass on to the child views.
+  // Lay out contents of child Views
   for (int i = 0, count = GetChildViewCount(); i < count; ++i) {
     View* child = GetChildViewAt(i);
     child->Layout();
diff --git a/views/widget/widget_gtk.cc b/views/widget/widget_gtk.cc
index c9144d7..1e73e38 100644
--- a/views/widget/widget_gtk.cc
+++ b/views/widget/widget_gtk.cc
@@ -500,8 +500,6 @@ void WidgetGtk::Init(GtkWidget* parent,
                         GDK_KEY_RELEASE_MASK);
   SetRootViewForWidget(widget_, root_view_.get());
 
-  g_signal_connect_after(G_OBJECT(window_contents_), "size_request",
-                         G_CALLBACK(&OnSizeRequestThunk), this);
   g_signal_connect_after(G_OBJECT(window_contents_), "size_allocate",
                          G_CALLBACK(&OnSizeAllocateThunk), this);
   gtk_widget_set_app_paintable(window_contents_, true);
@@ -885,18 +883,6 @@ int WidgetGtk::GetFlagsForEventButton(const GdkEventButton& event) {
   return flags;
 }
 
-void WidgetGtk::OnSizeRequest(GtkWidget* widget, GtkRequisition* requisition) {
-  // Do only return the preferred size for child windows. GtkWindow interprets
-  // the requisition as a minimum size for top level windows, returning a
-  // preferred size for these would prevents us from setting smaller window
-  // sizes.
-  if (type_ == TYPE_CHILD) {
-    gfx::Size size(root_view_->GetPreferredSize());
-    requisition->width = size.width();
-    requisition->height = size.height();
-  }
-}
-
 void WidgetGtk::OnSizeAllocate(GtkWidget* widget, GtkAllocation* allocation) {
   // See comment next to size_ as to why we do this. Also note, it's tempting
   // to put this in the static method so subclasses don't need to worry about
diff --git a/views/widget/widget_gtk.h b/views/widget/widget_gtk.h
index 3700aa3..816a263 100644
--- a/views/widget/widget_gtk.h
+++ b/views/widget/widget_gtk.h
@@ -223,7 +223,6 @@ class WidgetGtk
 
   // Event handlers:
   CHROMEGTK_CALLBACK_1(WidgetGtk, gboolean, OnButtonPress, GdkEventButton*);
-  CHROMEGTK_CALLBACK_1(WidgetGtk, void, OnSizeRequest, GtkRequisition*);
   CHROMEGTK_CALLBACK_1(WidgetGtk, void, OnSizeAllocate, GtkAllocation*);
   CHROMEGTK_CALLBACK_1(WidgetGtk, gboolean, OnPaint, GdkEventExpose*);
   CHROMEGTK_CALLBACK_4(WidgetGtk, void, OnDragDataGet,