Closed Bug 1150444 Opened 9 years ago Closed 9 years ago

Intermittent test_WorkerDebugger.isFrozen.xul | Promise 1 out of 2 should be resolved. - got 2, expected 1

Categories

(Core :: DOM: Workers, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox39 --- unaffected
firefox40 --- fixed
firefox41 --- fixed
firefox-esr31 --- unaffected
firefox-esr38 --- unaffected

People

(Reporter: cbook, Assigned: ejpbruel)

References

()

Details

(Keywords: crash, intermittent-failure)

Attachments

(1 file)

Rev4 MacOSX Snow Leopard 10.6 mozilla-inbound opt test mochitest-other

https://treeherder.mozilla.org/logviewer.html#?job_id=8396343&repo=mozilla-inbound

02:35:26 INFO - 1048 INFO TEST-UNEXPECTED-FAIL | dom/workers/test/test_WorkerDebugger.isFrozen.xul | Promise 1 out of 2 should be resolved. - got 2, expected 1
Eddy, can you look into this?
Flags: needinfo?(ejpbruel)
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #108)
> Eddy, can you look into this?

I will look into this this week.
Assignee: nobody → ejpbruel
Flags: needinfo?(ejpbruel)
Well, this turned out to be a simple fix for a change :-)

This test checks that when navigating forward from a page with a worker to a page with another worker in the bfcache, the worker in the former page is frozen as it is moved to the bfcache, while the worker in the latter is thawed as it is moved out of the bfcache.

However, the test also checks that the first worker is frozen before the second worker is thawed. Apparently, that isn't always the case, which explains why this test fails intermittently.

Luckily, we don't actually depend on a particular order here. All we need is the guarantee that the first worker will eventually be frozen, and the second worker will eventually be thawed. I've updated the test accordingly.
Attachment #8601981 - Flags: review?(khuey)
https://hg.mozilla.org/mozilla-central/rev/7a5258332448
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
Not fixed :/
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Blocks: 1214248
Now that bug 1178726 has been fixed this test no longer exists.
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.