Closed Bug 615913 Opened 9 years ago Closed 9 years ago

fix focus hyperlink tests

Categories

(Core :: Disability Access APIs, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla2.0b8

People

(Reporter: surkov, Assigned: surkov)

References

(Blocks 1 open bug)

Details

(Keywords: access)

Attachments

(1 file)

Attached patch patchSplinter Review
after bug 498015 DOM focus event processing will happen async, we need to fit hyperlink tests for that.
Attachment #494432 - Flags: review?(marco.zehe)
Comment on attachment 494432 [details] [diff] [review]
patch

>-      testStates(normalHyperlinkAcc,
>-                 (STATE_FOCUSABLE | STATE_LINKED),
>-                 (EXT_STATE_HORIZONTAL));
>-      testFocus("NormalHyperlink", normalHyperlinkAcc, false, true);
>-      testStates(normalHyperlinkAcc,
>-                 (STATE_FOCUSABLE | STATE_FOCUSED | STATE_LINKED),
>-                 (EXT_STATE_HORIZONTAL));
>+      testStates(normalHyperlinkAcc, STATE_LINKED, EXT_STATE_HORIZONTAL);

Took me a moment to figure out that STATE_FOCUSABLE is now being tested in the checker. r=me. Thanks!
Attachment #494432 - Flags: review?(marco.zehe) → review+
landed on 2.0 - http://hg.mozilla.org/mozilla-central/rev/2adf67fdc19a
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
You need to log in before you can comment on or make changes to this bug.