Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(861)

Unified Diff: chrome/browser/ui/views/avatar_menu_button.h

Issue 7331017: Multi-Profiles: Add icon chooser to profiles menu (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: close menu on button pressed Created 9 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/avatar_menu_button.h
diff --git a/chrome/browser/ui/views/avatar_menu_button.h b/chrome/browser/ui/views/avatar_menu_button.h
index 9674995dcc5ad7ac1b906652aa77d109e8d7c1db..5337958a32fc4e7162a338f1730dfbe0d05ed808 100644
--- a/chrome/browser/ui/views/avatar_menu_button.h
+++ b/chrome/browser/ui/views/avatar_menu_button.h
@@ -16,6 +16,7 @@
namespace gfx {
class Canvas;
}
+class Browser;
// AvatarMenuButton
//
@@ -25,18 +26,21 @@ class Canvas;
class AvatarMenuButton : public views::MenuButton,
public views::ViewMenuDelegate {
public:
- // Creates a new button. The object will take ownership of the menu model.
- AvatarMenuButton(const std::wstring& text, ui::MenuModel* menu_model);
+ // Creates a new button.
+ AvatarMenuButton(Browser* browser, bool has_menu);
sky 2011/07/08 23:03:35 Document has_menu. And make this take a Profile in
sail 2011/07/08 23:07:55 Added documentation. I still need the browser para
virtual ~AvatarMenuButton();
// views::MenuButton
virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE;
+ virtual bool HitTest(const gfx::Point& point) const OVERRIDE;
private:
// views::ViewMenuDelegate
virtual void RunMenu(views::View* source, const gfx::Point& pt) OVERRIDE;
+ Browser* browser_;
+ bool has_menu_;
scoped_ptr<ui::MenuModel> menu_model_;
DISALLOW_COPY_AND_ASSIGN(AvatarMenuButton);

Powered by Google App Engine
This is Rietveld 408576698