summaryrefslogtreecommitdiff
path: root/dom
diff options
context:
space:
mode:
authorJeremy Andrews <athenian200@outlook.com>2023-10-23 22:56:21 -0500
committerJeremy Andrews <athenian200@outlook.com>2023-10-24 00:59:11 -0500
commitf7c0599e1ee455e272a6ad4ba88d2bbf40cd506c (patch)
tree8489753fb26af181f658f43147b2455e2ea58d64 /dom
parent855109b02862ed5680500de105147bf6e79adc45 (diff)
downloaduxp-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
Diffstat (limited to 'dom')
-rw-r--r--dom/media/platforms/wmf/WMFDecoderModule.cpp8
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,