New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 608259 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Leaves the project on 2018/03/02
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Feature

Blocking:
issue 503491



Sign in to add a comment

[ReadableStream] Reflect spec changes: RS/RBS merge, getReader() redesign

Project Member Reported by tyoshino@chromium.org, May 2 2016

Issue description

The following issues have recently closed.
https://github.com/whatwg/streams/issues/379
https://github.com/whatwg/streams/issues/294

We're reflecting them to our ReadableStream implementation.
https://codereview.chromium.org/1902673003/

 
Blocking: 503491
Cc: domenic@chromium.org

Comment 3 by n...@fb.com, May 2 2016

Labels: DevRel-Facebook
Project Member

Comment 4 by bugdroid1@chromium.org, May 6 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b2496216b74c6d5872871c933a046340fc1fe7cf

commit b2496216b74c6d5872871c933a046340fc1fe7cf
Author: tyoshino <tyoshino@chromium.org>
Date: Fri May 06 15:19:18 2016

Reflect recent spec changes to V8 Extra ReadableStream impl

Readable byte stream is not yet implemented in this CL.

BUG= 503491 , 608259 
R=domenic,yhirano

Review-Url: https://codereview.chromium.org/1902673003
Cr-Commit-Position: refs/heads/master@{#392058}

[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/LayoutTests/http/tests/streams/readable-streams/bad-strategies.js
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/LayoutTests/http/tests/streams/readable-streams/bad-underlying-sources.js
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/LayoutTests/http/tests/streams/readable-streams/cancel.js
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/LayoutTests/http/tests/streams/readable-streams/general.js
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/LayoutTests/http/tests/streams/resources/rs-test-templates.js
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/LayoutTests/http/tests/streams/resources/test-utils.js
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/Source/core/streams/ReadableStream.js
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/Source/core/streams/ReadableStreamController.h
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/Source/core/streams/ReadableStreamOperations.cpp
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/Source/core/streams/ReadableStreamOperations.h
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/Source/core/streams/ReadableStreamOperationsTest.cpp
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/Source/core/streams/UnderlyingSourceBase.cpp
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/Source/modules/fetch/BodyStreamBuffer.cpp
[modify] https://crrev.com/b2496216b74c6d5872871c933a046340fc1fe7cf/third_party/WebKit/Source/modules/fetch/ReadableStreamDataConsumerHandle.cpp

Status: Fixed (was: Started)
Summary: [ReadableStream] Reflect spec changes: RS/RBS merge, getReader() redesign (was: [ReadableStream] Reflect spec changes: RS/RBS merge, getReader() redeisng)

Sign in to add a comment