Closed
Bug 1230042
Opened 9 years ago
Closed 9 years ago
Intermittent TEST-UNEXPECTED-TIMEOUT | /_mozilla/service-workers/service-worker/clients-matchall-include-uncontrolled.https.html | Verify matchAll() respect includeUncontrolled - Test timed out
Categories
(Testing :: web-platform-tests, defect)
Testing
web-platform-tests
Tracking
(firefox45 fixed, firefox46 fixed)
RESOLVED
FIXED
mozilla46
People
(Reporter: philor, Assigned: ehsan.akhgari)
References
Details
(Keywords: intermittent-failure)
Attachments
(2 files)
774 bytes,
patch
|
bkelly
:
review+
|
Details | Diff | Splinter Review |
2.48 KB,
patch
|
bkelly
:
review+
|
Details | Diff | Splinter Review |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 3•9 years ago
|
||
bkelly: this seems to be a relatively common orange. Might just need a long timeout or something, but you seem like the right person to look at it.
Comment 4•9 years ago
|
||
Error message: TEST-UNEXPECTED-ERROR | /_mozilla/service-workers/service-worker/clients-matchall-include-uncontrolled.https.html | Error: assert_equals: expected 4 but got 5 I wonder if there is an extra window from a previous tests that is being picked up. It could be racing with GC to remove the window or something.
Comment hidden (Intermittent Failures Robot) |
Comment 6•9 years ago
|
||
Sucks that this mainly triggers on osx 10.10 which doesn't seem to run in try builds.
Reporter | ||
Comment 7•9 years ago
|
||
It does, you just have to beg for it. try: -b o -p macosx64 -u web-platform-tests-4[10.10] -t none (or everything from checking every checkbox in the "Restrict tests to platform(s)" section of trychooser if you want all platforms including 10.10)
Reporter | ||
Comment 8•9 years ago
|
||
Or in theory you could add them to an existing try push that has a completed Mac build, with either the treeherder UI from the downarrow menu at the top right of the push, "Add new Jobs", or with http://try-extender.herokuapp.com/, though neither seems to actually work, or with https://mozilla-ci-tools.readthedocs.org/en/latest/scripts.html which will work but requires a fair amount of effort.
Comment 9•9 years ago
|
||
Ehsan asked me to disable this test so he can investigate it next week.
Assignee: bkelly → ehsan
Updated•9 years ago
|
Keywords: leave-open
Reporter | ||
Comment 12•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/127e0653fefb
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 14•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9380b7a2b375
Attachment #8698447 -
Flags: review?(bkelly)
Updated•9 years ago
|
Attachment #8698447 -
Flags: review?(bkelly) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: leave-open
Comment 16•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6d57f8b28334
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Comment 17•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/d044c3abe019
You need to log in
before you can comment on or make changes to this bug.
Description
•