1878300 - https://dmnsgn.github.io/media-codecs/ crash tab process

Status: RESOLVED FIXED

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

Description

[rinsuki]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:124.0) Gecko/20100101 Firefox/124.0

Steps to reproduce:

1. Open https://dmnsgn.github.io/media-codecs/ in Firefox Nightly

Actual results:

tab process crashes.

mozregression shows this pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=c4ed504122f310fc62c530dc429a40a61a8bcbdf&tochange=98d2ecb3f476864bbf310ab893e7a4ae4f77f9e6

Reproduced in:

• macOS 14.4 beta (23E5180j) + Firefox Nightly (2024-02-01)
• Windows Server 2019 + Firefox Nightly (2024-02-02)

Comment 1

[Alice0775 White]

Crash report: https://crash-stats.mozilla.org/report/index/0e9ebe62-5a52-4e8d-85d1-e0b240240202

MOZ_CRASH Reason: MOZ_CRASH(The string should always contain a valid codec at this point.)

Top 7 frames of crashing thread:

0  xul.dll  mozilla::dom::VideoEncoderConfigInternal::ToEncoderConfig const  dom/media/webcodecs/VideoEncoder.cpp:225
1  xul.dll  mozilla::EncoderSupport::Supports  dom/media/EncoderTraits.h:17
2  xul.dll  mozilla::dom::CanEncode  dom/media/webcodecs/VideoEncoder.cpp:380
3  xul.dll  mozilla::dom::VideoEncoder::IsConfigSupported  dom/media/webcodecs/VideoEncoder.cpp:566
4  xul.dll  mozilla::dom::VideoEncoder_Binding::isConfigSupported  dom/bindings/VideoEncoderBinding.cpp:1844
5  xul.dll  mozilla::dom::StaticMethodPromiseWrapper  dom/bindings/BindingUtils.cpp:3297
6  ?  @0x000002a7a50230bb  

Comment 2

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

:padenot, since you are the author of the regressor, bug 1749047, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Comment 3

[Paul Adenot (:padenot)]

Attachment: Bug 1878300 - Only accept vp8 for vp8 codec string. r?chunmin

Comment 4

[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 5

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

Set release status flags based on info from the regressing bug 1749047

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/45b561797261
Only accept vp8 for vp8 codec string. r=chunmin

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/45027 for changes under testing/web-platform/tests

Comment 8

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/45b561797261

Comment 9

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot