marionette/event.js is still using nsIDOMWindowUtils.sendKeyEvent() which was already obsolete for a year (at least). When I work on bug 1154183, I hit this issue. Bug 1154183 will change shortcut key handling of reserved shortcut keys (e.g., Ctrl+T, Ctrl+W, Ctrl+N). They will be handled with keydown event. But charCode of keydown event is always 0. Therefore, native event handler and TextInputProcessor add some necessary information to WidgetKeyboardEvent which is internal class of DOM KeyboardEvent. However, nsIDOMWindowUtils.sendKeyEvent() cannot set such additional information due to its poor arguments. Therefore, all automated tests should use nsITextInputProcessor instead. Looks like that marionette/event.js is ported from EventUtils.js for mochitests. So, marionette staff should port current EventUtils.synthesizeKey() and _getTIP(). Andreas Tolfsen, could you take a look for this? Bug 1154183 should be fixed as soon as possible.

I changed the approach to fix bug 1154183, then, this becomes not necessary to fix soon. However, this should be done for bug 1134542, anyway. So, I still hope that somebody of Marionette staff tries to fix this bug.

nsIDOMWindowUtils.sendKeyEvent() has been made obsolete so this removes its use in Marionette Review commit: https://reviewboard.mozilla.org/r/51715/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/51715/

Can you confirm whether this patch is an import of (select features from) the newer EventUtils.js used by Mochitests?

(In reply to Andreas Tolfsen ‹:ato› from comment #4) > Can you confirm whether this patch is an import of (select features from) > the newer EventUtils.js used by Mochitests? Yup, literally copied things over and then "restyled" them to match the rest of the file.

Comment on attachment 8750981 [details] MozReview Request: Bug 1255806: remove use of nsIDOMWindowUtils.sendKeyEvent from Marionette r?ato https://reviewboard.mozilla.org/r/51715/#review49081 Well try appears to be passing, which wasn’t the case when I tried to import the whole, new Mochitest EventUtils.js file. Please ship it.