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

Unified Diff: chrome/browser/tab_menu_model.cc

Issue 3056003: Attemp 2 at: (Closed)
Patch Set: Fix chromeos breakage Created 10 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
« no previous file with comments | « chrome/browser/tab_menu_model.h ('k') | chrome/browser/tab_menu_model_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/tab_menu_model.cc
diff --git a/chrome/browser/tab_menu_model.cc b/chrome/browser/tab_menu_model.cc
index 41667b20bf606418eb9575dc427b5937119a9f18..6b8dc17e973bdc2f4a2a7d3e99cac2bbad86f4cd 100644
--- a/chrome/browser/tab_menu_model.cc
+++ b/chrome/browser/tab_menu_model.cc
@@ -10,11 +10,9 @@
#include "grit/generated_resources.h"
TabMenuModel::TabMenuModel(menus::SimpleMenuModel::Delegate* delegate,
- bool is_pinned,
- bool allow_toolbar_toggle,
- bool is_toolbar_visible)
+ bool is_pinned)
: menus::SimpleMenuModel(delegate) {
- Build(is_pinned, allow_toolbar_toggle, is_toolbar_visible);
+ Build(is_pinned);
}
// static
@@ -29,8 +27,7 @@ bool TabMenuModel::AreVerticalTabsEnabled() {
#endif
}
-void TabMenuModel::Build(bool is_pinned, bool allow_toolbar_toggle,
- bool is_toolbar_visible) {
+void TabMenuModel::Build(bool is_pinned) {
AddItemWithStringId(TabStripModel::CommandNewTab, IDS_TAB_CXMENU_NEWTAB);
AddSeparator();
AddItemWithStringId(TabStripModel::CommandReload, IDS_TAB_CXMENU_RELOAD);
@@ -39,12 +36,6 @@ void TabMenuModel::Build(bool is_pinned, bool allow_toolbar_toggle,
AddItemWithStringId(
TabStripModel::CommandTogglePinned,
is_pinned ? IDS_TAB_CXMENU_UNPIN_TAB : IDS_TAB_CXMENU_PIN_TAB);
- if (allow_toolbar_toggle) {
- AddItemWithStringId(
- TabStripModel::CommandToggleToolbar,
- is_toolbar_visible ? IDS_TAB_CXMENU_HIDE_TOOLBAR :
- IDS_TAB_CXMENU_SHOW_TOOLBAR);
- }
AddSeparator();
AddItemWithStringId(TabStripModel::CommandCloseTab,
IDS_TAB_CXMENU_CLOSETAB);
« no previous file with comments | « chrome/browser/tab_menu_model.h ('k') | chrome/browser/tab_menu_model_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698