GitLab

This change is a fixed version of
<https://chromium-review.googlesource.com/c/1334147>.

This change:

1) Has MenuController consider command to be an accelerator key, along with
   control and alt, for the purpose of not searching for mnemonics;
2) Has MenuController stop handling a key press, and cancel the menu,
   when the key press looks like it is a window-level accelerator instead.

Testing:
1) Open the app menu
2) With the app menu still open, hit cmd-T
3) The app menu should dismiss
4) A new tab should appear

Bug: 885138
Change-Id: Ib9c9e1517ae9ad3f9ec55eadb9bdfe61762df4c5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1742520
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Auto-Submit: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#685346}