Put SVGResourceContainer subtypes into their own property trees.

third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.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 "core/paint/PaintPropertyTreeBuilderTest.h"
 
 #include "core/html/HTMLIFrameElement.h"
 #include "core/layout/LayoutTreeAsText.h"
 #include "core/paint/ObjectPaintProperties.h"
 #include "core/paint/PaintPropertyTreePrinter.h"
@@ skipping 3311 matching lines @@
 
   Element* mask = document().getElementById("mask");
   mask->setAttribute(HTMLNames::styleAttr, "height: 200px");
   document().view()->updateAllLifecyclePhases();
 
   ASSERT_EQ(properties, paintPropertiesForElement("mask"));
   ASSERT_EQ(maskClip, properties->maskClip());
   EXPECT_EQ(FloatRoundedRect(8, 8, 100, 200), maskClip->clipRect());
 }
 
+TEST_P(PaintPropertyTreeBuilderTest, SVGResource) {
+  // This test verifies the clip node generated for mask's implicit clip
+  // is correctly invalidated when a box resizes.
+  setBodyInnerHTML(
+      "<svg id='svg' xmlns='http://www.w3.org/2000/svg' >"
+      " <g transform='scale(1000)'>"
+      "   <marker id='markerMiddle'  markerWidth='2' markerHeight='2' refX='5' "
+      "refY='5' markerUnits='strokeWidth'>"

[Xianzhu, 2017/03/23 17:54:15]

Nit: indent the continuation.

[chrishtr, 2017/03/23 22:52:53]

Done.

+      "     <g id='transformInsideMarker' transform='scale(4)'>"
+      "       <circle cx='5' cy='5' r='7' fill='green'/>"
+      "     </g>"
+      "   </marker>"
+      " </g>"
+      " <g id='transformOutsidePath' transform='scale(2)'>"
+      "   <path d='M 130 135 L 180 135 L 180 185' "
+      "marker-mid='url(#markerMiddle)' fill='none' stroke-width='8px' "

[Xianzhu, 2017/03/23 17:54:15]

Nit: Ditto

[chrishtr, 2017/03/23 22:52:53]

Done.

+      "stroke='black'/>"
+      " </g>"
+      "</svg>");
+
+  const ObjectPaintProperties* transformInsideMarkerProperties =
+      paintPropertiesForElement("transformInsideMarker");
+  const ObjectPaintProperties* transformOutsidePathProperties =
+      paintPropertiesForElement("transformOutsidePath");
+  const ObjectPaintProperties* svgProperties = paintPropertiesForElement("svg");
+
+  // The <marker> object resets to a new paint property tree, so the
+  // transform within it should have the root as parent.
+  EXPECT_EQ(TransformPaintPropertyNode::root(),
+            transformInsideMarkerProperties->transform()->parent());
+
+  // Whereas this is not true of the transform above the path.
+  EXPECT_EQ(svgProperties->svgLocalToBorderBoxTransform(),
+            transformOutsidePathProperties->transform()->parent());
+}
+
 }  // namespace blink