Closed Bug 1180274 Opened 10 years ago Closed 9 years ago

Fix the semantics of ExtendableEvent::WaitUntil

Categories

(Core :: DOM: Service Workers, defect)

Product:
Core
Component:
DOM: Service Workers
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
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!
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.