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

Unified Diff: chrome/browser/ui/views/content_setting_bubble_contents.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/content_setting_bubble_contents.cc
diff --git a/chrome/browser/ui/views/content_setting_bubble_contents.cc b/chrome/browser/ui/views/content_setting_bubble_contents.cc
index 1045d27281f17b728212f3b09bec83fd603e6175..f662be4e4fe4b892f8eda09e5e45528fc72e6999 100644
--- a/chrome/browser/ui/views/content_setting_bubble_contents.cc
+++ b/chrome/browser/ui/views/content_setting_bubble_contents.cc
@@ -468,7 +468,8 @@ void ContentSettingBubbleContents::OnMenuButtonClicked(
MediaMenuPartsMap::iterator j(media_menus_.find(
static_cast<views::MenuButton*>(source)));
DCHECK(j != media_menus_.end());
- menu_runner_.reset(new views::MenuRunner(j->second->menu_model.get()));
+ menu_runner_.reset(new views::MenuRunner(j->second->menu_model.get(),
+ views::MenuRunner::HAS_MNEMONICS));
gfx::Point screen_location;
views::View::ConvertPointToScreen(j->first, &screen_location);
@@ -477,8 +478,7 @@ void ContentSettingBubbleContents::OnMenuButtonClicked(
j->first,
gfx::Rect(screen_location, j->first->size()),
views::MENU_ANCHOR_TOPLEFT,
- ui::MENU_SOURCE_NONE,
- views::MenuRunner::HAS_MNEMONICS));
+ ui::MENU_SOURCE_NONE));
}
int ContentSettingBubbleContents::GetPreferredMediaMenuWidth(

Powered by Google App Engine
This is Rietveld 408576698