Skip to content

Add the Priority Hints changes to the fetch spec #1523

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Dec 15, 2022
Merged

Add the Priority Hints changes to the fetch spec #1523

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9b565fc
Integrate Priority Hints
pmeenan Oct 31, 2022
724abaa
final nits modulo tests
annevk Dec 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
45 changes: 35 additions & 10 deletions fetch.bs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1829,8 +1829,12 @@ not always relevant and might require different behavior.
<hr>

<p>A <a for=/>request</a> has an associated
<dfn export for=request id=concept-request-priority>priority</dfn> (null or a
user-agent-defined object). Unless otherwise stated it is null.
<dfn export for=request>priority</dfn>, which is "<code>high</code>", "<code>low</code>", or
"<code>auto</code>". Unless stated otherwise it is "<code>auto</code>".

<p>A <a for=/>request</a> has an associated
<dfn export for=request oldids=concept-request-priority>internal priority</dfn> (null or an
<a>implementation-defined</a> object). Unless otherwise stated it is null.

<p>A <a for=/>request</a> has an associated
<dfn export for=request id=concept-request-origin>origin</dfn>, which is
Expand Down Expand Up @@ -4243,13 +4247,16 @@ the request.
<a for=/>header value</a>) to <var>request</var>'s <a for=request>header list</a>.

<li>
<p>If <var>request</var>'s <a for=request>priority</a> is null, then use <var>request</var>'s
<a for=request>initiator</a>, <a for=request>destination</a>, and
<a for=request>render-blocking</a> appropriately in setting <var>request</var>'s
<a for=request>priority</a> to a user-agent-defined object.
<p>If <var>request</var>'s <a for=request>internal priority</a> is null, then use
<var>request</var>'s <a for=request>priority</a>, <a for=request>initiator</a>,
<a for=request>destination</a>, and <a for=request>render-blocking</a> in an
<a>implementation-defined</a> manner to set <var>request</var>'s
<a for=request>internal priority</a> to an <a>implementation-defined</a> object.

<p class=note>The user-agent-defined object could encompass stream weight and dependency for
HTTP/2, and equivalent information used to prioritize dispatch and processing of HTTP/1 fetches.
<p class=note>The <a>implementation-defined</a> object could encompass stream weight and
dependency for HTTP/2, priorities used in <cite>Extensible Prioritization Scheme for HTTP</cite>
for transports where it applies (including HTTP/3), and equivalent information used to prioritize
dispatch and processing of HTTP/1 fetches. [[!RFC9218]]

<li>
<p>If <var>request</var> is a <a>subresource request</a>, then:
Expand Down Expand Up @@ -7009,6 +7016,7 @@ dictionary RequestInit {
boolean keepalive;
AbortSignal? signal;
RequestDuplex duplex;
RequestPriority priority;
any window; // can only be set to null
};

Expand All @@ -7018,6 +7026,7 @@ enum RequestCredentials { "omit", "same-origin", "include" };
enum RequestCache { "default", "no-store", "reload", "no-cache", "force-cache", "only-if-cached" };
enum RequestRedirect { "follow", "error", "manual" };
enum RequestDuplex { "half" };
enum RequestPriority { "high", "low", "auto" };
</pre>

<p class="note no-backref">"<code>serviceworker</code>" is omitted from
Expand Down Expand Up @@ -7115,6 +7124,9 @@ object), initially null.
set when {{RequestInit/body}} is a {{ReadableStream}} object. <span class=note>See
<a href="https://github.com/whatwg/fetch/issues/1254">issue #1254</a> for defining
"<code>full</code>".</span>

<dt>{{RequestInit/priority}}
<dd>A string to set <var>request</var>'s <a for=request>priority</a>.
</dl>

<dt><code><var>request</var> . <a attribute for=Request>method</a></code>
Expand Down Expand Up @@ -7299,8 +7311,8 @@ constructor steps are:
<dt><a for=request>window</a>
<dd><var>window</var>.

<dt><a for=request>priority</a>
<dd><var>request</var>'s <a for=request>priority</a>.
<dt><a for=request>internal priority</a>
<dd><var>request</var>'s <a for=request>internal priority</a>.

<dt><a for=request>origin</a>
<dd><var>request</var>'s <a for=request>origin</a>. <span class=note>The propagation of the
Expand Down Expand Up @@ -7463,6 +7475,18 @@ constructor steps are:
<li><p>If <var>init</var>["{{RequestInit/signal}}"] <a for=map>exists</a>, then set
<var>signal</var> to it.

<li>
<p>If <var>init</var>["{{RequestInit/priority}}"] <a for=map>exists</a>, then:

<ol>
<li><p>If <var>request</var>'s <a for=request>internal priority</a> is not null, then update
<var>request</var>'s <a for=request>internal priority</a> in an <a>implementation-defined</a>
manner.

<li><p>Otherwise, set <var>request</var>'s <a for=request>priority</a> to
<var>init</var>["{{RequestInit/priority}}"].
</ol>

<li><p>Set <a>this</a>'s <a for=Request>request</a> to <var>request</var>.

<li><p>Set <a>this</a>'s <a for=Request>signal</a> to a <a for=/>new</a> {{AbortSignal}} object
Expand Down Expand Up @@ -8545,6 +8569,7 @@ Odin Hørthe Omdal,
Olli Pettay,
Ondřej Žára,
O. Opsec,
Patrick Meenan,
Perry Jiang<!-- perryjiang; Github -->,
Philip Jägenstedt,
R. Auburn,
Expand Down