-
Notifications
You must be signed in to change notification settings - Fork 22.4k
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
ReadableStreamDefaultController.close() - invalid exception reasons #34690
base: main
Are you sure you want to change the base?
ReadableStreamDefaultController.close() - invalid exception reasons #34690
Conversation
@@ -34,19 +29,15 @@ None ({{jsxref("undefined")}}). | |||
### Exceptions | |||
|
|||
- {{jsxref("TypeError")}} | |||
- : Thrown if the source object is not a `ReadableStreamDefaultController`. | |||
|
|||
- : Thrown if `close()` is called when the stream is not readable — because it is already closed, cancelled, or errored — or because it has been requested to close by the underlying source but it has not yet done so because there are still enqueued chunks to read. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the only content change.
Preview URLs External URLs (1)URL:
|
Is this intended to close #34597? |
@Josh-Cena No, it only partially fixes it. The original comment states that there the same problem is found in some nearby issues. I didn't look at those, though I will later if no one else does. |
Ok; because the PR description containing the substring "fixes #xxx", even if prefixed by "partially", would link this PR. |
Thanks @Josh-Cena - I didn't realize it was that sensitive. Have modified to something that hopefully wouldn't trigger a close. |
Partial fix for #34597
Also some layout only fixes.