diff options
author | afakhry <afakhry@chromium.org> | 2014-11-21 13:06:16 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-11-21 21:06:42 +0000 |
commit | 5546373f24680054ad4e1eca425b5b1b7291dfe9 (patch) | |
tree | 0a6a169e4256477a15c168038766f5cd3d8fc4c5 /win8 | |
parent | f3c56d51dac268caa4186b27eb3f738ce85b8d20 (diff) | |
download | chromium_src-5546373f24680054ad4e1eca425b5b1b7291dfe9.zip chromium_src-5546373f24680054ad4e1eca425b5b1b7291dfe9.tar.gz chromium_src-5546373f24680054ad4e1eca425b5b1b7291dfe9.tar.bz2 |
Regression: Search+Key pops up app launcher
We decided to fix this issue even though it only happens when the
following 'wrong' key sequence is pressed:
- SearchKey [pressed]
- Key [pressed]
- SearchKey [released]
- Key [released]
Resulted wrong behavior: the AppLancher appears.
The Cause:
----------
When the search key is used as a modifier, the following pressed key is
rewritten and consumed. It doesn't get forwarded to
AcceleratorController managed by the AcceleratorManager in the
FocusManager. The AcceleratorController needs to know the previous
Accelerator to determine the correct behavior.
The 'Suggested' Fix:
--------------------
Track the system-wide current and previous key accelerators and require
that the ToggleAppList would only be triggered if LWIN [pressed] is
immediately followed by LWIN [released].
BUG=410835
TEST=manual, ash_unittests
Review URL: https://codereview.chromium.org/727583002
Cr-Commit-Position: refs/heads/master@{#305278}
Diffstat (limited to 'win8')
0 files changed, 0 insertions, 0 deletions