Bug 1251523 - Remove TextureFlags::DEALLOCATE_CLIENT from AndroidSurfaceTextureData r=nical
authorSotaro Ikeda <sotaro.ikeda.g@gmail.com>
Mon, 14 Mar 2016 18:51:21 -0700
changeset 288674 727ac6e2c2273a0902e710ae8a388078fe3c45cc
parent 288673 a5365af3f8c6b1b308286328818ed273cb1a24c7
child 288675 27a4c68860fd1a25a85751b0b01352b9cba5ee05
push id30087
push usercbook@mozilla.com
push dateTue, 15 Mar 2016 09:43:43 +0000
treeherdermozilla-central@5e14887312d4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnical
bugs1251523
milestone48.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 1251523 - Remove TextureFlags::DEALLOCATE_CLIENT from AndroidSurfaceTextureData r=nical
gfx/layers/opengl/TextureClientOGL.cpp
--- a/gfx/layers/opengl/TextureClientOGL.cpp
+++ b/gfx/layers/opengl/TextureClientOGL.cpp
@@ -76,19 +76,16 @@ AndroidSurfaceTextureData::CreateTexture
 {
   MOZ_ASSERT(XRE_IsParentProcess(),
              "Can't pass an android surfaces between processes.");
 
   if (!aSurfTex || !XRE_IsParentProcess()) {
     return nullptr;
   }
 
-  // XXX - This is quite sad and slow.
-  aFlags |= TextureFlags::DEALLOCATE_CLIENT;
-
   if (aOriginPos == gl::OriginPos::BottomLeft) {
     aFlags |= TextureFlags::ORIGIN_BOTTOM_LEFT;
   }
 
   return TextureClient::CreateWithData(
     new AndroidSurfaceTextureData(aSurfTex, aSize),
     aFlags, aAllocator
   );