Note: There are a few cases of duplicates in user autocompletion which are being worked on.

test_frames.html depends on SimpleTest.waitForFocus(), which makes the test timing-sensitive

RESOLVED FIXED in mozilla8

Status

()

Core
Disability Access APIs
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: hsivonen, Unassigned)

Tracking

(Blocks: 1 bug, {intermittent-failure})

Trunk
mozilla8
intermittent-failure
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

http://mxr-test.konigsberg.mozilla.org/mozilla-central/source/accessible/tests/mochitest/states/test_frames.html?force=1 calls addA11yLoadEvent which in turn calls SimpleTest.waitForFocus().

SimpleTest.waitForFocus() doesn't really do what it's advertised to do, so the test as a whole becomes timing-sensitive. See bug 554873.

Updated

7 years ago
Depends on: 554873
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Blocks: 373864
I tried to call addA11yLoadEvent from onload, which has worked in the past for working around waitForFocus brokenness, but that simple approach doesn't work here.
Created attachment 442372 [details] [diff] [review]
Disable test_frames.html

In case a real fix isn't available by Monday, I suggest disabling this test so that the HTML5 parser can be enabled.
Attachment #442372 - Flags: review?(jst)
I think it is really important to get the HTML5 parser turned on by default so that we can catch and triage other bugs that we don't know about (or cover in our mochitests). So yes, please simply disable this test with your landing (unless this bug is fixed).

Henri, just out of interest, do you know what is different about the html5 parser that is more frequently exposing the weakness of SimpleTest.waitForFocus? Does the problem still occur when the parser is on the main thread?
(In reply to comment #7)
> I think it is really important to get the HTML5 parser turned on by default so
> that we can catch and triage other bugs that we don't know about (or cover in
> our mochitests). So yes, please simply disable this test with your landing
> (unless this bug is fixed).

Thanks.

> Henri, just out of interest, do you know what is different about the html5
> parser that is more frequently exposing the weakness of
> SimpleTest.waitForFocus? 

The timing of things may differ a little, so the moment when the docshell navigates away from about:blank can be different relative to other asynchronous things that are going on.

> Does the problem still occur when the parser is on the main thread?

This particular test works for me locally and on the tryserver. It failed only on the real tinderbox where the the freedom to experiment with the single-threaded mode isn't available.
Created attachment 443053 [details] [diff] [review]
Disable test_frames.html, with comment
Attachment #442372 - Attachment is obsolete: true
Attachment #443053 - Flags: review?(roc)
Attachment #442372 - Flags: review?(jst)
Attachment #443053 - Flags: review?(roc) → review+
Test disabled:
http://hg.mozilla.org/mozilla-central/rev/1f551cd5803b
Whiteboard: [orange] → [orange][test disabled]

Comment 11

7 years ago
I enabled tests as part of bug 638106, it sounds they don't fail anymore.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 12

7 years ago
sorry it should be worksforme, these tests never were dependent on SimpleTest.waitForFocus() explicitly, at least no more than other a11y tests, a11y in fx4 is not so sensible to timing issues so I assume the original problem has gone.
Resolution: FIXED → WORKSFORME
Whiteboard: [orange][test disabled] → [orange]
Comment hidden (Treeherder Robot)
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Updated

6 years ago
Blocks: 518190
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Comment 76

6 years ago
Bug 676213 is a fix for this bug because nsDocAccessible::mContent is not updated (see patch of bug 638106) yet what means the document tree is not created at this point, so that the test is running too early.
Depends on: 676213
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Comment 86

6 years ago
fixed by bug 676213 (mozilla 8), please reopen if you see the issue
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
(Assignee)

Updated

5 years ago
Keywords: intermittent-failure
(Assignee)

Updated

5 years ago
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.