[ServiceWorker] should update() during top level script evaluation be ignored? (#800)

The wpt tests we imported from blink has this code at the top level of one of its service worker scripts:

>  // update() during the script evaluation should be ignored.
>  self.registration.update();

I don't see anything in the spec that does this.  It seems like it might be a good idea, though, to avoid potential infinite async update loops.  For example, if the install event fails or something I think you could end up with an endless set of failing update jobs running in the background.

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