GitLab

The logic in NSMenuItem(ChromeAdditions) attempts to mimic Cocoa's logic for
determining whether a NSMenuItem will fire for a given NSEvent.

In keyboard layouts that produce non-ascii characters, Cocoa uses the event's
"characters" property to process keyEquivalents, and will manually apply the
shift modifier if necessary. Our logic was failing to do so. This meant that key
combinations such as [cmd + shift + t] on 2-Set Korean would attempt to match a
NSMenuItem with keyEquivalent "t" rather than a NSMenuItem with keyEquivalent
"T".

Bug: 889970
Change-Id: I81fdf11c4ffa6322d78e981cdbff6956cd8f4b79
Reviewed-on: https://chromium-review.googlesource.com/1254462Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595453}