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

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

Issue 12483006: wrench_menu: use NativeTheme colors under aura (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove unused constant in common_theme.cc Created 7 years, 9 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/wrench_menu.h
diff --git a/chrome/browser/ui/views/wrench_menu.h b/chrome/browser/ui/views/wrench_menu.h
index 7f13609b28f5dc910656719c43af20fe16c16f6d..8228475b24b8dcfac60ebe991f0317ba7f80f851 100644
--- a/chrome/browser/ui/views/wrench_menu.h
+++ b/chrome/browser/ui/views/wrench_menu.h
@@ -41,7 +41,7 @@ class WrenchMenu : public views::MenuDelegate,
bool supports_new_separators);
virtual ~WrenchMenu();
- void Init(ui::MenuModel* model);
+ void Init(views::View* view, ui::MenuModel* model);
sky 2013/03/19 21:08:26 It seems as though the only need you have for view
Bobby Powers 2013/03/20 14:54:38 Ah, yes! Very good. I've reworked the patch to d
// Shows the menu relative to the specified view.
void RunMenu(views::MenuButton* host);
@@ -107,6 +107,7 @@ class WrenchMenu : public views::MenuDelegate,
// that is created.
void PopulateMenu(views::MenuItemView* parent,
ui::MenuModel* model,
+ views::View* view,
int* next_id);
// Adds a new menu to |parent| to represent the MenuModel/index pair passed

Powered by Google App Engine
This is Rietveld 408576698