| commit | 8950bff5d0440600cfb794d3f93c4dc332ce7970 | |
| author | skuhne@chromium.org <skuhne@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Thu, 12 Sep 2013 21:09:05 +0000 (12 21:09 +0000) | ||
| committer | skuhne@chromium.org <skuhne@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Thu, 12 Sep 2013 21:09:05 +0000 (12 21:09 +0000) | ||
| tree | faeb50a297eb23c3f774b09792f546b39f59da52 | treesnapshot (tar.gz zip) |
| parent | ed118a8f2b0dc81512753f1756da307dae33dcf6 | commitdiff |
Fixing problem where opening a menu while closing another can close the new menu unexpectedly
The problem was as follows:
When the app list menu gets closed it gets closed delayed. The right click menu opens. Once the menu got animated out, it will destruct itself. When doing so it will surrender its activation state to the system and the menu controller will close all menus.
To avoid this I remove the activation state before the app list gets destroyed. (At that point in time it is not active anymore anyways). I also tried to set the focus in the menu runner on its parent - but that widget is not activatable and that does not work.
BUG=285785
TEST=visual
Review URL: https://chromiumcodereview.appspot.com/23457008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222868 0039d316-1c4b-4281-b951-d872f2087c98
The problem was as follows:
When the app list menu gets closed it gets closed delayed. The right click menu opens. Once the menu got animated out, it will destruct itself. When doing so it will surrender its activation state to the system and the menu controller will close all menus.
To avoid this I remove the activation state before the app list gets destroyed. (At that point in time it is not active anymore anyways). I also tried to set the focus in the menu runner on its parent - but that widget is not activatable and that does not work.
BUG=285785
TEST=visual
Review URL: https://chromiumcodereview.appspot.com/23457008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222868 0039d316-1c4b-4281-b951-d872f2087c98
| ash/wm/app_list_controller.cc | diffblobblamehistory |