711906 - Remove EventUtils.sendKey()

Status: NEW

(Core :: DOM: UI Events & Focus Handling, enhancement, P4)

Description

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

EventUtils.sendKey() just adds "VK_" prefix to the key name and calls synthesizeKey(). There is no reason which should be existing. We should remove it.

Comment 1

[Karun Dambiec]

I am not sure if it should be removed, as we are replacing uses of synthesizeKey with sendKey in Bug 720087.

Comment 2

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

(In reply to Karun Dambiec from comment #1)
> I am not sure if it should be removed, as we are replacing uses of
> synthesizeKey with sendKey in Bug 720087.

If we recommend these functions instead of synthesizeKey, we should make all of syntheizeKey feature available by them.

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #0)
> EventUtils.sendKey() just adds "VK_" prefix to the key name and calls
> synthesizeKey(). There is no reason which should be existing. We should
> remove it.

sendKey() is easier to use and goes along sendChar() and sendString()...


(In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #2)
> If we recommend these functions instead of synthesizeKey, we should make all
> of syntheizeKey feature available by them.

I don't know whether these functions need to support more features,
but we clearly need a module owner decision about whether to remove or use these functions.

Comment 4

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

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