WIP - loop in LayoutManager for preferred size changes

ui/message_center/views/message_popup_bubble.cc

 // Copyright (c) 2012 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 "ui/message_center/views/message_popup_bubble.h"
 
 #include "base/bind.h"
 #include "base/stl_util.h"
 #include "ui/message_center/message_center.h"
 #include "ui/message_center/message_center_style.h"
@@ skipping 49 matching lines @@
   for (NotificationList::PopupNotifications::const_iterator iter =
            popup_notifications.begin();
        iter != popup_notifications.end(); ++iter) {
     // NotificationViews are expanded by default here because MessagePopupBubble
     // hasn't been tested yet with changing subview sizes, and such changes
     // could come if those subviews were initially collapsed and allowed to be
     // expanded by users. TODO(dharcourt): Fix.
     content_->AddChildView(
         NotificationView::Create(*(*iter), message_center_, true));
   }
-  content_->SizeToPreferredSize();
-  content_->InvalidateLayout();
+  views::ScopedMaybeLayout layout(content_);
+//  content_->SizeToPreferredSize();
+//  content_->InvalidateLayout();
   Layout();
   if (GetWidget())
     GetWidget()->GetRootView()->SchedulePaint();
 }
 
 // The timer to call OnAutoClose for |notification|.
 class MessagePopupBubble::AutocloseTimer {
  public:
   AutocloseTimer(Notification* notification, MessagePopupBubble* bubble);
 
@@ skipping 146 matching lines @@
     scoped_ptr<AutocloseTimer> release_timer(iter->second);
     autoclose_timers_.erase(iter);
   }
 }
 
 size_t MessagePopupBubble::NumMessageViewsForTest() const {
   return contents_view_->NumMessageViews();
 }
 
 }  // namespace message_center