Update widget size from StatusAreaView::Layout()

chrome/browser/chromeos/status/status_area_view.cc

Index: chrome/browser/chromeos/status/status_area_view.cc
diff --git a/chrome/browser/chromeos/status/status_area_view.cc b/chrome/browser/chromeos/status/status_area_view.cc
index 5f67535a7d0faa5729df7d7762fb5101f4bb54f3..0a11d9c4e01dec0993e712766f5d7b59549171b0 100644
--- a/chrome/browser/chromeos/status/status_area_view.cc
+++ b/chrome/browser/chromeos/status/status_area_view.cc
@@ -88,6 +88,10 @@ void StatusAreaView::Layout() {
       cur_x += cur_size.width() + kSeparation;
     }
   }
+#if defined(USE_AURA)
+  if (GetWidget())

[sky, 2011/12/13 16:40:41]

This makes it easy to get stuck in a loop. Can we instead invoke
PreferredSizeChanged when toggling visibility?

[stevenjb, 2011/12/14 02:10:51]

We already do; I will override PreferredSizeChanged() and move this chunk of
code there.
Done.

+    GetWidget()->SetSize(GetPreferredSize());
+#endif
 }
 
 void StatusAreaView::ChildPreferredSizeChanged(View* child) {
@@ -96,10 +100,6 @@ void StatusAreaView::ChildPreferredSizeChanged(View* child) {
   // BrowserView know to relayout, which will reset the bounds of this view.
   Layout();
   PreferredSizeChanged();
-#if defined(USE_AURA)
-  if (GetWidget())
-    GetWidget()->SetSize(GetPreferredSize());
-#endif
 }
 
 void StatusAreaView::MakeButtonsActive(bool active) {