Bug 1341496 - Part 2: Don't use a separate ReadLock for the second component alpha texture as they should always be locked/unlocked at the same time. r?nical draft
authorMatt Woodrow <mwoodrow@mozilla.com>
Fri, 07 Apr 2017 16:36:13 +1200
changeset 557682 86370a556914d409810072c78534896c39579e86
parent 557681 70503bbbc3dc89100af5c2cc4c27af92b599108c
child 557683 4e7b7f2bca337afd0f6372f4722fc2181b7976d6
push id52782
push usermwoodrow@mozilla.com
push dateFri, 07 Apr 2017 04:36:35 +0000
reviewersnical
bugs1341496
milestone55.0a1
Bug 1341496 - Part 2: Don't use a separate ReadLock for the second component alpha texture as they should always be locked/unlocked at the same time. r?nical MozReview-Commit-ID: IdkX2CVntX4
gfx/layers/client/ContentClient.cpp
--- a/gfx/layers/client/ContentClient.cpp
+++ b/gfx/layers/client/ContentClient.cpp
@@ -326,17 +326,19 @@ ContentClientRemoteBuffer::CreateBackBuf
       mForwarder->GetCompositorBackendType(),
       mTextureFlags | ExtraTextureFlags(),
       TextureAllocationFlags::ALLOC_CLEAR_BUFFER_WHITE
     );
     if (!mTextureClientOnWhite || !AddTextureClient(mTextureClientOnWhite)) {
       AbortTextureClientCreation();
       return;
     }
-    mTextureClientOnWhite->EnableBlockingReadLock();
+    // We don't enable the readlock for the white buffer since we always
+    // use them together and waiting on the lock for the black
+    // should be sufficient.
   }
 }
 
 void
 ContentClientRemoteBuffer::CreateBuffer(ContentType aType,
                                         const IntRect& aRect,
                                         uint32_t aFlags,
                                         RefPtr<gfx::DrawTarget>* aBlackDT,