Re: [slightlyoff/ServiceWorker] should ServiceWorkerContainer.getRegistrations() check the uninstalling flag (#943)

I agree to add checking uninstalling flag to `getRegistrations()`. To me, setting the uninstalling flag of a registration means it won't be valid but just continue to control the existing clients until they unload. So, I think `self.registration` should return `undefined` for consistency. WDYT?

One occasion we might have to consider is when the uninstalling registration is being reclaimed by re-`register()` before being cleared up. In this case, `getRegistration()`, `getRegistrations()` and `self.registration` in the same global will start to return the registration again. I think it seems reasonable. Does anyone have any concern about this behavior?

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/slightlyoff/ServiceWorker/issues/943#issuecomment-238491085