summaryrefslogtreecommitdiffstats
path: root/win8
diff options
context:
space:
mode:
authorafakhry <afakhry@chromium.org>2014-11-21 13:06:16 -0800
committerCommit bot <commit-bot@chromium.org>2014-11-21 21:06:42 +0000
commit5546373f24680054ad4e1eca425b5b1b7291dfe9 (patch)
tree0a6a169e4256477a15c168038766f5cd3d8fc4c5 /win8
parentf3c56d51dac268caa4186b27eb3f738ce85b8d20 (diff)
downloadchromium_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