1123675 - fetch event respondWith() should report progress back to original document

Status: RESOLVED WORKSFORME

(Core :: DOM: Core & HTML, defect)

Description

[Ben Kelly [:bkelly, not reviewing]]

Talking in #whatwg channel today, the topic of network progress events to the document came up.  In particular, if a SW does this, then we should be reporting network progress for the fetch();

  event.respondWith(fetch(url))

Note, we don't know the fetch() Response is associated with the document until event.respondWith() is called.

At that point, though, we should ideally set the nsIProgressEventSink from the document on the Response's underlying nsIChannel.  We might have to also somehow proxy through the original request so that the document associates the progress correctly.

If the Response has fully consumed the nsIChannel at this point, and the channel is closed, then we cannot report progress.  We could send a 100% progress callback or simply do nothing in this case.

We should also make sure this is set up correctly for Response's returned from event.default().

Not sure we need this before pref'ing on, but setting the dependency there as I don't see a follow-on meta bug.

Comment 1

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

we think this is done.