455840 - change test_nsIAccessible_focus.html mochitest to use eventQueue

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Marco Zehe (:MarcoZ)]

In test_nsIAccessible_focus.html, when run with all tests, the test for focused state on the "aria-link" element fails. However right prior to that, the test for the corresponding dom node having focus passes.

Determine why getFinalState returns the wrong info in this case.

I already tried prolonging the timeout in this window.setTimeout function block, but even at 250, which is a quarter of a second, the result is no different, so the state is apparently not set at all.

The test is currently marked as known fail and wrapped in an if statement so the other tests for this state still have a chance to succeed.

Comment 1

[David Bolter [:davidb] (NeedInfo me for attention)]

Attachment: patch

I can't recreate this bug.

Comment 2

[Marco Zehe (:MarcoZ)]

Comment on attachment 403277 [details] [diff] [review]
patch

After discussing on IRC, we think it makes sense to tie this file into our new event handling rather than patching it up the other way. Cancelling review request.

Comment 3

[alexander :surkov (:asurkov)]

Marco, do we want to do anything here?

Comment 4

[Marco Zehe (:MarcoZ)]

We should see if we have an equivalent test like the one indicated in the patch. If that works, close as WFM.

Comment 5

[alexander :surkov (:asurkov)]

Attachment: patch

change mochitest to use eventQueue

Comment 6

[Marco Zehe (:MarcoZ)]

Comment on attachment 518630 [details] [diff] [review]
patch

Yes, that's looking good, thanks! r=me

Comment 7

[alexander :surkov (:asurkov)]

landed 2.0 - http://hg.mozilla.org/mozilla-central/rev/b78e947a68e5