Bug 1370079: P1. Don't check for DXVA support when checking for VP9 support. r=mattwoodrow
authorJean-Yves Avenard <jyavenard@mozilla.com>
Thu, 08 Jun 2017 13:54:00 +0200
changeset 363085 cc4812a78c235db11fb2de247acec6c6ef1ff09b
parent 363084 16a2f50d486db9d24262b5888a4269ae81d56381
child 363086 381c3ee3d00e12bec3f9873f17867b0e1c68c914
push id31994
push usercbook@mozilla.com
push dateFri, 09 Jun 2017 10:56:24 +0000
treeherdermozilla-central@7c9d96bbc400 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmattwoodrow
bugs1370079, 1338011
milestone55.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1370079: P1. Don't check for DXVA support when checking for VP9 support. r=mattwoodrow SupportsMimeType is called in the content process, for which, following bug 1338011, dxva is marked as disabled. CanCreateWMFDecoder already performs the check of allowing VP9 only if it's hardware accelerated, so we don't need to test if dxva is enabled prior. MozReview-Commit-ID: 7Jvaop6ZznU
dom/media/platforms/wmf/WMFDecoderModule.cpp
--- a/dom/media/platforms/wmf/WMFDecoderModule.cpp
+++ b/dom/media/platforms/wmf/WMFDecoderModule.cpp
@@ -249,17 +249,17 @@ WMFDecoderModule::Supports(const TrackIn
       }
     }
     return true;
   }
   if (aTrackInfo.mMimeType.EqualsLiteral("audio/mpeg") &&
       CanCreateWMFDecoder<CLSID_CMP3DecMediaObject>()) {
     return true;
   }
-  if (MediaPrefs::PDMWMFVP9DecoderEnabled() && sDXVAEnabled) {
+  if (MediaPrefs::PDMWMFVP9DecoderEnabled()) {
     if ((VPXDecoder::IsVP8(aTrackInfo.mMimeType)
          || VPXDecoder::IsVP9(aTrackInfo.mMimeType))
         && CanCreateWMFDecoder<CLSID_WebmMfVpxDec>()) {
       return true;
     }
   }
 
   // Some unsupported codec.