1841195 - Use VideoColorSpaceInternal in VideoFrame

Status: RESOLVED FIXED

(Core :: Audio/Video: Web Codecs, task, P3)

Description

[C.M.Chang[:chunmin]]

D182476 in bug 1831451 will introduce a VideoColorSpaceInternal class holding data from VideoColorSpace to avoid a unrooted hazard issue. We should use it in VideoFrame as well.

Comment 1

[C.M.Chang[:chunmin]]

Attachment: Bug 1841195 - Move VideoColorSpaceInternal to WebCodecsUtils

Comment 2

[C.M.Chang[:chunmin]]

Attachment: Bug 1841195 - Use VideoColorSpaceInternal in VideoFrame for greater flexibility

This patch replaces VideoColorSpaceInit with VideoColorSpaceInternal in
VideoFrame, enabling more flexible utilities and conversion functions. By
adopting VideoColorSpaceInternal, we can more easily log and map color space
details to underlying graphics properties, simplifying future color-related
enhancements and ensuring consistent handling of color space information
throughout the pipeline.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3f07a964d016
Move VideoColorSpaceInternal to WebCodecsUtils r=media-playback-reviewers,padenot
https://hg.mozilla.org/integration/autoland/rev/6d2bb8e6781c
Use VideoColorSpaceInternal in VideoFrame for greater flexibility r=media-playback-reviewers,padenot

Comment 4

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/3f07a964d016
https://hg.mozilla.org/mozilla-central/rev/6d2bb8e6781c