Bug 223558 - Streams: new test failure for canceling the branches of an errored tee'd stream
Summary: Streams: new test failure for canceling the branches of an errored tee'd stream
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-03-20 16:10 PDT by Mattias Buelens
Modified: 2021-04-08 11:37 PDT (History)
9 users (show)

See Also:

Attachments
Patch (8.68 KB, patch)
2021-04-07 08:49 PDT, youenn fablet 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mattias Buelens 2021-03-20 16:10:22 PDT 
The WPT test "ReadableStream teeing: erroring a teed stream should properly handle canceled branches" was improved in https://github.com/web-platform-tests/wpt/pull/28153. The test now checks that the cancel promises of the two branches are rejected with the correct reason.

This updated test fails in Safari, see: https://wpt.fyi/results/streams/readable-streams/tee.any.html?diff&filter=ADC&run_id=5700976721788928&run_id=5767793259577344
Chrome, Edge and Firefox still pass the updated test.
Comment 1 Radar WebKit Bug Importer 2021-03-27 16:11:11 PDT 
<rdar://problem/75924807>
Comment 2 youenn fablet 2021-04-07 08:49:31 PDT 
Created attachment 425401 [details]
Patch
Comment 3 EWS Watchlist 2021-04-07 08:50:36 PDT 
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess
Comment 4 EWS 2021-04-08 11:37:40 PDT 
Committed r275667 (236303@main): <https://commits.webkit.org/236303@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 425401 [details].