Closed Bug 651628 Opened 13 years ago Closed 13 years ago

Four tests in /accessible/tests/mochitest/ have been 'temporarily disabled' since 2009

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla7

People

(Reporter: dholbert, Assigned: surkov)

References

(Depends on 1 open bug, Blocks 1 open bug, )

Details

Attachments

(3 files)

I just noticed this output go by in my build output:
> Makefile:120: test_childAtPoint.html temporarily disabled
> Makefile:120: test_childAtPoint.xul temporarily disabled
> Makefile:120: test_elm_media.html temporarily disabled
> Makefile:120: test_nsIAccessible_comboboxes.xul temporarily disabled

These tests were all disabled (in 4 separate changesets*) in 2009, so I think "temporarily" is stretching it a bit. :)

We probably don't want to leave these tests disabled indefinitely, so I'm filing this bug on fixing them up.

* The 4 test-disabling csets were:

http://hg.mozilla.org/mozilla-central/diff/fff539595608/accessible/tests/mochitest/Makefile.in
http://hg.mozilla.org/mozilla-central/diff/d550af7a2c63/accessible/tests/mochitest/Makefile.in
http://hg.mozilla.org/mozilla-central/diff/324d492445e3/accessible/tests/mochitest/Makefile.in
http://hg.mozilla.org/mozilla-central/diff/d3ee07a4c551/accessible/tests/mochitest/Makefile.in
Blocks: a11ytestdev
no idea why David disabled them, two changesets, one them doesn't refer to any bug, another one points to bug 482598 where there's no any reason or any reference as well.
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Attachment #535040 - Flags: review?(marco.zehe)
Comment on attachment 535040 [details] [diff] [review]
enable childAtPoint

r=me. I seem to remember there was a problem with one of these tests being tied to a crashtest somehow, but don't remember the specifics. May long be fixed. Get these re-enabled and we'll see how they fare.
Attachment #535040 - Flags: review?(marco.zehe) → review+
Attachment #535292 - Flags: review?(marco.zehe)
Attachment #535296 - Flags: review?(marco.zehe)
Comment on attachment 535292 [details] [diff] [review]
enable comboboxes states

r=me thanks!
Attachment #535292 - Flags: review?(marco.zehe) → review+
Comment on attachment 535296 [details] [diff] [review]
enable media test

r=me, cool thanks!
Attachment #535296 - Flags: review?(marco.zehe) → review+
comboboxes states enabled - http://hg.mozilla.org/mozilla-central/rev/81b8f46ad93f

media actions enabled - http://hg.mozilla.org/mozilla-central/rev/dfbd3c6e3683
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: