[Slimming Paint] Blink-side contiguous allocation of display items.

Source/core/paint/SVGMaskPainter.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/paint/SVGMaskPainter.h"
 
 #include "core/layout/svg/LayoutSVGResourceMasker.h"
 #include "core/paint/CompositingRecorder.h"
 #include "core/paint/LayoutObjectDrawingRecorder.h"
@@ skipping 10 matching lines @@
     ASSERT(context);
     ASSERT(m_mask.style());
     ASSERT_WITH_SECURITY_IMPLICATION(!m_mask.needsLayout());
 
     m_mask.clearInvalidationMask();
 
     FloatRect paintInvalidationRect = object.paintInvalidationRectInLocalCoordinates();
     if (paintInvalidationRect.isEmpty() || !m_mask.element()->hasChildren())
         return false;
 
+    BeginCompositingDisplayItem beginCompositingContent(object, SkXfermode::kSrcOver_Mode, 1, &paintInvalidationRect);
     if (RuntimeEnabledFeatures::slimmingPaintEnabled()) {
         ASSERT(context->displayItemList());
         if (context->displayItemList()->displayItemConstructionIsDisabled())
             return true;
-        context->displayItemList()->add(BeginCompositingDisplayItem::create(object, SkXfermode::kSrcOver_Mode, 1, &paintInvalidationRect));
+        context->displayItemList()->add(beginCompositingContent);
     } else {
-        BeginCompositingDisplayItem beginCompositingContent(object, SkXfermode::kSrcOver_Mode, 1, &paintInvalidationRect);
         beginCompositingContent.replay(*context);
     }
 
     return true;
 }
 
 void SVGMaskPainter::finishEffect(const LayoutObject& object, GraphicsContext* context)
 {
     ASSERT(context);
     ASSERT(m_mask.style());
     ASSERT_WITH_SECURITY_IMPLICATION(!m_mask.needsLayout());
 
     FloatRect paintInvalidationRect = object.paintInvalidationRectInLocalCoordinates();
     {
         ColorFilter maskLayerFilter = m_mask.style()->svgStyle().maskType() == MT_LUMINANCE
             ? ColorFilterLuminanceToAlpha : ColorFilterNone;
         CompositingRecorder maskCompositing(*context, object, SkXfermode::kDstIn_Mode, 1, &paintInvalidationRect, maskLayerFilter);
         drawMaskForLayoutObject(context, object, object.objectBoundingBox());
     }
 
+    EndCompositingDisplayItem endCompositingContent(object);
     if (RuntimeEnabledFeatures::slimmingPaintEnabled()) {
         ASSERT(context->displayItemList());
         if (context->displayItemList()->displayItemConstructionIsDisabled())
             return;
-        context->displayItemList()->add(EndCompositingDisplayItem::create(object));
+        context->displayItemList()->add(endCompositingContent);
     } else {
-        EndCompositingDisplayItem endCompositingContent(object);
         endCompositingContent.replay(*context);
     }
 }
 
 void SVGMaskPainter::drawMaskForLayoutObject(GraphicsContext* context, const LayoutObject& layoutObject, const FloatRect& targetBoundingBox)
 {
     ASSERT(context);
 
     AffineTransform contentTransformation;
     RefPtr<const SkPicture> maskContentPicture = m_mask.createContentPicture(contentTransformation, targetBoundingBox, context);
 
+    DrawingDisplayItem maskPicture(layoutObject, DisplayItem::SVGMask, maskContentPicture);
     TransformRecorder recorder(*context, layoutObject, contentTransformation);
     if (RuntimeEnabledFeatures::slimmingPaintEnabled()) {
         ASSERT(context->displayItemList());
         if (context->displayItemList()->displayItemConstructionIsDisabled())
             return;
-        context->displayItemList()->add(DrawingDisplayItem::create(layoutObject, DisplayItem::SVGMask, maskContentPicture));
+        context->displayItemList()->add(maskPicture);
     } else {
-        DrawingDisplayItem maskPicture(layoutObject, DisplayItem::SVGClip, maskContentPicture);
         maskPicture.replay(*context);
     }
 }
 
 }