Update avatar bubble if profile info changes

chrome/browser/ui/views/avatar_menu_bubble_view.cc

Index: chrome/browser/ui/views/avatar_menu_bubble_view.cc
diff --git a/chrome/browser/ui/views/avatar_menu_bubble_view.cc b/chrome/browser/ui/views/avatar_menu_bubble_view.cc
index 9f1b0816bbb73414ca795ccb99911cccdf942864..da7c78dc57707c328426caa908800fe30a286efa 100644
--- a/chrome/browser/ui/views/avatar_menu_bubble_view.cc
+++ b/chrome/browser/ui/views/avatar_menu_bubble_view.cc
@@ -497,5 +497,9 @@ void AvatarMenuBubbleView::OnAvatarMenuModelChanged(
   add_profile_link_->SetEnabledColor(SkColorSetRGB(0xe3, 0xed, 0xf6));
   AddChildView(add_profile_link_);
 
-  PreferredSizeChanged();
+  // If the bubble has already been shown then resize and reposition the bubble.
+  if (GetWidget()) {

[Peter Kasting, 2011/12/12 21:43:43]

This is actually kind of worrisome.  If we can reach here without GetWidget()
being true that means that we could  probably leak the views created above. 
Normally we only create and add child views in response to ourselves being added
to a view hierarchy, i.e. in an OnViewHierarchyChanged() override.  This class
does it in the constructor instead which is bad form because it means whoever
constructs this class needs to ensure it gets properly destroyed if it is never
added to a view hierarchy.

If we fix this problem then perhaps the original bug goes away as well?

[sail, 2011/12/13 01:58:34]

Done.
I still needed the Layout/SizeToContents() call to get the button to reposition.

+    Layout();
+    SizeToContents();
+  }
 }