"Harness status: Timeout" when running "extendable-event-waituntil.https.html" test

RESOLVED FIXED in Firefox 43

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: noemi, Assigned: nsm)

Tracking

({dev-doc-complete, site-compat})

Trunk
mozilla43
dev-doc-complete, site-compat
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox42 affected, firefox43 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Test run such as |./mach web-platform-tests _mozilla/service-workers/service-worker/extendable-event-waituntil.https.html|

Actual Result:
Harness status: Timeout
Found 6 tests:
* 4 Pass
* 2 Timeout:	
** Test ExtendableEvent waitUntil reject precedence -> timed out
** Test activate event waitUntil rejected -> timed out
Traces: https://pastebin.mozilla.org/8838590
Bug 1189644 - Update waitUntil() and activation to spec. r?catalinb

waitUntil() has been updated to accept multiple calls and concatenate them into Promise.all().
activation does not fail any more even if the promise(s) passed to waitUntil() reject
Attachment #8651289 - Flags: review?(catalin.badea392)
Assignee: nobody → nsm.nikhil
Status: NEW → ASSIGNED
Comment on attachment 8651289 [details]
MozReview Request: Bug 1189644 - Update waitUntil() and activation to spec. r?catalinb

https://reviewboard.mozilla.org/r/16895/#review15131

Ship It!
Attachment #8651289 - Flags: review?(catalin.badea392) → review+
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/471a45b97ce2264a08a933bdddcf8c71f65b001a
changeset:  471a45b97ce2264a08a933bdddcf8c71f65b001a
user:       Nikhil Marathe <nsm.nikhil@gmail.com>
date:       Fri Aug 21 16:13:54 2015 -0700
description:
Bug 1189644 - Update waitUntil() and activation to spec. r=catalinb

waitUntil() has been updated to accept multiple calls and concatenate them into Promise.all().
activation does not fail any more even if the promise(s) passed to waitUntil() reject.

Update web-platform-tests expected data
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7babb5b9d74adb274384b0f8897ae72cfea7275c
changeset:  7babb5b9d74adb274384b0f8897ae72cfea7275c
user:       Nikhil Marathe <nsm.nikhil@gmail.com>
date:       Mon Aug 24 13:54:25 2015 -0700
description:
Bug 1189644 - Follow up to fix test. a=bustage

CLOSED TREE
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/158b57669166ad5caf9ff96d8c449ddde669585b
changeset:  158b57669166ad5caf9ff96d8c449ddde669585b
user:       Nikhil Marathe <nsm.nikhil@gmail.com>
date:       Mon Aug 24 14:42:14 2015 -0700
description:
Bug 1189644 - Update waitUntil() and activation to spec. r=catalinb

waitUntil() has been updated to accept multiple calls and concatenate them into Promise.all().
activation does not fail any more even if the promise(s) passed to waitUntil() reject.

Update web-platform-tests expected data
https://hg.mozilla.org/mozilla-central/rev/158b57669166
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-firefox43: --- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
(Reporter)

Comment 7

4 years ago
Hi,

just checked on m-c (c46370eea81a revision) and the tests successfully run. Thanks for fixing it!.

Summary

Harness status: OK

Found 6 tests
6 Pass
Details
Result	Test Name
Pass	Test install event waitUntil fulfilled	
Pass	Test ExtendableEvent multiple waitUntil fulfilled.	
Pass	Test ExtendableEvent waitUntil reject precedence.	
Pass	Test activate event waitUntil fulfilled	
Pass	Test install event waitUntil rejected	
Pass	Test activate event waitUntil rejected.

Updated

4 years ago
Duplicate of this bug: 1180274
You need to log in before you can comment on or make changes to this bug.