Re: [whatwg/fetch] Add colorprofile destination (#1331)

@annevk commented on this pull request.

Modulo nit this looks good. Though as a follow-up the order of destinations in the table needs adjusting. It used to be lexicographical within a given initiator, but that got lost after some additions.

> @@ -1840,9 +1845,9 @@ otherwise, it is unset.
 
 <p>A <dfn export>subresource request</dfn> is a <a for=/>request</a>
 whose <a for=request>destination</a> is "<code>audio</code>", "<code>audioworklet</code>",
-"<code>font</code>", "<code>image</code>", "<code>manifest</code>", "<code>paintworklet</code>",
-"<code>script</code>", "<code>style</code>", "<code>track</code>", "<code>video</code>",
-"<code>xslt</code>", or the empty string.
+"<code>colorprofile</code>" "<code>font</code>", "<code>image</code>", "<code>manifest</code>",

Lacks a comma.

-- 
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/1331#pullrequestreview-777489858

Received on Tuesday, 12 October 2021 14:50:14 UTC