1891082 - Video on https://webvideo-js.pages.dev/ does not play

Status: VERIFIED FIXED

(Core :: Audio/Video: Web Codecs, defect)

Description

[Mayank Bansal]

Go to https://webvideo-js.pages.dev/
Click on the video to start playing

AR: Video keeps on loading. There is no error in console.
ER: It should play, Chrome does.

Comment 1

[Mayank Bansal]

Attachment: about:support

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:padenot, could you have a look please?

For more information, please visit BugBot documentation.

Comment 3

[Jim Mathies [:jimm]]

Support for webcodecs has not shipped yet.

Comment 4

[Paul Adenot (:padenot)]

This is caused by the website not passing extradata/description to the decoder. Chrome plays it though.

Comment 5

[Paul Adenot (:padenot)]

https://github.com/crux14/webvideo.js/blob/master/src/media/codec/decoderWebCodecs.ts#L160

Comment 6

[C.M.Chang[:chunmin]]

Attachment: Bug 1891082 - Correct log for audio rate

Comment 7

[C.M.Chang[:chunmin]]

Attachment: Bug 1891082 - Add a function for making AAC specific config

Depends on D219146

Comment 8

[C.M.Chang[:chunmin]]

Attachment: Bug 1891082 - Implement CodecChangeMonitor for AAC in MediaChangeMonitor

Comment 9

[C.M.Chang[:chunmin]]

Attachment: Bug 1891082 - Enable decoder reconfiguration in DecoderAgent for AAC/ADTS data through MediaChangeMonitor

Depends on D220505

Comment 10

[C.M.Chang[:chunmin]]

In brief, when the description is not present in AudioDecoderConfig for mp4a.* codec, the data stream is assumed to be in adts format. However, the provided data is in raw AAC format, so the audio cannot be decoded correctly.

Comment 11

[C.M.Chang[:chunmin]]

Attachment: Bug 1891082 - Remove outdated comments in DecoderAgent

Depends on D220506

Comment 12

[C.M.Chang[:chunmin]]

Attachment: Bug 1891082 - Add WPT testing AAC AudioDecoderConfig w/o description

Depends on D221224

Comment 13

[Phabricator Automation]

Comment on attachment 9423068 [details]
Bug 1891082 - Add WPT testing AAC AudioDecoderConfig w/o description

Revision D221279 was moved to bug 1917721. Setting attachment 9423068 [details] to obsolete.

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/173ac1b6ad6e
Correct log for audio rate r=media-playback-reviewers,alwu,padenot
https://hg.mozilla.org/integration/autoland/rev/8e10bbcaccea
Add a function for making AAC specific config r=media-playback-reviewers,padenot
https://hg.mozilla.org/integration/autoland/rev/85611c424c13
Implement CodecChangeMonitor for AAC in MediaChangeMonitor r=media-playback-reviewers,padenot
https://hg.mozilla.org/integration/autoland/rev/02c9b1fb72c8
Enable decoder reconfiguration in DecoderAgent for AAC/ADTS data through MediaChangeMonitor r=media-playback-reviewers,padenot
https://hg.mozilla.org/integration/autoland/rev/ec6625c1ead2
Remove outdated comments in DecoderAgent r=media-playback-reviewers,padenot

Comment 15

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/173ac1b6ad6e
https://hg.mozilla.org/mozilla-central/rev/8e10bbcaccea
https://hg.mozilla.org/mozilla-central/rev/85611c424c13
https://hg.mozilla.org/mozilla-central/rev/02c9b1fb72c8
https://hg.mozilla.org/mozilla-central/rev/ec6625c1ead2

Comment 16

[Ardelean Oana, Desktop Test Engineering [:oardelean]]

Reproducible on a 2024-09-08 Nightly build on Windows 10.
Verified as fixed on Firefox 132.0b2 and Firefox Nightly 133.0a1 on Windows 10, Ubuntu 22, macOS 14.