Closed Bug 1134224 Opened 5 years ago Closed 5 years ago

Intermittent test_bug1132395.html | Test timed out. - expected PASS

Categories

(Core :: DOM: Workers, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox38 --- wontfix
firefox38.0.5 --- wontfix
firefox39 --- wontfix
firefox40 --- fixed
firefox41 --- fixed
firefox-esr31 --- unaffected
firefox-esr38 --- fixed

People

(Reporter: cbook, Assigned: baku)

References

(Blocks 1 open bug, )

Details

(Keywords: intermittent-failure)

Attachments

(2 files, 1 obsolete file)

Windows 8 64-bit b2g-inbound pgo test mochitest-4

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

00:36:40 INFO - 404 INFO TEST-UNEXPECTED-FAIL | dom/workers/test/test_bug1132395.html | Test timed out. - expected PASS
Blocks: 1132395
Flags: needinfo?(amarchesini)
Attached patch timeout.patch (obsolete) — Splinter Review
Flags: needinfo?(amarchesini)
Attachment #8589687 - Flags: review?(ehsan)
Assignee: nobody → amarchesini
OS: Windows 8 → All
Hardware: x86 → All
Comment on attachment 8589687 [details] [diff] [review]
timeout.patch

Review of attachment 8589687 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/workers/test/test_bug1132395.html
@@ +20,4 @@
>      SimpleTest.finish();
>    }
> +
> +  sw.port.postMessage('go');

Hmm, don't you also need to call sw.port.start() on both sides?  r=me if this works.
Attachment #8589687 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/a70e27fa70b1
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
(In reply to Treeherder Robot from comment #47)

:(
Flags: needinfo?(amarchesini)
Target Milestone: mozilla40 → ---
https://treeherder.mozilla.org/#/jobs?repo=try&revision=aed30cb2c93b

debugging it...
Flags: needinfo?(amarchesini)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Any updates here on this frequent failure? Otherwise, I think we're at a point where this test may need skipping until someone has more time to devote to it.
I'm not working on this. I'm happy if somebody else wants to take it.
Assignee: amarchesini → nobody
Flags: needinfo?(amarchesini)
Andrew, do you have anybody else who can take a look at this long-running unowned orange before I disable the test?
Flags: needinfo?(overholt)
Attached patch test.patchSplinter Review
This is not a fix. This is just for debugging this issue next time it happens.
Attachment #8589687 - Attachment is obsolete: true
Attachment #8614663 - Flags: review?(ehsan)
baku told me he'd investigate more once his logging patch gets in.  Assigning to him to ensure it's not set to nobody.
Assignee: nobody → amarchesini
Flags: needinfo?(overholt)
Comment on attachment 8614663 [details] [diff] [review]
test.patch

Review of attachment 8614663 [details] [diff] [review]:
-----------------------------------------------------------------

(Logging patches for tests don't really need review...)
Attachment #8614663 - Flags: review?(ehsan) → review+
Keywords: leave-open
Any news on this frequent ongoing failure? Otherwise, can we please just skip the test for now until there's time to investigate further?
Flags: needinfo?(amarchesini)
Blocks: 1176225
Attached patch disable2.patchSplinter Review
Flags: needinfo?(amarchesini)
Attachment #8624710 - Flags: review?(ryanvm)
Attachment #8624710 - Flags: review?(ryanvm) → review+