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

Status: RESOLVED FIXED

(Testing :: web-platform-tests, defect)

Description

[Noemí Freire (:noemi)]

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

Comment 1

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Attachment: MozReview Request: Bug 1189644 - Update waitUntil() and activation to spec. r?catalinb

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

Comment 2

[Cătălin Badea (:catalinb)]

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!

Comment 3

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/471a45b97ce2264a08a933bdddcf8c71f65b001a
changeset:  471a45b97ce2264a08a933bdddcf8c71f65b001a
user:       Nikhil Marathe <[e-mail]>
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

Comment 4

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/7babb5b9d74adb274384b0f8897ae72cfea7275c
changeset:  7babb5b9d74adb274384b0f8897ae72cfea7275c
user:       Nikhil Marathe <[e-mail]>
date:       Mon Aug 24 13:54:25 2015 -0700
description:
Bug 1189644 - Follow up to fix test. a=bustage

CLOSED TREE

Comment 5

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/158b57669166ad5caf9ff96d8c449ddde669585b
changeset:  158b57669166ad5caf9ff96d8c449ddde669585b
user:       Nikhil Marathe <[e-mail]>
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

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/158b57669166

Comment 7

[Noemí Freire (:noemi)]

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.

Comment 9

[Kohei Yoshino [:kohei]]

MDN docs have been updated by chrisdavidmills:
https://developer.mozilla.org/en-US/Firefox/Releases/43#Service_Workers
https://developer.mozilla.org/en-US/docs/Web/API/ExtendableEvent/waitUntil

I have posted the site compatibility doc:
https://www.fxsitecompat.com/en-US/docs/2015/service-workers-extendableevent-waituntil-implementation-has-been-updated/