Re: [ServiceWorker] Restrict openWindow() to http(s) schemes? (#699)

Apologies for flip-flopping, but I'm no longer convinced the changes in #696 were a good idea.

Sure, `clients.openWindow('about:blank')` creates a window that cannot be used by the SW, and is arguably useless, but as @annevk points out, a developer would open a page on their origin that navigates to `about:blank` with JS, or a redirect. Or, they could do `clients.openWindow('/whatever').then(c => c.navigate('about:blank'))` once #681 lands.

Making `openWindow` accept a different set of URLs to `window.open` seems needlessly complicated.

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