1121878 - Fix some missing map between WM_APPCOMMAND and KeyboardEvent.key

Status: RESOLVED FIXED

(Core :: Widget: Win32, defect)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

In Bug 865561, I mapped a lot of commands of WM_APPCOMMAND to KeyboardEvent.key. However, I missed to map some commands.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Patch

Current mapping is here:
https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent.key#Key_values_of_WM_APPCOMMAND_message_on_Windows

D3E doesn't define some key names for APPCOMMAND_BASS_BOOST, APPCOMMAND_CORRECTION_LIST, APPCOMMAND_DICTATE_OR_COMMAND_CONTROL_TOGGLE, APPCOMMAND_MIC_ON_OFF_TOGGLE, APPCOMMAND_MICROPHONE_VOLUME_DOWN, APPCOMMAND_MICROPHONE_VOLUME_MUTE, APPCOMMAND_MICROPHONE_VOLUME_UP, APPCOMMAND_TREBLE_DOWN and APPCOMMAND_TREBLE_UP.

So, APPCOMMAND_COPY, APPCOMMAND_CUT, APPCOMMAND_MEDIA_CHANNEL_DOWN, APPCOMMAND_MEDIA_CHANNEL_UP and APPCOMMAND_MEDIA_FAST_FORWARD should be mapped.

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/836b7ff0d7e2

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Comment on attachment 8549504 [details] [diff] [review]
Patch

Approval Request Comment
[Feature/regressing bug #]: This is a follow up bug of bug 865561.
[User impact if declined]: Web developers don't want small changes of our DOM event behavior repeatedly. So, uplifting this to 37 make the changes once. 
[Describe test coverage new/current, TBPL]: Bug 865561 was landed in 37 cycle. (about a half of a month ago) And this is adding key mapping which are forgotten in bug 865561.
[Risks and why]: As I said above, Uplifting this makes web developers happy since they don't need to test/check our behavior both 37 and 38 for keydown, keypress and keyup event handling. If we don't uplift this, similar changes occur both in 37 and 38. It's too bad for them and makes key value table in MDN messy.
[String/UUID change made/needed]: Nothing.

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/836b7ff0d7e2

Comment 5

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8549504 [details] [diff] [review]
Patch

Aurora+

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/0281145a7ef5

Comment 7

[Jean-Yves Perrier [:teoli]]

:masayuki. I think the documentation is up-to-date (Thank you for doing it, it is a gory one, but so much useful!)

Can you confirm/infirm it?

Comment 8

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Yeah, I did it.