Skip to content

Commit

Permalink
Bug 1823317 - Downgrade diagnostic assert in FFmpegAudioDecoder. r=me…
Browse files Browse the repository at this point in the history
…dia-playback-reviewers,padenot

Differential Revision: https://phabricator.services.mozilla.com/D172954
  • Loading branch information
kinetiknz committed Mar 20, 2023
1 parent 8ecaf4e commit 38dde8a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions dom/media/platforms/ffmpeg/FFmpegAudioDecoder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ FFmpegAudioDecoder<LIBAV_VER>::FFmpegAudioDecoder(FFmpegLibWrapper* aLib,
}

if (mCodecID == AV_CODEC_ID_MP3) {
MOZ_DIAGNOSTIC_ASSERT(
aConfig.mCodecSpecificConfig.is<Mp3CodecSpecificData>());
// Downgraded from diagnostic assert due to BMO 1776524 on Android.
MOZ_ASSERT(aConfig.mCodecSpecificConfig.is<Mp3CodecSpecificData>());
// Gracefully handle bad data. If don't hit the preceding assert once this
// has been shipped for awhile, we can remove it and make the following code
// non-conditional.
Expand Down

0 comments on commit 38dde8a

Please sign in to comment.