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

Unified Diff: chrome/browser/ui/views/bookmarks/bookmark_editor_view.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: chrome/browser/ui/views/bookmarks/bookmark_editor_view.cc
diff --git a/chrome/browser/ui/views/bookmarks/bookmark_editor_view.cc b/chrome/browser/ui/views/bookmarks/bookmark_editor_view.cc
index 220e34ba4ff2100af02bf9966b810ffa65e7d541..022aa5a00f0c3ecf1e696904a95f3c5b00602f05 100644
--- a/chrome/browser/ui/views/bookmarks/bookmark_editor_view.cc
+++ b/chrome/browser/ui/views/bookmarks/bookmark_editor_view.cc
@@ -246,18 +246,12 @@ void BookmarkEditorView::ShowContextMenuForView(
tree_model_->GetRoot());
context_menu_runner_.reset(new views::MenuRunner(
- GetMenuModel(), views::MenuRunner::HAS_MNEMONICS |
- views::MenuRunner::CONTEXT_MENU |
- views::MenuRunner::ASYNC));
-
- if (context_menu_runner_->RunMenuAt(source->GetWidget()->GetTopLevelWidget(),
- NULL,
- gfx::Rect(point, gfx::Size()),
- views::MENU_ANCHOR_TOPRIGHT,
- source_type) ==
- views::MenuRunner::MENU_DELETED) {
- return;
- }
+ GetMenuModel(),
+ views::MenuRunner::HAS_MNEMONICS | views::MenuRunner::CONTEXT_MENU));
+
+ context_menu_runner_->RunMenuAt(source->GetWidget()->GetTopLevelWidget(),
+ NULL, gfx::Rect(point, gfx::Size()),
+ views::MENU_ANCHOR_TOPRIGHT, source_type);
}
const char* BookmarkEditorView::GetClassName() const {

Powered by Google App Engine
This is Rietveld 408576698