Bug 1393087 - P4. Remove soft assertion. r?kinetik
MediaRawData::mKeyframe is now only set when both the alpha channel and standard channel are keyframes. Causing this assertion to be often false.
MozReview-Commit-ID: 5zYtFNyJSQB
--- a/dom/media/platforms/agnostic/VPXDecoder.cpp
+++ b/dom/media/platforms/agnostic/VPXDecoder.cpp
@@ -118,21 +118,16 @@ VPXDecoder::Flush()
});
}
RefPtr<MediaDataDecoder::DecodePromise>
VPXDecoder::ProcessDecode(MediaRawData* aSample)
{
MOZ_ASSERT(mTaskQueue->IsCurrentThreadIn());
-#if defined(DEBUG)
- NS_ASSERTION(IsKeyframe(*aSample, mCodec) == aSample->mKeyframe,
- "VPX Decode Keyframe error sample->mKeyframe and sample data out of sync");
-#endif
-
if (vpx_codec_err_t r = vpx_codec_decode(&mVPX, aSample->Data(), aSample->Size(), nullptr, 0)) {
LOG("VPX Decode error: %s", vpx_codec_err_to_string(r));
return DecodePromise::CreateAndReject(
MediaResult(NS_ERROR_DOM_MEDIA_DECODE_ERR,
RESULT_DETAIL("VPX error: %s", vpx_codec_err_to_string(r))),
__func__);
}