Re: [ServiceWorker] Possible run-to-completion violation in "Handle Service Worker Client Unload" (#746)

Good catch! Thanks for spotting this.

In this case, the registration should not be cleared as it's already under an update with its installing worker. To resolve this, I added a condition (ef4bff338f5808c0965d3d55c8800e35a79ddb90) checking whether a registration has a non-null newest worker before clearing the registration: [Handle Service Worker Client Unload](https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#on-client-unload-algorithm) step 5. Indeed, this newest worker check was added to all the other part of the steps that try to clear a registration. (I think I missed it for this algorithm.)

---
Reply to this email directly or view it on GitHub:
https://github.com/slightlyoff/ServiceWorker/issues/746#issuecomment-138790424