Bug 1372321 - Log the mask rect when passing a WrImageMask to WR. r=jrmuizel draft
authorKartikaya Gupta <kgupta@mozilla.com>
Tue, 13 Jun 2017 08:42:39 -0400
changeset 593332 e1311d451cb73365f7a21198231f9505a6695407
parent 593331 3f1dbd33553bd1053f9feaf8db7a8fd9ac3f4896
child 633072 9dd1767773aec3f6b7bf90c678d7ff32a7df553b
push id63654
push userkgupta@mozilla.com
push dateTue, 13 Jun 2017 12:43:18 +0000
reviewersjrmuizel
bugs1372321
milestone56.0a1
Bug 1372321 - Log the mask rect when passing a WrImageMask to WR. r=jrmuizel MozReview-Commit-ID: 3CdjfiwzZiJ
gfx/webrender_bindings/WebRenderAPI.cpp
--- a/gfx/webrender_bindings/WebRenderAPI.cpp
+++ b/gfx/webrender_bindings/WebRenderAPI.cpp
@@ -578,18 +578,19 @@ DisplayListBuilder::PopStackingContext()
   wr_dp_pop_stacking_context(mWrState);
 }
 
 void
 DisplayListBuilder::PushClip(const WrRect& aClipRect,
                              const WrImageMask* aMask)
 {
   uint64_t clip_id = wr_dp_push_clip(mWrState, aClipRect, aMask);
-  WRDL_LOG("PushClip id=%" PRIu64 " r=%s m=%p\n", clip_id,
-      Stringify(aClipRect).c_str(), aMask);
+  WRDL_LOG("PushClip id=%" PRIu64 " r=%s m=%p b=%s\n", clip_id,
+      Stringify(aClipRect).c_str(), aMask,
+      aMask ? Stringify(aMask->rect).c_str() : "none");
   mClipIdStack.push_back(clip_id);
 }
 
 void
 DisplayListBuilder::PopClip()
 {
   WRDL_LOG("PopClip id=%" PRIu64 "\n", mClipIdStack.back());
   mClipIdStack.pop_back();
@@ -862,30 +863,32 @@ DisplayListBuilder::PushBoxShadow(const 
                         aBlurRadius, aSpreadRadius, aBorderRadius,
                         aClipMode);
 }
 
 WrClipRegionToken
 DisplayListBuilder::PushClipRegion(const WrRect& aMain,
                                    const WrImageMask* aMask)
 {
-  WRDL_LOG("PushClipRegion r=%s m=%p\n", Stringify(aMain).c_str(), aMask);
+  WRDL_LOG("PushClipRegion r=%s m=%p b=%s\n", Stringify(aMain).c_str(), aMask,
+      aMask ? Stringify(aMask->rect).c_str() : "none");
   return wr_dp_push_clip_region(mWrState,
                                 aMain,
                                 nullptr, 0,
                                 aMask);
 }
 
 WrClipRegionToken
 DisplayListBuilder::PushClipRegion(const WrRect& aMain,
                                    const nsTArray<WrComplexClipRegion>& aComplex,
                                    const WrImageMask* aMask)
 {
-  WRDL_LOG("PushClipRegion r=%s cl=%d m=%p\n", Stringify(aMain).c_str(),
-      (int)aComplex.Length(), aMask);
+  WRDL_LOG("PushClipRegion r=%s cl=%d m=%p b=%s\n", Stringify(aMain).c_str(),
+      (int)aComplex.Length(), aMask,
+      aMask ? Stringify(aMask->rect).c_str() : "none");
   return wr_dp_push_clip_region(mWrState,
                                 aMain,
                                 aComplex.Elements(), aComplex.Length(),
                                 aMask);
 }
 
 Maybe<uint64_t>
 DisplayListBuilder::TopmostClipId()