native key event handling tests should use synthesizeNativeKey of EventUtils

RESOLVED FIXED in mozilla29

Status

()

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: masayuki, Assigned: masayuki)

Tracking

Trunk
mozilla29
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

The second patch of bug 930374 introduces synthesizeNativeKey of EventUtils. All tests testing native key event handling should use it.
Created attachment 8344650 [details] [diff] [review]
Patch

Mac's French keyboard case for Mac of test_keycodes.xul hasn't work! This also fixes the bug and add tests for it.
Attachment #8344650 - Flags: review?(bugs)

Comment 2

5 years ago
Comment on attachment 8344650 [details] [diff] [review]
Patch

Hmm, this is a massive patch.

Could you explain the changes to TextInputHandler.mm.
Do those affect to tests only?
(In reply to Olli Pettay [:smaug] from comment #2)
> Comment on attachment 8344650 [details] [diff] [review]
> Patch
> 
> Hmm, this is a massive patch.
> 
> Could you explain the changes to TextInputHandler.mm.
> Do those affect to tests only?

Yes, it runs only when nsIDOMWindowUtils::SendNativeKeyEvent() is called. The method loads a keyboard layout from the specified integer value which is defined by us. At handling native keyboard events, TextInputHanlder gets keyboard layout information with other APIs which get _current_ state.

Comment 4

5 years ago
Comment on attachment 8344650 [details] [diff] [review]
Patch

rs=me, make sure to run these tests few times on try before landing.
Attachment #8344650 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/2700df154644
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.