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

Unified Diff: chrome/browser/ui/views/toolbar/wrench_menu.cc

Issue 331993009: MacViews: Run native Cocoa context menus to support Services. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add VIEWS_EXPORT for unit test access Created 6 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/toolbar/wrench_menu.cc
diff --git a/chrome/browser/ui/views/toolbar/wrench_menu.cc b/chrome/browser/ui/views/toolbar/wrench_menu.cc
index 2ec6147031c8350df15da7d52933302083848bf1..534db67150bf0cff8afbfcf61dc94df4b68451a5 100644
--- a/chrome/browser/ui/views/toolbar/wrench_menu.cc
+++ b/chrome/browser/ui/views/toolbar/wrench_menu.cc
@@ -985,7 +985,8 @@ void WrenchMenu::Init(ui::MenuModel* model) {
DCHECK(command_id_to_entry_.find(i) == command_id_to_entry_.end());
#endif // defined(DEBUG)
- menu_runner_.reset(new views::MenuRunner(root_));
+ menu_runner_.reset(
+ new views::MenuRunner(root_, views::MenuRunner::HAS_MNEMONICS));
}
void WrenchMenu::RunMenu(views::MenuButton* host) {
@@ -997,8 +998,7 @@ void WrenchMenu::RunMenu(views::MenuButton* host) {
host,
bounds,
views::MENU_ANCHOR_TOPRIGHT,
- ui::MENU_SOURCE_NONE,
- views::MenuRunner::HAS_MNEMONICS) ==
+ ui::MENU_SOURCE_NONE) ==
views::MenuRunner::MENU_DELETED)
return;
if (bookmark_menu_delegate_.get()) {

Powered by Google App Engine
This is Rietveld 408576698