Closed Bug 1726596 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 30089 - Update colorSpace nullability and encoder config

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox93 fixed)

RESOLVED FIXED
93 Branch
Tracking Status
firefox93 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

b'Chris Cunningham <chcunningham@chromium.org>' wrote:

Update colorSpace nullability and encoder config

Per https://github.com/w3c/webcodecs/pull/346

VideoColorSpace attributes are no longer nullable (their internals still
are). And VideoEncoderConfig.colorSpace is now removed (UA picks output
color space).

Work remains to fully plumb the correct output colorspace. See bug
crbug/1241448

Bug: 1241470, 1241448

Change-Id: Id8d8c78e5b936bb696e42c1d121e4c0102fa7abf

Reviewed-on: https://chromium-review.googlesource.com/3106040
WPT-Export-Revision: 475db4aa0bdf92d6a4f979476f5e0b5f42ad8052

CI Results

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

Total 4 tests and 17 subtests

Status Summary

Firefox

OK : 4
FAIL: 54

Chrome

OK : 4
PASS: 50
FAIL: 4

Safari

OK : 4
FAIL: 54

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/webcodecs/video-encoder.any.html
Test VideoEncoder construction: FAIL (Chrome: PASS, Safari: FAIL)
Test VideoEncoder.configure(): FAIL (Chrome: PASS, Safari: FAIL)
Test successful configure(), encode(), and flush(): FAIL (Chrome: PASS, Safari: FAIL)
encodeQueueSize test: FAIL (Chrome: PASS, Safari: FAIL)
Test successful reset() and re-confiugre(): FAIL (Chrome: PASS, Safari: FAIL)
Test successful encode() after re-configure().: FAIL (Chrome: PASS, Safari: FAIL)
Verify closed VideoEncoder operations: FAIL (Chrome: PASS, Safari: FAIL)
Verify unconfigured VideoEncoder operations: FAIL (Chrome: PASS, Safari: FAIL)
Verify encoding closed frames throws.: FAIL (Chrome: PASS, Safari: FAIL)
Encode video with negative timestamp: FAIL (Chrome: PASS, Safari: FAIL)
/webcodecs/video-encoder.any.worker.html
Test VideoEncoder construction: FAIL (Chrome: PASS, Safari: FAIL)
Test VideoEncoder.configure(): FAIL (Chrome: PASS, Safari: FAIL)
Test successful configure(), encode(), and flush(): FAIL (Chrome: PASS, Safari: FAIL)
encodeQueueSize test: FAIL (Chrome: PASS, Safari: FAIL)
Test successful reset() and re-confiugre(): FAIL (Chrome: PASS, Safari: FAIL)
Test successful encode() after re-configure().: FAIL (Chrome: PASS, Safari: FAIL)
Verify closed VideoEncoder operations: FAIL (Chrome: PASS, Safari: FAIL)
Verify unconfigured VideoEncoder operations: FAIL (Chrome: PASS, Safari: FAIL)
Verify encoding closed frames throws.: FAIL (Chrome: PASS, Safari: FAIL)
Encode video with negative timestamp: FAIL (Chrome: PASS, Safari: FAIL)
/webcodecs/videoFrame-construction.any.worker.html
Test we can construct a VideoFrame.: FAIL (Chrome: PASS, Safari: FAIL)
Test closed VideoFrame.: FAIL (Chrome: FAIL, Safari: FAIL)
Test we can construct a VideoFrame with a negative timestamp.: FAIL (Chrome: PASS, Safari: FAIL)
Test we can construct an odd-sized VideoFrame.: FAIL (Chrome: PASS, Safari: FAIL)
Test constructing VideoFrames from closed ImageBitmap throws.: FAIL (Chrome: PASS, Safari: FAIL)
Test invalid buffer constructed VideoFrames: FAIL (Chrome: PASS, Safari: FAIL)
Test Uint8Array(ArrayBuffer) constructed I420 VideoFrame: FAIL (Chrome: PASS, Safari: FAIL)
Test ArrayBuffer constructed I420 VideoFrame: FAIL (Chrome: PASS, Safari: FAIL)
Test planar constructed I420 VideoFrame with colorSpace: FAIL (Chrome: PASS, Safari: FAIL)
Test buffer constructed I420+Alpha VideoFrame: FAIL (Chrome: PASS, Safari: FAIL)
Test buffer constructed NV12 VideoFrame: FAIL (Chrome: PASS, Safari: FAIL)
Test buffer constructed RGB VideoFrames: FAIL (Chrome: PASS, Safari: FAIL)
Test VideoFrame constructed VideoFrame: FAIL (Chrome: PASS, Safari: FAIL)
Test we can construct a VideoFrame from an offscreen canvas.: FAIL (Chrome: PASS, Safari: FAIL)
Test I420A VideoFrame and alpha={keep,discard}: FAIL (Chrome: PASS, Safari: FAIL)
Test RGBA, BGRA VideoFrames with alpha={keep,discard}: FAIL (Chrome: PASS, Safari: FAIL)
Test a VideoFrame constructed from canvas can drop the alpha channel.: FAIL (Chrome: FAIL, Safari: FAIL)
/webcodecs/videoFrame-construction.any.html
Test we can construct a VideoFrame.: FAIL (Chrome: PASS, Safari: FAIL)
Test closed VideoFrame.: FAIL (Chrome: FAIL, Safari: FAIL)
Test we can construct a VideoFrame with a negative timestamp.: FAIL (Chrome: PASS, Safari: FAIL)
Test we can construct an odd-sized VideoFrame.: FAIL (Chrome: PASS, Safari: FAIL)
Test constructing VideoFrames from closed ImageBitmap throws.: FAIL (Chrome: PASS, Safari: FAIL)
Test invalid buffer constructed VideoFrames: FAIL (Chrome: PASS, Safari: FAIL)
Test Uint8Array(ArrayBuffer) constructed I420 VideoFrame: FAIL (Chrome: PASS, Safari: FAIL)
Test ArrayBuffer constructed I420 VideoFrame: FAIL (Chrome: PASS, Safari: FAIL)
Test planar constructed I420 VideoFrame with colorSpace: FAIL (Chrome: PASS, Safari: FAIL)
Test buffer constructed I420+Alpha VideoFrame: FAIL (Chrome: PASS, Safari: FAIL)
Test buffer constructed NV12 VideoFrame: FAIL (Chrome: PASS, Safari: FAIL)
Test buffer constructed RGB VideoFrames: FAIL (Chrome: PASS, Safari: FAIL)
Test VideoFrame constructed VideoFrame: FAIL (Chrome: PASS, Safari: FAIL)
Test we can construct a VideoFrame from an offscreen canvas.: FAIL (Chrome: PASS, Safari: FAIL)
Test I420A VideoFrame and alpha={keep,discard}: FAIL (Chrome: PASS, Safari: FAIL)
Test RGBA, BGRA VideoFrames with alpha={keep,discard}: FAIL (Chrome: PASS, Safari: FAIL)
Test a VideoFrame constructed from canvas can drop the alpha channel.: FAIL (Chrome: FAIL, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7775d8194b92
[wpt PR 30089] - Update colorSpace nullability and encoder config, a=testonly
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 93 Branch
You need to log in before you can comment on or make changes to this bug.