Bug 1251910: Fix YUV422 IOSurface. r=mattwoodrow. draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Sun, 28 Feb 2016 19:14:40 +1100
changeset 335268 b8fc6e1f67f8173adb6bf8928f4d9cef5fa902b9
parent 335267 4cba1f83f797b5cf6013a47683eff3f66e7d139b
child 515099 264872e2e1751568036f5c612d542bda0b6335cd
push id11749
push userbmo:jyavenard@mozilla.com
push dateSun, 28 Feb 2016 08:15:38 +0000
reviewersmattwoodrow
bugs1251910, 1249273, 1248323
milestone47.0a1
Bug 1251910: Fix YUV422 IOSurface. r=mattwoodrow. Bug 1249273 reverted the changes of bug 1248323. MozReview-Commit-ID: 4ri1gTOf2Dg
gfx/layers/composite/ImageHost.cpp
--- a/gfx/layers/composite/ImageHost.cpp
+++ b/gfx/layers/composite/ImageHost.cpp
@@ -352,17 +352,17 @@ ImageHost::Composite(LayerComposite* aLa
       // BindTextureSource above should have returned false!
       MOZ_ASSERT(false);
       return;
     }
 
     bool isAlphaPremultiplied =
         !(mCurrentTextureHost->GetFlags() & TextureFlags::NON_PREMULTIPLIED);
     RefPtr<TexturedEffect> effect =
-        CreateTexturedEffect(mCurrentTextureHost->GetFormat(),
+        CreateTexturedEffect(mCurrentTextureHost->GetReadFormat(),
             mCurrentTextureSource.get(), aFilter, isAlphaPremultiplied,
             GetRenderState());
     if (!effect) {
       return;
     }
 
     if (!GetCompositor()->SupportsEffect(effect->mType)) {
       return;
@@ -592,17 +592,17 @@ ImageHost::GenEffect(const gfx::Filter& 
   if (!mCurrentTextureHost->BindTextureSource(mCurrentTextureSource)) {
     return nullptr;
   }
   bool isAlphaPremultiplied = true;
   if (mCurrentTextureHost->GetFlags() & TextureFlags::NON_PREMULTIPLIED) {
     isAlphaPremultiplied = false;
   }
 
-  return CreateTexturedEffect(mCurrentTextureHost->GetFormat(),
+  return CreateTexturedEffect(mCurrentTextureHost->GetReadFormat(),
                               mCurrentTextureSource,
                               aFilter,
                               isAlphaPremultiplied,
                               GetRenderState());
 }
 
 void
 ImageHost::SetImageContainer(ImageContainerParent* aImageContainer)