Re: [whatwg/fetch] Revisit `destination` for `<iframe>`, `<frame>`, `<embed>`, and `<object>`. (#948)

jungkees commented on this pull request.



>  
 <p>A <dfn export>navigation request</dfn> is a <a for=/>request</a> whose
 <a for=request>destination</a> is
-"<code>document</code>".
+"<code>document</code>", "<code>embed</code>", "<code>frame</code>", "<code>iframe</code>",
+or "<code>object</code>".

1. Yes `object` and `embed` return early in [Handle Fetch](https://w3c.github.io/ServiceWorker/#handle-fetch) step 11.
1. I think all the three cases are OK: 1) is force-reload as @mikewest said 2) is checking if the request is for main resource for navigation preload 3) is to fill up fetchEvent.replacesClientId for navigation requests. Fetch requests went through [process a navigate fetch](https://html.spec.whatwg.org/#process-a-navigate-fetch) should have a replaces client id at this point.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/948#discussion_r378600910