Closed Bug 1180274 Opened 9 years ago Closed 9 years ago

Fix the semantics of ExtendableEvent::WaitUntil

Categories

(Core :: DOM: Service Workers, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1189644

People

(Reporter: ehsan.akhgari, Unassigned)

References

Details

(Keywords: dev-doc-complete)

The spec <http://slightlyoff.github.io/ServiceWorker/spec/service_worker/#extendable-event-waituntil-method> says that we should throw InvalidStateError if waitUntil is called outside of the event handler, and that multiple calls to it should append promises to the list of extend lifetime promises.

Our current implementation still dishonors second and later callers and doesn't implement the error checking mandated by the spec.
Keywords: dev-doc-needed
Blocks: ServiceWorkers-postv1
No longer blocks: ServiceWorkers-v1
I believe that this is currently fixed by bug 1181056 and bug 1189644. Is this correct ehsan?
Flags: needinfo?(ehsan)
Yes, you're right!
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(ehsan)
Resolution: --- → DUPLICATE
I've added some information about this behaviour and the update to Firefox at

https://developer.mozilla.org/en-US/docs/Web/API/ExtendableEvent
https://developer.mozilla.org/en-US/docs/Web/API/ExtendableEvent/waitUntil$edit

I've also added an entry to the relevant release notes:

https://developer.mozilla.org/en-US/Firefox/Releases/43#Service_Workers

Let me now if this reads ok. Thanks!
You need to log in before you can comment on or make changes to this bug.