SVG root viewport clip in paint property tree

third_party/WebKit/Source/core/layout/svg/LayoutSVGRootTest.cpp

 // Copyright 2016 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/layout/svg/LayoutSVGRoot.h"
 
 #include "core/layout/LayoutTestHelper.h"
 #include "core/layout/svg/LayoutSVGShape.h"
 #include "core/layout/svg/SVGLayoutSupport.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 30 matching lines @@
     setBodyInnerHTML(
         "<svg id='root' style='border: 10px solid red; width: 200px; height: 100px; overflow: hidden' viewBox='0 0 200 100'>"
         "   <rect id='rect' x='80' y='80' width='100' height='100'/>"
         "</svg>");
 
     const LayoutSVGRoot* root = toLayoutSVGRoot(getLayoutObjectByElementId("root"));
     const LayoutSVGShape* svgRect = toLayoutSVGShape(getLayoutObjectByElementId("rect"));
 
     LayoutRect rect = SVGLayoutSupport::clippedOverflowRectForPaintInvalidation(*svgRect, *root);
     // (80, 80, 100, 100) added by root's content rect offset from border rect, clipped by (10, 10, 200, 100).
-    EXPECT_EQ(LayoutRect(90, 90, 100, 30), rect);
+    EXPECT_EQ(LayoutRect(90, 90, 100, 20), rect);

[Xianzhu, 2016/09/14 21:36:00]

This is also because of the change in SVGLayoutSupport.cpp.

 
     LayoutRect rootOverflowRect = static_cast<const LayoutObject*>(root)->localOverflowRectForPaintInvalidation();
     // SVG root with overflow:hidden doesn't include overflow from children, just border box rect.
     EXPECT_EQ(LayoutRect(0, 0, 220, 120), rootOverflowRect);
 
     rect = rootOverflowRect;
     EXPECT_TRUE(root->mapToVisualRectInAncestorSpace(root, rect));
     // LayoutSVGRoot should not apply overflow clip on its own rect.
     EXPECT_EQ(LayoutRect(0, 0, 220, 120), rect);
 }
@@ skipping 15 matching lines @@
     LayoutRect rootOverflowRect = static_cast<const LayoutObject*>(root)->localOverflowRectForPaintInvalidation();
     // SVG root doesn't have box decoration background, so just use clipped overflow of children.
     EXPECT_EQ(LayoutRect(80, 80, 100, 20), rootOverflowRect);
 
     rect = rootOverflowRect;
     EXPECT_TRUE(root->mapToVisualRectInAncestorSpace(root, rect));
     EXPECT_EQ(LayoutRect(80, 80, 100, 20), rect);
 }
 
 } // namespace blink