`ServiceWorkerRegistration.update()` should return `Promise<ServiceWorkerRegistration>`

[jespertheend]

Currently it is defined in lib.dom.ts to return Promise<void>.

See the MDN page for update() which mentions:

Return value
A Promise that resolves with a ServiceWorkerRegistration object.

The spec is a bit unclear because it mentions some job logic:

6. Let job be the result of running Create Job with update

but in the end it seems like this promise is always resolved with a ServiceWorkerRegistration.

[skyclouds2001]

but the spec idl says this method return void, though in browser implement is seems to return with a ServiceWorkerRegistration

[jespertheend]

Hmm interesting, seems like an issue was submitted about this 5 years ago but it has never been addressed.

[Bashamega]

Hello @saschanaz
Can I work on this issue?

[saschanaz]

No, this method returning an object is Chrome-only and other browser engines return undefined as the spec IDL says.

If we want to change this then we should change the spec. Also commented on the spec issue.

[jespertheend]

The spec has now been updated with a fix.

Seems like this is no longer an issue in TypeScript anymore either since 5.9.2.