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.

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

Add IsDisturbed predicate #378

Closed
domenic opened this issue Jul 22, 2015 · 6 comments
Closed

Add IsDisturbed predicate #378

domenic opened this issue Jul 22, 2015 · 6 comments

Comments

@domenic
Copy link
Member

domenic commented Jul 22, 2015

Should return true if stream is ever .cancel()ed or .read() from. See more details in https://etherpad.mozilla.org/streams-f2f-july

@tyoshino
Copy link
Member

Thanks for getting this discussed and summarizing it. FTR, could you please list the participants of the f2f (or put a link to some page with such a list) if no problem?

@wanderview
Copy link
Member

I added the attendees to the etherpad. Basically, me, @domenic, and @annevk since we were all in town for other meetings that week.

@tyoshino
Copy link
Member

Thanks, Ben.

Reading the note. @domenic , you objected to the always-chunked approach at https://lists.w3.org/Archives/Public/public-webapps/2014OctDec/0420.html Has it been resolved?

(This is off topic. I'm creating a new bug)

@domenic
Copy link
Member Author

domenic commented Jul 28, 2015

Yes. We can do chunked for now. In the future I would like to explore allowing setting content-length (and thus not auto-setting chunked) but we can leave that for another time.

@tyoshino
Copy link
Member

Created. Sorry for OT comments.

tyoshino added a commit that referenced this issue Aug 3, 2015
See the suggestion in the OP of #378.
@tyoshino
Copy link
Member

tyoshino commented Aug 3, 2015

Implemented as IsReadableStreamDisturbed() in #385.

tyoshino added a commit that referenced this issue Aug 18, 2015
See the suggestion in the OP of #378.
tyoshino added a commit that referenced this issue Aug 18, 2015
tyoshino added a commit that referenced this issue Aug 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants