1844206 - Rework IsConfigSupported for h265

Status: RESOLVED FIXED

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

Comment 1

[C.M.Chang[:chunmin]]

Attachment: Bug 1844206 - Reject IsConfigSupported with explicit error message

Comment 2

[C.M.Chang[:chunmin]]

Attachment: Bug 1844206 - Add error logs in Configure & IsConfigSupported

Depends on D183925

Comment 3

[C.M.Chang[:chunmin]]

Attachment: Bug 1844206 - Add error logs in VideoDecoderConfigInternal::Create

Depends on D183926

Comment 4

[C.M.Chang[:chunmin]]

Attachment: Bug 1844206 - Add utils parsing H265 codec string

This patch adds a new function to parse H265 info.

Most of the code is borrowed from chromium [1] and checked with the H265
spec [2].

[1] https://source.chromium.org/chromium/chromium/src/+/main:media/base/video_codecs.cc;l=734;drc=caae8720b6a98875e237ab2f9be2a95d34f93caf
[2] https://www.itu.int/rec/T-REC-H.265

Depends on D183927

Comment 5

[C.M.Chang[:chunmin]]

Attachment: Bug 1844206 - Validate H265 codec string in VideoDecoder

This patches allows VideoDecoder to validate a H265 string. However,
since we don't support H265 in our decode pipeline yet, decoding the
h265 video will yeild an error.

The difference with the patch is that the IsConfigSupported now resolves
the promise with an codec-unsupported message instead of throwing a
TypeError saying this codec is invalid.

Depends on D184052

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/51d73caa4afd
Reject IsConfigSupported with explicit error message r=padenot
https://hg.mozilla.org/integration/autoland/rev/698f13a71041
Add error logs in Configure & IsConfigSupported r=padenot
https://hg.mozilla.org/integration/autoland/rev/c735fa0ce643
Add error logs in VideoDecoderConfigInternal::Create r=padenot
https://hg.mozilla.org/integration/autoland/rev/6ff04682013b
Add utils parsing H265 codec string r=media-playback-reviewers,alwu
https://hg.mozilla.org/integration/autoland/rev/e25889ce01c8
Validate H265 codec string in VideoDecoder r=padenot

Comment 7

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/51d73caa4afd
https://hg.mozilla.org/mozilla-central/rev/698f13a71041
https://hg.mozilla.org/mozilla-central/rev/c735fa0ce643
https://hg.mozilla.org/mozilla-central/rev/6ff04682013b
https://hg.mozilla.org/mozilla-central/rev/e25889ce01c8