Closed Bug 573298 Opened 14 years ago Closed 14 years ago

Intermittent failure in test_websocket.html | shouldn't connect yet in test 10!

Categories

(Core :: DOM: Core & HTML, defect)

x86
Windows Server 2003
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: philor, Unassigned)

References

Details

(Keywords: intermittent-failure)

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1277000921.1277002170.27677.gz WINNT 5.2 mozilla-central debug test mochitests-1/5 on 2010/06/19 19:28:41 s: mw32-ix-slave08 (lovely intermixing preserved): Sec-WebSocket-Origin: http://mochi.test:8888 Sec-WebSocket-Protocol: dropping our write flag and setting other socke29207 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_websocket.html | shouldn't connect yet in test 10! 29208 INFO TEST-PASS | /tests/content/base/test/test_websocket.html | bad readyState in test 11!
This intermittent failure has been already fixed in bug 573227. Test 10 tests if the ws connections are being serialized. So, it presumes that there isn't no websockets connections in the serialization queue. Because of this it must be the first test to be executed. Tests 1 until 5 fail when creating the WebSocket object, so in bug 573227 it has been reordered to be the 6th test.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Because there's many a slip between "the patch I've got up for review right now" and "what I finally landed by the time the bug was closed" the right way to express "I've already got a fix for that, in my current patch in another bug" is to mark this one as depending on that one, and then when you close that one close this one as fixed by that.
Status: RESOLVED → REOPENED
Depends on: 573227
Resolution: FIXED → ---
> is to mark this one as depending on that one, and then when you close that > one close this one as fixed by that. ok, sorry.
I believe this should be now fixed. See the patch in bug 572975.
Status: REOPENED → RESOLVED
Closed: 14 years ago14 years ago
Resolution: --- → FIXED
Whiteboard: [orange]
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.