Re: [whatwg/fetch] Set body with byte reading support (PR #1593)

Hmm, so currently Fetch has "If stream doesn’t [need more data](https://streams.spec.whatwg.org/#readablestream-need-more-data) ask the user agent to [suspend](https://fetch.spec.whatwg.org/#concept-fetch-suspend) the ongoing fetch.", while Web Serial has:

>1. Let desiredSize be the [desired size to fill up to the high water mark](https://streams.spec.whatwg.org/#readablestream-desired-size-to-fill-up-to-the-high-water-mark) for [this](https://webidl.spec.whatwg.org/#this).[[[readable]]](https://wicg.github.io/serial/#dfn-readable).
>2. If [this](https://webidl.spec.whatwg.org/#this).[[[readable]]](https://wicg.github.io/serial/#dfn-readable)'s [current BYOB request view](https://streams.spec.whatwg.org/#readablestream-current-byob-request-view) is non-null, then set desiredSize to [this](https://webidl.spec.whatwg.org/#this).[[[readable]]](https://wicg.github.io/serial/#dfn-readable)'s [current BYOB request view](https://streams.spec.whatwg.org/#readablestream-current-byob-request-view)'s [byte length](https://webidl.spec.whatwg.org/#buffersource-byte-length).

I guess I should revise the sentence to also check whether the current byob request view is null and suspend the fetch if it is. Am I understanding correct?

And if it's correct, should `need more data` be revised for byte streams?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/1593#issuecomment-1381465420

You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/fetch/pull/1593/c1381465420@github.com>

Received on Friday, 13 January 2023 08:20:16 UTC