diff options
author | pkotwicz@chromium.org <pkotwicz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-19 21:50:54 +0000 |
---|---|---|
committer | pkotwicz@chromium.org <pkotwicz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-19 21:50:54 +0000 |
commit | 7c07b341f0df31812b41888ef45ef20f0e3d6ad7 (patch) | |
tree | 2b69b4eb6e4b4cc98954d33f1756ec4e596da1da | |
parent | 70d5513d2ab3401b749e3756e4cfbfc787eb8c79 (diff) | |
download | chromium_src-7c07b341f0df31812b41888ef45ef20f0e3d6ad7.zip chromium_src-7c07b341f0df31812b41888ef45ef20f0e3d6ad7.tar.gz chromium_src-7c07b341f0df31812b41888ef45ef20f0e3d6ad7.tar.bz2 |
Do not repost events when the menu is closed for Linux Aura.
This CL ensures that we do not activate a new window when dismissing the context menu in all cases.
Without this CL:
- Clicking on a browser window from the wrench menu activates the window
- Clicking on non-browser window (e.g. terminal) does not activate the window
BUG=349069
TEST=Manual, see comment #20 on the bug
Review URL: https://codereview.chromium.org/204253002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@258112 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | ui/views/controls/menu/menu_controller.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/ui/views/controls/menu/menu_controller.cc b/ui/views/controls/menu/menu_controller.cc index ec467ec..16f2ad3 100644 --- a/ui/views/controls/menu/menu_controller.cc +++ b/ui/views/controls/menu/menu_controller.cc @@ -934,14 +934,15 @@ void MenuController::SetSelectionOnPointerDown(SubmenuView* source, } Cancel(exit_type); -#if !defined(OS_WIN) +#if defined(OS_CHROMEOS) // We're going to exit the menu and want to repost the event so that is // is handled normally after the context menu has exited. We call // RepostEvent after Cancel so that mouse capture has been released so // that finding the event target is unaffected by the current capture. RepostEvent(source, event); #endif - + // Do not repost events for Linux Aura because this behavior is more + // consistent with the behavior of other Linux apps. return; } |