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

I think it makes more sense to move Install step 22 to Register step 1 rather than Update step 1. This is because if a dev calls unregister(), but then the UA updates as part of the 24 hour cycle, it is unpredictable and non-reproducible across sessions, whereas Register is always triggered by JS invocation. Otherwise, I am happy with the suggestions in the comment above. The invariant then imposes linearity on the "is registration still around or is it cleared?" based on the order of calling unregister() and register().

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