[SPv2] Flatten property trees in PaintRecordBuilder into a single display list.

third_party/WebKit/Source/core/paint/SVGShapePainter.cpp

 // Copyright 2014 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 "core/paint/SVGShapePainter.h"
 
 #include "core/layout/svg/LayoutSVGResourceMarker.h"
 #include "core/layout/svg/LayoutSVGShape.h"
 #include "core/layout/svg/SVGLayoutSupport.h"
 #include "core/layout/svg/SVGMarkerData.h"
@@ skipping 201 matching lines @@
   LayoutSVGResourceMarker* markerMid = resources->markerMid();
   LayoutSVGResourceMarker* markerEnd = resources->markerEnd();
   if (!markerStart && !markerMid && !markerEnd)
     return;
 
   float strokeWidth = m_layoutSVGShape.strokeWidth();
 
   for (const MarkerPosition& markerPosition : *markerPositions) {
     if (const LayoutSVGResourceMarker* marker = SVGMarkerData::markerForType(
             markerPosition.type, markerStart, markerMid, markerEnd)) {
-      PaintRecordBuilder builder(boundingBox, nullptr, &paintInfo.context);
-      PaintInfo markerPaintInfo(builder.context(), paintInfo);
-
-      // It's expensive to track the transformed paint cull rect for each
-      // marker so just disable culling. The shape paint call will already
-      // be culled if it is outside the paint info cull rect.
-      markerPaintInfo.m_cullRect.m_rect = LayoutRect::infiniteIntRect();
-
-      paintMarker(markerPaintInfo, *marker, markerPosition, strokeWidth);
-      paintInfo.context.canvas()->PlaybackPaintRecord(builder.endRecording());
+      paintMarker(paintInfo, *marker, markerPosition, strokeWidth);
     }
   }
 }
 
 void SVGShapePainter::paintMarker(const PaintInfo& paintInfo,
                                   const LayoutSVGResourceMarker& marker,
                                   const MarkerPosition& position,
                                   float strokeWidth) {
   if (!marker.shouldPaint())
     return;
 
-  TransformRecorder transformRecorder(
-      paintInfo.context, marker,
-      marker.markerTransformation(position.origin, position.angle,
-                                  strokeWidth));
-  Optional<FloatClipRecorder> clipRecorder;
+  AffineTransform transform =
+      marker.markerTransformation(position.origin, position.angle, strokeWidth);
+
+  PaintCanvas* canvas = paintInfo.context.canvas();
+
+  canvas->save();
+  canvas->concat(affineTransformToSkMatrix(transform));
   if (SVGLayoutSupport::isOverflowHidden(&marker))
-    clipRecorder.emplace(paintInfo.context, marker, paintInfo.phase,
-                         marker.viewport());
-  SVGContainerPainter(marker).paint(paintInfo);
+    canvas->clipRect(marker.viewport());
+
+  // It's expensive to track the transformed paint cull rect for each
+  // marker so just disable culling. The shape paint call will already
+  // be culled if it is outside the paint info cull rect.
+  IntRect bounds(LayoutRect::infiniteIntRect());
+  PaintRecordBuilder builder(FloatRect(bounds), nullptr, &paintInfo.context);
+  PaintInfo markerPaintInfo(builder.context(), paintInfo);
+  markerPaintInfo.m_cullRect.m_rect = bounds;
+
+  SVGContainerPainter(marker).paint(markerPaintInfo);
+  canvas->PlaybackPaintRecord(builder.endRecording());
+
+  canvas->restore();
 }
 
 }  // namespace blink