Closed Bug 1496222 Opened 3 years ago Closed 3 years ago
ffvp9 fails to decode some VP9 files
46 bytes, text/x-phabricator-request
|Details | Review|
Those files: https://jyavenard.github.io/htmltests/mediatest/webm/vp9/yuv420p12.webm https://jyavenard.github.io/htmltests/mediatest/webm/vp9/yuv422p10.webm https://jyavenard.github.io/htmltests/mediatest/webm/vp9/yuv422p12.webm Fail to play with ffmpeg. They are made of a single frame. That frame is passed to avcodec_decode_video2 (https://searchfox.org/mozilla-central/source/dom/media/platforms/ffmpeg/FFmpegVideoDecoder.cpp#219) it will use all bytes, but doesn't return any frame. Then the decoder is drained, feeding null/zero to avcodec_decode_video2 which returns -1 Draining should never return -1.
If I disable MT decoding (by setting mCodecContext->thread_count = 1) then the first avcodec_decode_video2 returns 1
It's our code. https://searchfox.org/mozilla-central/source/dom/media/platforms/ffmpeg/FFmpegVideoDecoder.cpp#50 doesn't support 422 nor 420p12
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/44cc1c10c530 Add missing pixel format in FFmpeg decoder r=padenot
You need to log in before you can comment on or make changes to this bug.