The default bug view has changed. See this FAQ.

Sporadic TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_bug371576-1.html | Wrong order of execution - got ABEF, expected ABCDEF

RESOLVED FIXED in mozilla12

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: dholbert, Assigned: hsivonen)

Tracking

({intermittent-failure})

Trunk
mozilla12
x86
Windows XP
intermittent-failure
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=7377708&tree=Mozilla-Inbound
Rev3 WINNT 5.1 mozilla-inbound debug test mochitests-1/5 on 2011-11-13 14:56:52 PST for push 7ff1026f9880
{
9112 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_bug371576-1.html | Wrong order of execution - got ABEF, expected ABCDEF
}
Henri has been poking at parser stuff a lot lately. Could this be a regression from that?
(Assignee)

Comment 2

5 years ago
This test has been bogus (by depending on a race condition) ever since bug 591981 got fixed. I'm surprised this hadn't gone orange sooner. I think the test should be removed, since there's no obvious way to change the test to make sense in the HTML5-compliant world.
So do we have tests that ensure that script execution happen in the correct order for both inline and DOM-inserted scripts? And that that order doesn't get messed up if someone sets innerHTML?
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Comment 6

5 years ago
(In reply to Jonas Sicking (:sicking) from comment #3)
> So do we have tests that ensure that script execution happen in the correct
> order for both inline and DOM-inserted scripts?

There's no "right" order for DOM-inserted scripts. That's why this test should be removed from the tree.
(Assignee)

Updated

5 years ago
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
(Assignee)

Comment 7

5 years ago
Created attachment 588397 [details] [diff] [review]
Remove the test

The test expect script-inserted scripts to have a deterministic execution order relative to parser-inserted scripts. The premise of the test is wrong per HTML5. See http://hsivonen.iki.fi/script-execution/

There's nothing to replace the test with, because the execution order is non-deterministic by spec.
Attachment #588397 - Flags: review?(bugs)
Comment on attachment 588397 [details] [diff] [review]
Remove the test

rs=me
Attachment #588397 - Flags: review?(bugs) → review+
(Assignee)

Comment 9

5 years ago
Thanks.

https://hg.mozilla.org/integration/mozilla-inbound/rev/0153a4156538
https://hg.mozilla.org/mozilla-central/rev/0153a4156538
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Keywords: intermittent-failure
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.