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)]

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][[e-mail]]]

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)]

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)]

(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)]

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)]

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)]

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

Comment 8

[BugBot [:suhaib / :marco/ :calixte]]

Clear a needinfo that is pending on an inactive user.

Inactive users most likely will not respond; if the missing information is essential and cannot be collected another way, the bug maybe should be closed as INCOMPLETE.

For more information, please visit BugBot documentation.