1566663 - Check event modifiers (or lack thereof) are handled appropriately in de-XBLed <handler>s

Status: NEW

(Toolkit :: UI Widgets, task)

Description

[Geoff Lankow (:darktrojan)]

In XBL, a <handler> function would only run if the event's key state matched the modifiers attribute. A lot of these have been replaced with addEventListener functions which run whatever the key state. This may result in unintended behaviour, especially if event.preventDefault() or event.stopPropagation() is called in the function. Bug 1561650 comment 7 for example.

It might be worthwhile checking these migrations. OTOH, waiting until bugs appear might also be acceptable.