Bug 1274503 part.1 AKEYCODE_ASSIST shouldn't be mapped to "Help" r?smaug draft
authorMasayuki Nakano <masayuki@d-toybox.com>
Fri, 04 Nov 2016 21:08:04 +0900
changeset 446501 bbcce8f4da84afabf3ace384a68a147990f6c60b
parent 446500 f3af723aef25d78b086187d283a55e2801c5841f
child 446502 8b6f5ffe5b112097656e2d3161b2803c1dd75684
push id37817
push usermasayuki@d-toybox.com
push dateThu, 01 Dec 2016 15:00:58 +0000
reviewerssmaug
bugs1274503
milestone53.0a1
Bug 1274503 part.1 AKEYCODE_ASSIST shouldn't be mapped to "Help" r?smaug I mapped AKEYCODE_ASSIST to "Help". However, Chromium doesn't map the keycode to anything. Additionally, Android has a new keycode AKEYCODE_HELP and Chromium maps it to "Help". So, we should use Chromium's mapping for it. MozReview-Commit-ID: 19zTP0xclYI
widget/NativeKeyToDOMKeyName.h
--- a/widget/NativeKeyToDOMKeyName.h
+++ b/widget/NativeKeyToDOMKeyName.h
@@ -358,17 +358,16 @@ KEY_MAP_GTK     (Execute, GDK_Execute)
 KEY_MAP_WIN_CMD (Find, APPCOMMAND_FIND)
 KEY_MAP_GTK     (Find, GDK_Find)
 
 // Help
 KEY_MAP_WIN     (Help, VK_HELP)
 KEY_MAP_WIN_CMD (Help, APPCOMMAND_HELP)
 KEY_MAP_COCOA   (Help, kVK_Help)
 KEY_MAP_GTK     (Help, GDK_Help)
-KEY_MAP_ANDROID (Help, AKEYCODE_ASSIST)
 
 // Pause
 KEY_MAP_WIN     (Pause, VK_PAUSE)
 KEY_MAP_GTK     (Pause, GDK_Pause)
 // Break is typically mapped to Alt+Pause or Ctrl+Pause on GTK.
 KEY_MAP_GTK     (Pause, GDK_Break)
 KEY_MAP_ANDROID (Pause, AKEYCODE_BREAK)