Re: [ServiceWorker] Soft Update and Register can race when setting registration scriptURL (#789)

To clarify, I propose:

1. Update algorithm take an optional script URL as an argument
2. In Update, after the queue synchronization:
  1. If script URL was not provided, then set script URL to newestWorker.scriptURL
  2. Use script URL instead of registration's registering script URL

Then Register algorithm passes the content provided script URL to the Update algorithm.  The update() and Soft Update algorithms do not pass a script URL to the Update algorithm, and therefore get the default "newest worker" script URL.

And in the Register algorithm, change step 4.2.3 from

> If newestWorker is not null, scriptURL equals newestWorker's script url, and scriptURL equals registration's registering script url, then: 

To just:

> If newestWorker is not null, and scriptURL equals newestWorker's script url, then:

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

Received on Monday, 23 November 2015 14:47:36 UTC