Bug 1400868 - Destroy backface-hidden display items that we're not going to process. r=jrmuizel
authorKartikaya Gupta <kgupta@mozilla.com>
Mon, 18 Sep 2017 12:22:19 -0400
changeset 381641 05c577db81427fe623d81ebb016f90148795670f
parent 381640 a2a1b037d34287da3bfac7a287a55aae779541f3
child 381642 407de8da5b2b42c48c3d742496a27dd567fe1e79
push id32535
push userkwierso@gmail.com
push dateTue, 19 Sep 2017 21:06:08 +0000
treeherdermozilla-central@c0d1f9eb2a40 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjrmuizel
bugs1400868
milestone57.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 1400868 - Destroy backface-hidden display items that we're not going to process. r=jrmuizel MozReview-Commit-ID: EPZwJ2upjXX
gfx/layers/wr/WebRenderLayerManager.cpp
--- a/gfx/layers/wr/WebRenderLayerManager.cpp
+++ b/gfx/layers/wr/WebRenderLayerManager.cpp
@@ -251,16 +251,17 @@ WebRenderLayerManager::CreateWebRenderCo
       = item->GetSameCoordinateSystemChildren();
     if (item->ShouldFlattenAway(aDisplayListBuilder)) {
       aDisplayList->AppendToBottom(itemSameCoordinateSystemChildren);
       item->Destroy(aDisplayListBuilder);
       continue;
     }
 
     if (item->BackfaceIsHidden() && aSc.IsBackfaceVisible()) {
+      item->Destroy(aDisplayListBuilder);
       continue;
     }
 
     savedItems.AppendToTop(item);
 
     bool forceNewLayerData = false;
     size_t layerCountBeforeRecursing = mLayerScrollData.size();
     if (apzEnabled) {