Bug 1681043 - P6. Remove no longer necessary workaround. r=alwu,bryce

The WMFDecoderModule now uses a crash guard around the loading of the VP9/VP8 MFT. We don't want to always assume that we have a VPx decoder in the GPU process in particular, as it will only be true if a HW decoder is available.

Differential Revision: https://phabricator.services.mozilla.com/D100309
This commit is contained in:
Jean-Yves Avenard 2021-01-03 23:24:35 +00:00
Родитель 19f9c7de4d
Коммит ea48adf8c2
1 изменённых файлов: 0 добавлений и 9 удалений

Просмотреть файл

@ -390,15 +390,6 @@ bool PDMFactory::Supports(const SupportDecoderParams& aParams,
if (mEMEPDM) { if (mEMEPDM) {
return mEMEPDM->Supports(aParams, aDiagnostics); return mEMEPDM->Supports(aParams, aDiagnostics);
} }
if (VPXDecoder::IsVPX(aParams.MimeType(),
VPXDecoder::VP8 | VPXDecoder::VP9) &&
!aParams.mConfig.GetAsVideoInfo()->HasAlpha()) {
// Work around bug 1521370, where trying to instantiate an external decoder
// could cause a crash.
// We always ship a VP8/VP9 decoder (libvpx) and optionally we have ffvpx.
// So we can speed up the test by assuming that this codec is supported.
return true;
}
RefPtr<PlatformDecoderModule> current = RefPtr<PlatformDecoderModule> current =
GetDecoderModule(aParams, aDiagnostics); GetDecoderModule(aParams, aDiagnostics);