Bug 1370079: P1. Don't check for DXVA support when checking for VP9 support. r?mattwoodrow draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Thu, 08 Jun 2017 13:54:00 +0200
changeset 591022 11aa999ea3cf341c3125847a4d1db88b923bb324
parent 591021 53e35ca66330e003b778b11e496778515c1db230
child 632397 deb08b715d949f3d985a41bb9197b45d3a432302
push id62926
push userbmo:jyavenard@mozilla.com
push dateThu, 08 Jun 2017 14:35:29 +0000
reviewersmattwoodrow
bugs1370079, 1338011
milestone55.0a1
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.