Summary: | [Streams API] teeReadableStream should not directly use stream.getReader() | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | youenn fablet <youennf> | ||||||
Component: | WebCore Misc. | Assignee: | youenn fablet <youennf> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | benjamin, calvaris, commit-queue, darin | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
youenn fablet
2015-11-20 02:46:52 PST
Created attachment 265947 [details]
Patch
Comment on attachment 265947 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=265947&action=review > Source/WebCore/ChangeLog:8 > + No change in behavior. This is not true. The comment below says “may be disrupted by user scripts”, so there is an observable behavior change we could and should test. Created attachment 266342 [details]
Patch for landing
(In reply to comment #2) > Comment on attachment 265947 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=265947&action=review > > > Source/WebCore/ChangeLog:8 > > + No change in behavior. > > This is not true. The comment below says “may be disrupted by user scripts”, > so there is an observable behavior change we could and should test. Right, I added a test to cover non regression. Comment on attachment 266342 [details] Patch for landing Clearing flags on attachment: 266342 Committed r192877: <http://trac.webkit.org/changeset/192877> All reviewed patches have been landed. Closing bug. |