Closed Bug 1723762 Opened 5 years ago Closed 5 years ago

show_bug.cgi?id=227690

Tracking

()

Status:

RESOLVED FIXED

Milestone:

92 Branch

Tracking Flags:

Tracking

Status

firefox92

---

fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)

Assignee

Description

•

5 years ago

Sync web-platform-tests PR 29880 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/29880
Details from upstream follow.

b'Youenn Fablet <youennf@gmail.com>' wrote:

WebKit export of https://bugs.webkit.org/show_bug.cgi?id=227690

WebKit export from bug: ReadableStream's pipeTo() and pipeThrough() don't handle options in spec-perfect way

Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)

Assignee

Updated

•

5 years ago

Component: web-platform-tests → JavaScript Engine

Product: Testing → Core

Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)

Assignee

Comment 1

•

5 years ago

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=0f3fa450d3574ceaf68a091ea6937b564330bdac

Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)

Assignee

Comment 2

•

5 years ago

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 5 tests

Status Summary

Firefox

PASS : 21
FAIL : 33
ERROR: 4

Chrome

OK : 4
PASS : 172

Safari

PASS : 120
FAIL : 9
ERROR: 4

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

/streams/piping/pipe-through.any.html
Piping through a duck-typed pass-through transform stream should work: FAIL
Piping through a transform errored on the writable end does not cause an unhandled promise rejection: FAIL
pipeThrough should not call pipeTo on this: FAIL
pipeThrough should not call pipeTo on the ReadableStream prototype: FAIL
pipeThrough should brand-check this and not allow 'null': FAIL
pipeThrough should brand-check this and not allow 'undefined': FAIL
pipeThrough should brand-check this and not allow '0': FAIL
pipeThrough should brand-check this and not allow 'NaN': FAIL
pipeThrough should brand-check this and not allow 'true': FAIL
pipeThrough should brand-check this and not allow 'ReadableStream': FAIL
pipeThrough should brand-check this and not allow '[object ReadableStream]': FAIL
/streams/piping/pipe-through.any.worker.html
Piping through a duck-typed pass-through transform stream should work: FAIL
Piping through a transform errored on the writable end does not cause an unhandled promise rejection: FAIL
pipeThrough should not call pipeTo on this: FAIL
pipeThrough should not call pipeTo on the ReadableStream prototype: FAIL
pipeThrough should brand-check this and not allow 'null': FAIL
pipeThrough should brand-check this and not allow 'undefined': FAIL
pipeThrough should brand-check this and not allow '0': FAIL
pipeThrough should brand-check this and not allow 'NaN': FAIL
pipeThrough should brand-check this and not allow 'true': FAIL
pipeThrough should brand-check this and not allow 'ReadableStream': FAIL
pipeThrough should brand-check this and not allow '[object ReadableStream]': FAIL

New Tests That Don't Pass

/streams/piping/pipe-through.any.sharedworker.html: ERROR [GitHub], SKIP [Gecko-android-em-7.0-x86_64-lite-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-qr-opt, Gecko-windows10-32-qr-debug, Gecko-windows10-32-qr-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt] (Chrome: OK, Safari: ERROR)
Piping through a duck-typed pass-through transform stream should work: FAIL (Chrome: PASS)
Piping through a transform errored on the writable end does not cause an unhandled promise rejection: FAIL (Chrome: PASS)
pipeThrough should not call pipeTo on this: FAIL (Chrome: PASS)
pipeThrough should not call pipeTo on the ReadableStream prototype: FAIL (Chrome: PASS)
pipeThrough should brand-check this and not allow 'null': FAIL (Chrome: PASS)
pipeThrough should brand-check this and not allow 'undefined': FAIL (Chrome: PASS)
pipeThrough should brand-check this and not allow '0': FAIL (Chrome: PASS)
pipeThrough should brand-check this and not allow 'NaN': FAIL (Chrome: PASS)
pipeThrough should brand-check this and not allow 'true': FAIL (Chrome: PASS)
pipeThrough should brand-check this and not allow 'ReadableStream': FAIL (Chrome: PASS)
pipeThrough should brand-check this and not allow '[object ReadableStream]': FAIL (Chrome: PASS)
/streams/piping/pipe-through.any.serviceworker.html: ERROR [GitHub], SKIP [Gecko-android-em-7.0-x86_64-lite-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-qr-opt, Gecko-windows10-32-qr-debug, Gecko-windows10-32-qr-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt] (Chrome: OK, Safari: ERROR)
/streams/piping/pipe-through.any.html: ERROR [GitHub], SKIP [Gecko-android-em-7.0-x86_64-lite-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-qr-opt, Gecko-windows10-32-qr-debug, Gecko-windows10-32-qr-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt] (Chrome: OK, Safari: ERROR)
Piping through a duck-typed pass-through transform stream should work: FAIL (Chrome: PASS, Safari: PASS)
Piping through a transform errored on the writable end does not cause an unhandled promise rejection: FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should not call pipeTo on this: FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should not call pipeTo on the ReadableStream prototype: FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow 'null': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow 'undefined': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow '0': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow 'NaN': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow 'true': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow 'ReadableStream': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow '[object ReadableStream]': FAIL (Chrome: PASS, Safari: PASS)
/streams/piping/pipe-through.any.worker.html: ERROR [GitHub], SKIP [Gecko-android-em-7.0-x86_64-lite-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-qr-opt, Gecko-windows10-32-qr-debug, Gecko-windows10-32-qr-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt] (Chrome: OK, Safari: ERROR)
Piping through a duck-typed pass-through transform stream should work: FAIL (Chrome: PASS, Safari: PASS)
Piping through a transform errored on the writable end does not cause an unhandled promise rejection: FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should not call pipeTo on this: FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should not call pipeTo on the ReadableStream prototype: FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow 'null': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow 'undefined': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow '0': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow 'NaN': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow 'true': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow 'ReadableStream': FAIL (Chrome: PASS, Safari: PASS)
pipeThrough should brand-check this and not allow '[object ReadableStream]': FAIL (Chrome: PASS, Safari: PASS)
/streams/piping/pipe-through.any.js: SKIP (Chrome: SKIP, Safari: SKIP)

Tests Disabled in Gecko Infrastructure

Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)

Assignee

Updated

•

5 years ago

Depends on: 1724306

Pulsebot

Comment 3

•

5 years ago

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/30484218dd85 [wpt PR 29880] - WebKit export of https://bugs.webkit.org/show_bug.cgi?id=227690, a=testonly

Marian-Vasile Laza

Comment 4

•

5 years ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/30484218dd85

Status: NEW → RESOLVED

Closed: 5 years ago

status-firefox92: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → 92 Branch

You need to log in before you can comment on or make changes to this bug.

Bugzilla

[wpt-sync] Sync PR 29880 - WebKit export of https://bugs.webkit.org/show_bug.cgi?id=227690

Categories

(Core :: JavaScript Engine, task, P4)

Tracking

()

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Crash Data

Security

(public)

User Story

Description

Updated

Comment 1

Comment 2

CI Results

Status Summary

Firefox

Chrome

Safari

Links

Details

Firefox-only Failures

New Tests That Don't Pass

Tests Disabled in Gecko Infrastructure

Updated

Comment 3

Comment 4