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

Unified Diff: trunk/src/chrome/browser/ui/views/menu_test_base.cc

Issue 416903002: Revert 285142 "Open the WrenchMenu on mouseover when dragging a ..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: 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: trunk/src/chrome/browser/ui/views/menu_test_base.cc
===================================================================
--- trunk/src/chrome/browser/ui/views/menu_test_base.cc (revision 285300)
+++ trunk/src/chrome/browser/ui/views/menu_test_base.cc (working copy)
@@ -37,26 +37,18 @@
false, false, next);
}
-int MenuTestBase::GetMenuRunnerFlags() {
- return views::MenuRunner::HAS_MNEMONICS;
-}
-
void MenuTestBase::SetUp() {
button_ = new views::MenuButton(
NULL, base::ASCIIToUTF16("Menu Test"), this, true);
menu_ = new views::MenuItemView(this);
BuildMenu(menu_);
- menu_runner_.reset(new views::MenuRunner(menu_, GetMenuRunnerFlags()));
+ menu_runner_.reset(
+ new views::MenuRunner(menu_, views::MenuRunner::HAS_MNEMONICS));
ViewEventTestBase::SetUp();
}
void MenuTestBase::TearDown() {
- // We cancel the menu first because certain operations (like a menu opened
- // with views::MenuRunner::FOR_DROP) don't take kindly to simply pulling the
- // runner out from under them.
- menu_runner_->Cancel();
-
menu_runner_.reset();
menu_ = NULL;
ViewEventTestBase::TearDown();
« no previous file with comments | « trunk/src/chrome/browser/ui/views/menu_test_base.h ('k') | trunk/src/chrome/browser/ui/views/menu_view_drag_and_drop_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698