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

Unified Diff: chrome/browser/ui/views/menu_model_adapter_test.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/menu_model_adapter_test.cc
diff --git a/chrome/browser/ui/views/menu_model_adapter_test.cc b/chrome/browser/ui/views/menu_model_adapter_test.cc
index 732e2fd14e9f8c3bb40dfae720722a3fe46b9c7d..1bde65a20624f93fe0fabe9d8e2347df6751499a 100644
--- a/chrome/browser/ui/views/menu_model_adapter_test.cc
+++ b/chrome/browser/ui/views/menu_model_adapter_test.cc
@@ -174,8 +174,8 @@ class MenuModelAdapterTest : public ViewEventTestBase,
this, true);
menu_ = menu_model_adapter_.CreateMenu();
- menu_runner_.reset(new views::MenuRunner(
- menu_, views::MenuRunner::HAS_MNEMONICS | views::MenuRunner::ASYNC));
+ menu_runner_.reset(
+ new views::MenuRunner(menu_, views::MenuRunner::HAS_MNEMONICS));
ViewEventTestBase::SetUp();
}
@@ -199,11 +199,8 @@ class MenuModelAdapterTest : public ViewEventTestBase,
gfx::Point screen_location;
views::View::ConvertPointToScreen(source, &screen_location);
gfx::Rect bounds(screen_location, source->size());
- ignore_result(menu_runner_->RunMenuAt(source->GetWidget(),
- button_,
- bounds,
- views::MENU_ANCHOR_TOPLEFT,
- ui::MENU_SOURCE_NONE));
+ menu_runner_->RunMenuAt(source->GetWidget(), button_, bounds,
+ views::MENU_ANCHOR_TOPLEFT, ui::MENU_SOURCE_NONE);
}
// ViewEventTestBase implementation
« no previous file with comments | « chrome/browser/ui/views/frame/opaque_browser_frame_view.cc ('k') | chrome/browser/ui/views/menu_test_base.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698