diff options
author | Jeremy Andrews <athenian200@outlook.com> | 2023-10-23 22:56:21 -0500 |
---|---|---|
committer | Jeremy Andrews <athenian200@outlook.com> | 2023-10-24 00:59:11 -0500 |
commit | f7c0599e1ee455e272a6ad4ba88d2bbf40cd506c (patch) | |
tree | 8489753fb26af181f658f43147b2455e2ea58d64 | |
parent | 855109b02862ed5680500de105147bf6e79adc45 (diff) | |
download | uxp-f7c0599e1ee455e272a6ad4ba88d2bbf40cd506c.tar.gz |
Issue #2357 - WebM w/alpha renders black if WMF decoder is enabled.
Mozilla forgot to force use of VPXDecoder on Windows in their initial implementation, this is pretty much what they did for Android as well. Straight port of the Firefox 67 fix.
Ref: BZ 1528652
-rw-r--r-- | dom/media/platforms/wmf/WMFDecoderModule.cpp | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/dom/media/platforms/wmf/WMFDecoderModule.cpp b/dom/media/platforms/wmf/WMFDecoderModule.cpp index 98412a80a2..46e78fb17f 100644 --- a/dom/media/platforms/wmf/WMFDecoderModule.cpp +++ b/dom/media/platforms/wmf/WMFDecoderModule.cpp @@ -83,6 +83,14 @@ WMFDecoderModule::Startup() already_AddRefed<MediaDataDecoder> WMFDecoderModule::CreateVideoDecoder(const CreateDecoderParams& aParams) { + // Temporary - forces use of VPXDecoder when alpha is present. + // Bug 1263836 will handle alpha scenario once implemented. It will shift + // the check for alpha to PDMFactory but not itself remove the need for a + // check. + if (aParams.VideoConfig().HasAlpha()) { + return nullptr; + } + nsAutoPtr<WMFVideoMFTManager> manager( new WMFVideoMFTManager(aParams.VideoConfig(), aParams.mKnowsCompositor, |