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

Unified Diff: components/renderer_context_menu/views/toolkit_delegate_views.cc

Issue 2790773002: Cleanup MenuRunner API (Closed)
Patch Set: Rebase Created 3 years, 8 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: components/renderer_context_menu/views/toolkit_delegate_views.cc
diff --git a/components/renderer_context_menu/views/toolkit_delegate_views.cc b/components/renderer_context_menu/views/toolkit_delegate_views.cc
index 81dfe55f56040baca013071fad39a639f823cc37..1e5357c08dea2b7244cf6cd2b2faaf9f81879d30 100644
--- a/components/renderer_context_menu/views/toolkit_delegate_views.cc
+++ b/components/renderer_context_menu/views/toolkit_delegate_views.cc
@@ -22,17 +22,16 @@ void ToolkitDelegateViews::RunMenuAt(views::Widget* parent,
type == ui::MENU_SOURCE_TOUCH_EDIT_MENU)
? views::MENU_ANCHOR_BOTTOMCENTER
: views::MENU_ANCHOR_TOPLEFT;
- ignore_result(menu_runner_->RunMenuAt(
- parent, NULL, gfx::Rect(point, gfx::Size()), anchor_position, type));
+ menu_runner_->RunMenuAt(parent, NULL, gfx::Rect(point, gfx::Size()),
+ anchor_position, type);
}
void ToolkitDelegateViews::Init(ui::SimpleMenuModel* menu_model) {
menu_adapter_.reset(new views::MenuModelAdapter(menu_model));
menu_view_ = menu_adapter_->CreateMenu();
- menu_runner_.reset(
- new views::MenuRunner(menu_view_, views::MenuRunner::HAS_MNEMONICS |
- views::MenuRunner::CONTEXT_MENU |
- views::MenuRunner::ASYNC));
+ menu_runner_.reset(new views::MenuRunner(
+ menu_view_,
+ views::MenuRunner::HAS_MNEMONICS | views::MenuRunner::CONTEXT_MENU));
}
void ToolkitDelegateViews::Cancel() {

Powered by Google App Engine
This is Rietveld 408576698