- From: Marijn Kruisselbrink <notifications@github.com>
- Date: Tue, 12 Mar 2019 15:40:28 -0700
- To: w3c/FileAPI <FileAPI@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <w3c/FileAPI/pull/117/review/213685408@github.com>
mkruisselbrink commented on this pull request.
> +It must return the result of [=getting a stream for a Blob=] for the [=context object=].
+
+To <dfn lt="get a stream for a Blob|getting a stream for a Blob">get a stream for a {{Blob}}</dfn>
+|blob|, run the following steps:
+
+1. Let |stream| be the result of [=construct a ReadableStream object|constructing=] a
+ {{ReadableStream}} object.
+1. Return |stream| and run the following steps [=in parallel=]:
+ 1. While not all bytes of |blob| have been read:
+ 1. Let |bytes| be the byte sequence that results from reading a [=chunk=] from |blob|.
+ 1. If a [=file read error=] occured while reading |bytes|, [$ReadableStream/error$]
+ |stream| with a [=failure reason=] and abort these steps.
+ 1. [=ReadableStream/Enqueue=] |bytes| into |stream|.
+
+ Issue: We need to specify more concretely what reading from a Blob actually does, and what
+ possible errors can happen.
Good point, yeah. Not sure what we can helpfully say about chunk sizes though... What does fetch do for example? Especially when a response is read from the http cache it seems like that is sort of a similar situation as this one? Does it specify anything about how the body is streamed?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/FileAPI/pull/117#discussion_r264912742
Received on Tuesday, 12 March 2019 22:40:53 UTC