Re: [whatwg/fetch] Integration with preload (PR #1342)

Signed and attached. Looks good!

On Mon, Nov 8, 2021 at 12:07 PM Yoav Weiss ***@***.***> wrote:

> ***@***.**** commented on this pull request.
> ------------------------------
>
> In fetch.bs
> <https://github.com/whatwg/fetch/pull/1342#discussion_r744552887>:
>
> > +   <a>HTTP(S) scheme</a>
> +
> +   <li><p><var>request</var>'s <a for=request>mode</a> is "<code>same-origin</code>",
> +   "<code>cors</code>", or "<code>no-cors</code>"
> +
> +   <li><p><var>hasCustomHeaders</var> is false
> +
> +   <li><p><var>request</var>'s <a for=request>window</a> is not null
> +
> +   <li><p><var>request</var>'s <a for=request>method</a> is `<code>GET</code>`
> +  </ul>
> +
> +  <p>then perform the following steps:
> +
> +  <ol>
> +   <li><p>Let <var>onPreloadedResponseAvailable</var> be to run the following step
>
> This sentence is confusing to me. Can you clarify it?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/whatwg/fetch/pull/1342#pullrequestreview-799886548>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAAVT5XXEYS3UGGMQY3HJL3UK6OMJANCNFSM5GXJG7RQ>
> .
>


-- 
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/1342#issuecomment-963009084

Received on Monday, 8 November 2021 10:23:41 UTC