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

Unified Diff: ui/message_center/views/notifier_settings_view.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: ui/message_center/views/notifier_settings_view.cc
diff --git a/ui/message_center/views/notifier_settings_view.cc b/ui/message_center/views/notifier_settings_view.cc
index 4b12ede132d1ab07ecd3bf5216e5bbb408ab624c..80cb402e79af5b10d9567ef985bd1cfe6fac3205 100644
--- a/ui/message_center/views/notifier_settings_view.cc
+++ b/ui/message_center/views/notifier_settings_view.cc
@@ -686,8 +686,8 @@ void NotifierSettingsView::ButtonPressed(views::Button* sender,
void NotifierSettingsView::OnMenuButtonClicked(views::View* source,
const gfx::Point& point) {
notifier_group_menu_model_.reset(new NotifierGroupMenuModel(provider_));
- notifier_group_menu_runner_.reset(
- new views::MenuRunner(notifier_group_menu_model_.get()));
+ notifier_group_menu_runner_.reset(new views::MenuRunner(
+ notifier_group_menu_model_.get(), views::MenuRunner::CONTEXT_MENU));
gfx::Rect menu_anchor = source->GetBoundsInScreen();
menu_anchor.Inset(
gfx::Insets(0, kMenuWhitespaceOffset, 0, kMenuWhitespaceOffset));
@@ -696,8 +696,7 @@ void NotifierSettingsView::OnMenuButtonClicked(views::View* source,
notifier_group_selector_,
menu_anchor,
views::MENU_ANCHOR_BUBBLE_ABOVE,
- ui::MENU_SOURCE_MOUSE,
- views::MenuRunner::CONTEXT_MENU))
+ ui::MENU_SOURCE_MOUSE))
return;
MessageCenterView* center_view = static_cast<MessageCenterView*>(parent());
center_view->OnSettingsChanged();

Powered by Google App Engine
This is Rietveld 408576698