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

@noamr commented on this pull request.



> @@ -225,8 +225,15 @@ 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>"

Sure, I can add a note that these phases are only relevant for half-duplex fetches.
It's not just about Deno, they're also incorrect for WebSockets.

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

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