1459781 - service worker fetch event not happen when html generated from document.write

Status: NEW

(Core :: DOM: Service Workers, defect, P3)

Description

[James Yang]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.181 Safari/537.36

Steps to reproduce:

Using the code below:
https://gist.github.com/futurist/697b9d7153042dba4e53e9404509cf38

Online demo:
https://service-worker-fail-urzkvexbkh.now.sh/




Actual results:

The console log did not show the fetch event for `b.js`, since the url is generated by `document.write`



Expected results:

The console log should show the fetch event for `b.js`.

Comment 1

[James Yang]

the tested firefox UA is "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:59.0) Gecko/20100101 Firefox/59.0"

Comment 2

[Ben Kelly [:bkelly, not reviewing]]

Currently firefox implements the part of the html spec that creates a new window global on document.open().  There is nothing in the spec AFAICT, though, that causes the service worker controller from the original window to be inherited by the new window.  

This would be fixed by aligning the spec and firefox to other browsers in bug 1456313.

Comment 3

[James Yang]

I see, may I ask how to get around with this problem when have to use `document.open`? Any solution for current version of FF?

Comment 4

[Ben Kelly [:bkelly, not reviewing]]

NI myself to file a spec issue to determine what the correct service worker behavior should be here.  If the group agrees inheriting the service worker is desired we can possibly do a work around in gecko code to make it work that way.

Until then, though, I think the only work around would be to have your service worker provide the final html content directly via FetchEvent handling instead of using document.write().

Comment 5

[Ben Kelly [:bkelly, not reviewing]]

I wrote a demo of the problem here:

https://sw-document-open.glitch.me/

Chrome and safari inherit the controller, but edge matches firefox behavior.

I wrote a spec issue to discuss what the desired behavior should be:

https://github.com/w3c/ServiceWorker/issues/1313