Closed Bug 1705848 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 28557 - Streams: tests for readable byte stream fixes

Categories

(Core :: JavaScript Engine, task, P4)

Product:
Core
Component:
JavaScript Engine
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
90 Branch
Tracking Flags:
Tracking Status
firefox90 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Attachments

(1 file)

Notable wpt changes
143.82 KB, text/plain
Details

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

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

b'Mattias Buelens <mattias@buelens.com>' wrote:

Streams: tests for readable byte stream fixes

See whatwg/streams#1123 for the accompanying spec change.

Component: web-platform-tests → JavaScript Engine
Product: Testing → Core
Attached file Notable wpt changes — 
# CI Results

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

Total 71 tests and 24 subtests

## Status Summary

### Firefox
OK   : 24[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-opt, Gecko-windows10-32-debug, Gecko-windows10-32-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt] 49[GitHub]
PASS : 400[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-opt, Gecko-windows10-32-debug, Gecko-windows10-32-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt] 429[GitHub]
FAIL : 196[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux1804-64-tsan-opt, Gecko-windows10-32-debug, Gecko-windows10-32-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt] 814[GitHub]
ERROR: 8

### Chrome
OK   : 48
PASS : 1112
CRASH: 9
FAIL : 204

### Safari
OK   : 31
PASS : 519
FAIL : 493
ERROR: 26

## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=0e2652425b5383bf58fa6d1eb1f33b0608b64486)
[GitHub PR Head](https://wpt.fyi/results/?sha=473ec364ca7f450ab4b708ee2ca8417f30b4f0cc&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=473ec364ca7f450ab4b708ee2ca8417f30b4f0cc&label=pr_base)

## Details

### Firefox-only Failures
/streams/piping/flow-control.any.worker.html
  Piping from a non-empty ReadableStream into a WritableStream that does not desire chunks: FAIL
  Piping from a non-empty ReadableStream into a WritableStream that does not desire chunks, but then does: FAIL
  Piping from an empty ReadableStream into a WritableStream that does not desire chunks, but then the readable stream becomes non-empty and the writable stream starts desiring chunks: FAIL
  Piping from a ReadableStream to a WritableStream that desires more chunks before finishing with previous ones: FAIL
  Piping to a WritableStream that does not consume the writes fast enough exerts backpressure on the ReadableStream: FAIL
/streams/piping/flow-control.any.html
  Piping from a non-empty ReadableStream into a WritableStream that does not desire chunks: FAIL
  Piping from a non-empty ReadableStream into a WritableStream that does not desire chunks, but then does: FAIL
  Piping from an empty ReadableStream into a WritableStream that does not desire chunks, but then the readable stream becomes non-empty and the writable stream starts desiring chunks: FAIL
  Piping from a ReadableStream to a WritableStream that desires more chunks before finishing with previous ones: FAIL
  Piping to a WritableStream that does not consume the writes fast enough exerts backpressure on the ReadableStream: FAIL
/streams/piping/flow-control.any.serviceworker.html
  Piping from a non-empty ReadableStream into a WritableStream that does not desire chunks: FAIL
  Piping from a non-empty ReadableStream into a WritableStream that does not desire chunks, but then does: FAIL
  Piping from an empty ReadableStream into a WritableStream that does not desire chunks, but then the readable stream becomes non-empty and the writable stream starts desiring chunks: FAIL
  Piping from a ReadableStream to a WritableStream that desires more chunks before finishing with previous ones: FAIL
  Piping to a WritableStream that does not consume the writes fast enough exerts backpressure on the 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
/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/readable-byte-streams/general.any.serviceworker.html
  ReadableStream with byte source: pull() function is not callable: FAIL
  ReadableStream with byte source: autoAllocateChunkSize cannot be 0: FAIL
/streams/readable-byte-streams/general.any.worker.html
  ReadableStream with byte source: pull() function is not callable: FAIL
  ReadableStream with byte source: autoAllocateChunkSize cannot be 0: FAIL
/streams/readable-byte-streams/general.any.html
  ReadableStream with byte source: pull() function is not callable: FAIL
  ReadableStream with byte source: autoAllocateChunkSize cannot be 0: FAIL
/streams/transform-streams/general.any.serviceworker.html
  TransformStream can be constructed with a transform function: FAIL
  TransformStream can be constructed with no transform function: FAIL
  TransformStream writable starts in the writable state: FAIL
  Identity TransformStream: can read from readable what is put into writable: FAIL
  Uppercaser sync TransformStream: can read from readable transformed version of what is put into writable: FAIL
  Uppercaser-doubler sync TransformStream: can read both chunks put into the readable: FAIL
  Uppercaser async TransformStream: can read from readable transformed version of what is put into writable: FAIL
  Uppercaser-doubler async TransformStream: can read both chunks put into the readable: FAIL
  TransformStream: by default, closing the writable closes the readable (when there are no queued writes): FAIL
  TransformStream: by default, closing the writable waits for transforms to finish before closing both: FAIL
  TransformStream: by default, closing the writable closes the readable after sync enqueues and async done: FAIL
  TransformStream: by default, closing the writable closes the readable after async enqueues and async done: FAIL
  Transform stream should call transformer methods as methods: FAIL
  methods should not not have .apply() or .call() called: FAIL
  TransformStream start, transform, and flush should be strictly ordered: FAIL
  it should be possible to call transform() synchronously: FAIL
  closing the writable should close the readable when there are no queued chunks, even with backpressure: FAIL
  enqueue() should throw after controller.terminate(): FAIL
  enqueue() should throw after readable.cancel(): FAIL
  controller.terminate() should do nothing the second time it is called: FAIL
  terminate() should do nothing after readable.cancel(): FAIL
  start() should not be called twice: FAIL
  specifying a defined readableType should throw: FAIL
  specifying a defined writableType should throw: FAIL
/streams/transform-streams/general.any.worker.html
  TransformStream can be constructed with a transform function: FAIL
  TransformStream can be constructed with no transform function: FAIL
  TransformStream writable starts in the writable state: FAIL
  Identity TransformStream: can read from readable what is put into writable: FAIL
  Uppercaser sync TransformStream: can read from readable transformed version of what is put into writable: FAIL
  Uppercaser-doubler sync TransformStream: can read both chunks put into the readable: FAIL
  Uppercaser async TransformStream: can read from readable transformed version of what is put into writable: FAIL
  Uppercaser-doubler async TransformStream: can read both chunks put into the readable: FAIL
  TransformStream: by default, closing the writable closes the readable (when there are no queued writes): FAIL
  TransformStream: by default, closing the writable waits for transforms to finish before closing both: FAIL
  TransformStream: by default, closing the writable closes the readable after sync enqueues and async done: FAIL
  TransformStream: by default, closing the writable closes the readable after async enqueues and async done: FAIL
  Transform stream should call transformer methods as methods: FAIL
  methods should not not have .apply() or .call() called: FAIL
  TransformStream start, transform, and flush should be strictly ordered: FAIL
  it should be possible to call transform() synchronously: FAIL
  closing the writable should close the readable when there are no queued chunks, even with backpressure: FAIL
  enqueue() should throw after controller.terminate(): FAIL
  enqueue() should throw after readable.cancel(): FAIL
  controller.terminate() should do nothing the second time it is called: FAIL
  terminate() should do nothing after readable.cancel(): FAIL
  start() should not be called twice: FAIL
  specifying a defined readableType should throw: FAIL
  specifying a defined writableType should throw: FAIL
/streams/transform-streams/general.any.html
  TransformStream can be constructed with a transform function: FAIL
  TransformStream can be constructed with no transform function: FAIL
  TransformStream writable starts in the writable state: FAIL
  Identity TransformStream: can read from readable what is put into writable: FAIL
  Uppercaser sync TransformStream: can read from readable transformed version of what is put into writable: FAIL
  Uppercaser-doubler sync TransformStream: can read both chunks put into the readable: FAIL
  Uppercaser async TransformStream: can read from readable transformed version of what is put into writable: FAIL
  Uppercaser-doubler async TransformStream: can read both chunks put into the readable: FAIL
  TransformStream: by default, closing the writable closes the readable (when there are no queued writes): FAIL
  TransformStream: by default, closing the writable waits for transforms to finish before closing both: FAIL
  TransformStream: by default, closing the writable closes the readable after sync enqueues and async done: FAIL
  TransformStream: by default, closing the writable closes the readable after async enqueues and async done: FAIL
  Transform stream should call transformer methods as methods: FAIL
  methods should not not have .apply() or .call() called: FAIL
  TransformStream start, transform, and flush should be strictly ordered: FAIL
  it should be possible to call transform() synchronously: FAIL
  closing the writable should close the readable when there are no queued chunks, even with backpressure: FAIL
  enqueue() should throw after controller.terminate(): FAIL
  enqueue() should throw after readable.cancel(): FAIL
  controller.terminate() should do nothing the second time it is called: FAIL
  terminate() should do nothing after readable.cancel(): FAIL
  start() should not be called twice: FAIL
  specifying a defined readableType should throw: FAIL
  specifying a defined writableType should throw: FAIL
/streams/transform-streams/reentrant-strategies.any.html
  enqueue() inside size() should work: FAIL
  terminate() inside size() should work: FAIL
  error() inside size() should work: FAIL
  desiredSize inside size() should work: FAIL
  readable cancel() inside size() should work: FAIL
  pipeTo() inside size() should work: FAIL
  read() inside of size() should work: FAIL
  writer.write() inside size() should work: FAIL
  synchronous writer.write() inside size() should work: FAIL
  writer.close() inside size() should work: FAIL
  writer.abort() inside size() should work: FAIL
/streams/transform-streams/reentrant-strategies.any.serviceworker.html
  enqueue() inside size() should work: FAIL
  terminate() inside size() should work: FAIL
  error() inside size() should work: FAIL
  desiredSize inside size() should work: FAIL
  readable cancel() inside size() should work: FAIL
  pipeTo() inside size() should work: FAIL
  read() inside of size() should work: FAIL
  writer.write() inside size() should work: FAIL
  synchronous writer.write() inside size() should work: FAIL
  writer.close() inside size() should work: FAIL
  writer.abort() inside size() should work: FAIL
/streams/transform-streams/reentrant-strategies.any.worker.html
  enqueue() inside size() should work: FAIL
  terminate() inside size() should work: FAIL
  error() inside size() should work: FAIL
  desiredSize inside size() should work: FAIL
  readable cancel() inside size() should work: FAIL
  pipeTo() inside size() should work: FAIL
  read() inside of size() should work: FAIL
  writer.write() inside size() should work: FAIL
  synchronous writer.write() inside size() should work: FAIL
  writer.close() inside size() should work: FAIL
  writer.abort() inside size() should work: FAIL
/streams/readable-streams/reentrant-strategies.any.worker.html
  pipeTo() inside size() should behave as expected: FAIL
/streams/readable-streams/reentrant-strategies.any.html
  pipeTo() inside size() should behave as expected: FAIL
/streams/readable-streams/reentrant-strategies.any.serviceworker.html
  pipeTo() inside size() should behave as expected: FAIL

(See attachment for full changes)
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/37a7e6dc64f2
[wpt PR 28557] - Streams: tests for readable byte stream fixes, a=testonly
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4c91905f5c0b
[wpt PR 28557] - Streams: tests for readable byte stream fixes, a=testonly
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/15079f42f0bc
[wpt PR 28557] - Streams: tests for readable byte stream fixes, a=testonly

https://hg.mozilla.org/mozilla-central/rev/15079f42f0bc

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox90: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: