Summary: | [Fetch API] Add support for consuming a Request ReadableStream body | ||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | youenn fablet <youennf> | ||||||||||||||||||
Component: | WebCore Misc. | Assignee: | youenn fablet <youennf> | ||||||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||||||
Severity: | Normal | CC: | achristensen, cdumez, commit-queue, sam, webkit-bug-importer | ||||||||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||||||||
Hardware: | Unspecified | ||||||||||||||||||||
OS: | Unspecified | ||||||||||||||||||||
Attachments: |
|
Description
youenn fablet
2017-08-31 13:57:30 PDT
Created attachment 319519 [details]
Patch
Created attachment 319572 [details]
Patch
Created attachment 319574 [details]
Patch
Created attachment 319575 [details]
Patch
Created attachment 319597 [details]
Patch
Created attachment 319611 [details]
Patch
Created attachment 319625 [details]
Patch
Comment on attachment 319625 [details]
Patch
Let's add a test with an ArrayBuffer of size 0.
We should also add a test for sending a request to a server that sends a 401 response then we need to send the request again. Created attachment 319653 [details]
Patch for landing
Thanks for the review. (In reply to Alex Christensen from comment #8) > Comment on attachment 319625 [details] > Patch > > Let's add a test with an ArrayBuffer of size 0. Done (In reply to Alex Christensen from comment #9) > We should also add a test for sending a request to a server that sends a 401 > response then we need to send the request again. Currently, uploading a stream with a request is still forbidden. A follow-up patch will allow this in which case we might want to add these kinds of test, especially when upload will be done in full streaming mode. Comment on attachment 319653 [details] Patch for landing Clearing flags on attachment: 319653 Committed r221504: <http://trac.webkit.org/changeset/221504> All reviewed patches have been landed. Closing bug. |