Bug 1410183 - Replace the useless layers-dump call with a slightly more useful if-0 block. r?jrmuizel draft
authorKartikaya Gupta <kgupta@mozilla.com>
Thu, 19 Oct 2017 14:02:48 -0400
changeset 683428 9407162a95dec4077d362c2f6e365fcc83cd5672
parent 683427 34a16a8513a5decf0ef14481e7e0d2b4b24d37f8
child 736646 096c3d6a2706b752e365804dad4dfd460db15253
push id85375
push userkgupta@mozilla.com
push dateThu, 19 Oct 2017 18:03:06 +0000
reviewersjrmuizel
bugs1410183
milestone58.0a1
Bug 1410183 - Replace the useless layers-dump call with a slightly more useful if-0 block. r?jrmuizel MozReview-Commit-ID: AEqJEiLZjmx
gfx/layers/wr/WebRenderLayerManager.cpp
--- a/gfx/layers/wr/WebRenderLayerManager.cpp
+++ b/gfx/layers/wr/WebRenderLayerManager.cpp
@@ -246,19 +246,21 @@ WebRenderLayerManager::EndTransactionWit
                                                   nsDisplayListBuilder* aDisplayListBuilder)
 {
   MOZ_ASSERT(aDisplayList && aDisplayListBuilder);
   WrBridge()->RemoveExpiredFontKeys();
 
   AUTO_PROFILER_TRACING("Paint", "RenderLayers");
   mTransactionIncomplete = false;
 
-  if (gfxPrefs::LayersDump()) {
-    this->Dump();
-  }
+#if 0
+  // Useful for debugging, it dumps the display list *before* we try to build
+  // WR commands from it
+  nsFrame::PrintDisplayList(aDisplayListBuilder, *aDisplayList);
+#endif
 
   // Since we don't do repeat transactions right now, just set the time
   mAnimationReadyTime = TimeStamp::Now();
 
   WrBridge()->BeginTransaction();
   DiscardCompositorAnimations();
 
   LayoutDeviceIntSize size = mWidget->GetClientSize();