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

Unified Diff: ui/views/controls/menu/menu_runner_impl.cc

Issue 413323002: Resubmit: Open the WrenchMenu on mouseover when dragging a browser action (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix 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/views/controls/menu/menu_runner_impl.cc
diff --git a/ui/views/controls/menu/menu_runner_impl.cc b/ui/views/controls/menu/menu_runner_impl.cc
index 9fda83d0d88f7a131f458560238aaf40e49e66cd..4167be997e75d6b1a68d7074f1e31b62e7bdaefd 100644
--- a/ui/views/controls/menu/menu_runner_impl.cc
+++ b/ui/views/controls/menu/menu_runner_impl.cc
@@ -105,8 +105,7 @@ MenuRunner::RunResult MenuRunnerImpl::RunMenuAt(Widget* parent,
running_ = true;
for_drop_ = (run_types & MenuRunner::FOR_DROP) != 0;
- bool has_mnemonics =
- (run_types & MenuRunner::HAS_MNEMONICS) != 0 && !for_drop_;
+ bool has_mnemonics = (run_types & MenuRunner::HAS_MNEMONICS) != 0;
owns_controller_ = false;
if (!controller) {
// No menus are showing, show one.
@@ -131,6 +130,7 @@ MenuRunner::RunResult MenuRunnerImpl::RunMenuAt(Widget* parent,
bounds,
anchor,
(run_types & MenuRunner::CONTEXT_MENU) != 0,
+ (run_types & MenuRunner::NESTED_DRAG) != 0,
&mouse_event_flags);
// Get the time of the event which closed this menu.
closing_event_time_ = controller->closing_event_time();

Powered by Google App Engine
This is Rietveld 408576698