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

Unified Diff: chrome/browser/ui/toolbar/wrench_menu_model.h

Issue 431473003: Make all views platforms use 'new' (cros-style) wrench menu. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix compile errors Created 6 years, 4 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/toolbar/wrench_menu_model.h
diff --git a/chrome/browser/ui/toolbar/wrench_menu_model.h b/chrome/browser/ui/toolbar/wrench_menu_model.h
index 8a064ec9b31fe701f56e4fb97247c2c8b8106b62..965baf89a1026a94460dc69cdc9e7b635e036952 100644
--- a/chrome/browser/ui/toolbar/wrench_menu_model.h
+++ b/chrome/browser/ui/toolbar/wrench_menu_model.h
@@ -89,10 +89,7 @@ class WrenchMenuModel : public ui::SimpleMenuModel,
static const int kMinRecentTabsCommandId = 1001;
static const int kMaxRecentTabsCommandId = 1200;
- // TODO: remove |is_new_menu|.
- WrenchMenuModel(ui::AcceleratorProvider* provider,
- Browser* browser,
- bool is_new_menu);
+ WrenchMenuModel(ui::AcceleratorProvider* provider, Browser* browser);
virtual ~WrenchMenuModel();
// Overridden for ButtonMenuItemModel::Delegate:
@@ -144,23 +141,20 @@ class WrenchMenuModel : public ui::SimpleMenuModel,
WrenchMenuModel();
- void Build(bool is_new_menu);
+ void Build();
void AddGlobalErrorMenuItems();
// Appends everything needed for the clipboard menu: a menu break, the
- // clipboard menu content and the finalizing menu break. If the last break
- // is not needed it can be suppressed by setting |new_menu|
- // to false.
- void CreateCutCopyPasteMenu(bool new_menu);
+ // clipboard menu content and the finalizing menu break.
+ void CreateCutCopyPasteMenu();
// Add a menu item for the extension icons.
void CreateExtensionToolbarOverflowMenu();
// Appends everything needed for the zoom menu: a menu break, then the zoom
- // menu content and then another menu break. If the new menu type is used,
- // |new_menu| should be set to true.
- void CreateZoomMenu(bool new_menu);
+ // menu content and then another menu break.
+ void CreateZoomMenu();
void OnZoomLevelChanged(const content::HostZoomMap::ZoomLevelChange& change);
« no previous file with comments | « chrome/browser/ui/cocoa/wrench_menu/wrench_menu_controller.mm ('k') | chrome/browser/ui/toolbar/wrench_menu_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698