1263512 - [UI Events-key] Investigate KeyboardEvent.key difference between Chromium and Gecko

Status: NEW

(Core :: DOM: Events, defect)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(Got a cold, working slower)]

Chromium will ship KeyboardEvent.key in releases.
https://bugs.chromium.org/p/chromium/issues/detail?id=227231#c81

So, we should investigate the difference between Chromium and Gecko and fix the difference or report the issue to Chromium team.

Comment 1

[Olli Pettay [:smaug]]

On Windows better to compare also to Edge (assuming it has .key).

Comment 2

[garykac]

Edge does not really have |key| support yet.

It does have the |key| attribute and the current W3C spec was based originally on the IE implementation, but there have been many changes to the spec that have not yet been added to IE/Edge.

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(Got a cold, working slower)]

Filed about WM_APPCOMMAND issue on Chromium:
https://bugs.chromium.org/p/chromium/issues/detail?id=611682

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(Got a cold, working slower)]

(In reply to Masayuki Nakano [:masayuki] (Mozilla Japan) (offline: 4/29-5/8) from comment #3)
> Filed about WM_APPCOMMAND issue on Chromium:
> https://bugs.chromium.org/p/chromium/issues/detail?id=611682

Oops, this report is wrong. It's environment dependent issue. Chromium works fine for those keys. I'll check the key value difference of them later.

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(Got a cold, working slower)]

Filed a lot of differences...
https://bugs.chromium.org/p/chromium/issues/detail?id=612694
https://bugs.chromium.org/p/chromium/issues/detail?id=612732
https://bugs.chromium.org/p/chromium/issues/detail?id=612736
https://bugs.chromium.org/p/chromium/issues/detail?id=612737
https://bugs.chromium.org/p/chromium/issues/detail?id=612743
https://bugs.chromium.org/p/chromium/issues/detail?id=612749

Looks like that Chrome's KeyboardEvent.key implementation is wrong. It refers scancode rather than virtual key code. Therefore, the implementation is really broken...

Gary, could you stop releasing KeyboardEvent.key in release build?

Comment 6

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(Got a cold, working slower)]

https://bugs.chromium.org/p/chromium/issues/detail?id=613152
https://bugs.chromium.org/p/chromium/issues/detail?id=613156

Comment 7

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(Got a cold, working slower)]

Resetting assignee which I don't work on in this several months.