Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make writer.write() reject with the stream's stored error #712

Merged
merged 3 commits into from Mar 28, 2017
Merged

Make writer.write() reject with the stream's stored error #712

merged 3 commits into from Mar 28, 2017

Conversation

domenic
Copy link
Member

@domenic domenic commented Mar 28, 2017

Closes #700.

Tests: web-platform-tests/wpt#5240

(reminder to self: remember to do the multi-step test update dance before pressing the merge button)

@domenic domenic requested review from tyoshino and ricea March 28, 2017 08:59
@ricea
Copy link
Collaborator

ricea commented Mar 28, 2017

lgtm

@domenic domenic merged commit 990973c into master Mar 28, 2017
@domenic domenic deleted the use-storederror-for-write branch March 28, 2017 10:51
@tyoshino
Copy link
Member

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tyoshino tyoshino Awaiting requested review from tyoshino

@ricea ricea Awaiting requested review from ricea

Assignees
No one assigned
Labels
writable streams
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@domenic @ricea @tyoshino