Bug 1382744 - Have nsDisplayTransform reuse nsDisplayWrapList::CreateWebRenderCommands. r=ethlin
authorKartikaya Gupta <kgupta@mozilla.com>
Thu, 20 Jul 2017 14:14:27 -0400
changeset 418740 0e80f3606780c2d6ba84c70443ed9099099d6d8a
parent 418739 5e9d756f103639bfe8758d89c399b91d76901908
child 418741 3a683a5acc3ff0946f373f1ecfdf8dd290cf97fe
push id7566
push usermtabara@mozilla.com
push dateWed, 02 Aug 2017 08:25:16 +0000
treeherdermozilla-beta@86913f512c3c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersethlin
bugs1382744
milestone56.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 1382744 - Have nsDisplayTransform reuse nsDisplayWrapList::CreateWebRenderCommands. r=ethlin MozReview-Commit-ID: 5s3eECpao6s
layout/painting/nsDisplayList.cpp
--- a/layout/painting/nsDisplayList.cpp
+++ b/layout/painting/nsDisplayList.cpp
@@ -7608,22 +7608,18 @@ nsDisplayTransform::CreateWebRenderComma
                            aBuilder,
                            bounds,
                            origin,
                            animationsId,
                            nullptr,
                            transformForSC,
                            filters);
 
-  aManager->CreateWebRenderCommandsFromDisplayList(mStoredList.GetChildren(),
-                                                   aDisplayListBuilder,
-                                                   sc,
-                                                   aBuilder);
-
-  return true;
+  return mStoredList.CreateWebRenderCommands(aBuilder, sc, aParentCommands,
+                                             aManager, aDisplayListBuilder);
 }
 
 already_AddRefed<Layer> nsDisplayTransform::BuildLayer(nsDisplayListBuilder *aBuilder,
                                                        LayerManager *aManager,
                                                        const ContainerLayerParameters& aContainerParameters)
 {
   /* For frames without transform, it would not be removed for
    * backface hidden here.  But, it would be removed by the init