Re: [whatwg/fetch] Move `finalize and report timing` to controller (PR #1413)

@noamr commented on this pull request.



> @@ -232,8 +232,21 @@ lt="authentication entry">authentication entries</a> (for HTTP authentication).
 <a for=struct>items</a>:
 
 <dl>
- <dt><dfn for="fetch controller">state</dfn> (default "<code>ongoing</code>")
- <dd>"<code>ongoing</code>", "<code>terminated</code>", or "<code>aborted</code>"
+ <dt><dfn for="fetch controller">state</dfn> (default "<code>requesting</code>")
+ <dd>
+  "<code>requesting</code>", "<code>responding</code>", "<code>concluded</code>",
+  "<code>terminated</code>", or "<code>aborted</code>"
+
+  <p class="note">The <a for="fetch controller" lt="state">states</a> of the <a>fetch controller</a>
+  only make sense in the context of half-duplex HTTP fetches (where the <a for=/>request</a> is
+  complete before the <a for=/>response</a> starts). Handle with care when using them in other
+  contexts.</p>
+
+ <dt><dfn for="fetch controller">timing info</dfn>
+ <dd>A <a for=/>fetch timing info</a>

Usually those `<dl>` sections don't have a period

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

Message ID: <whatwg/fetch/pull/1413/review/975010146@github.com>

Received on Tuesday, 17 May 2022 08:00:22 UTC