Multi-Profiles: Change avatar menu to bubble view

chrome/browser/ui/views/avatar_menu_button.cc

Index: chrome/browser/ui/views/avatar_menu_button.cc
diff --git a/chrome/browser/ui/views/avatar_menu_button.cc b/chrome/browser/ui/views/avatar_menu_button.cc
index efe86a8ffa87ba5d3fc77645f444936edce20cbc..9e425f47374003454e6cc38b51949f3a4d24e28d 100644
--- a/chrome/browser/ui/views/avatar_menu_button.cc
+++ b/chrome/browser/ui/views/avatar_menu_button.cc
@@ -6,7 +6,8 @@
 
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/profile_menu_model.h"
-#include "chrome/browser/ui/views/avatar_menu.h"
+#include "chrome/browser/ui/views/avatar_menu_bubble_view.h"
+#include "chrome/browser/ui/views/frame/browser_view.h"
 #include "ui/gfx/canvas_skia.h"
 #include "views/widget/widget.h"
 
@@ -62,9 +63,15 @@ void AvatarMenuButton::RunMenu(views::View* source, const gfx::Point& pt) {
   if (!has_menu_)
     return;
 
-  menu_model_.reset(new ProfileMenuModel(browser_));
-  // The avatar menu will automatically delete itself when done.
-  AvatarMenu* avatar_menu =
-      new AvatarMenu(menu_model_.get(), browser_->profile());
-  avatar_menu->RunMenu(this);
+  BrowserView* browser_view = BrowserView::GetBrowserViewForNativeWindow(
+      browser_->window()->GetNativeHandle());
+
+  gfx::Point origin;
+  views::View::ConvertPointToScreen(this, &origin);
+  gfx::Rect bounds(0, 0, width(), height());
+  bounds.set_origin(origin);
+
+  AvatarMenuBubbleView* bubble_view = new AvatarMenuBubbleView(browser_);

[Robert Sesek, 2011/08/05 00:44:03]

Comment about ownership?

[sail, 2011/08/05 01:28:21]

Done.

+  Bubble::Show(browser_view->GetWidget(), bounds, BubbleBorder::TOP_LEFT,
+               bubble_view, bubble_view);
 }