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
authorMatt Woodrow <mwoodrow@mozilla.com>
Wed, 19 Apr 2017 15:16:13 +1200
changeset 353699 f4c724034728dbca08651d0b6047c8e778142237
parent 353698 c5785434af743a93ea15a6270dc5cab9259f0fa0
child 353700 f79d7564d39d7a622c2dbc6fd1722e74a40ab453
push id89322
push usermwoodrow@mozilla.com
push dateWed, 19 Apr 2017 04:05:13 +0000
treeherdermozilla-inbound@ad2f95eed556 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnical
bugs1341496
milestone55.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 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
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,