Take CSS Clip and contain: paint into account when computing visual rects.

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/layout/LayoutTestHelper.h"
 #include "core/layout/LayoutTreeAsText.h"
 #include "core/layout/api/LayoutViewItem.h"
 #include "core/paint/ObjectPaintProperties.h"
 #include "platform/graphics/paint/GeometryMapper.h"
 #include "platform/graphics/paint/TransformPaintPropertyNode.h"
@@ skipping 876 matching lines @@
     LayoutRect absoluteClipRect = localClipRect;
     absoluteClipRect.move(123, 456);
 
     FrameView* frameView = document().view();
 
     LayoutObject* clip = document().getElementById("clip")->layoutObject();
     const ObjectPaintProperties* clipProperties = clip->objectPaintProperties();
     EXPECT_EQ(frameView->contentClip(), clipProperties->cssClip()->parent());
     EXPECT_EQ(frameView->scrollTranslation(), clipProperties->cssClip()->localTransformSpace());
     EXPECT_EQ(FloatRoundedRect(FloatRect(absoluteClipRect)), clipProperties->cssClip()->clipRect());
-    // TODO(chrishtr): GeomeryMapper rports 40x60 size (i.e. including clip), whereas the old code reports 100x100 (unclipped).
-    // Fix.
-    // CHECK_VISUAL_RECT(clip, frameView->layoutView());
+// TODO(chrishtr): the old visual rect code is not able to apply CSS clip to fixed-position elements.
+//    CHECK_VISUAL_RECT(clip, frameView->layoutView());
 
     LayoutObject* fixed = document().getElementById("fixed")->layoutObject();
     const ObjectPaintProperties* fixedProperties = fixed->objectPaintProperties();
     EXPECT_EQ(clipProperties->cssClip(), fixedProperties->localBorderBoxProperties()->propertyTreeState.clip);
     EXPECT_EQ(frameView->preTranslation(), fixedProperties->localBorderBoxProperties()->propertyTreeState.transform->parent());
     EXPECT_EQ(TransformationMatrix().translate(654, 321), fixedProperties->localBorderBoxProperties()->propertyTreeState.transform->matrix());
     EXPECT_EQ(LayoutPoint(), fixedProperties->localBorderBoxProperties()->paintOffset);
     CHECK_VISUAL_RECT(fixed, frameView->layoutView());
 }
 
+TEST_F(PaintPropertyTreeBuilderTest, CSSClipAbsPositionDescendant)
+{
+    // This test verifies that clip tree hierarchy being generated correctly for the hard case
+    // such that a fixed position element getting clipped by an absolute position CSS clip.
+    setBodyInnerHTML(
+        "<style>"
+        "  #clip {"
+        "    position: absolute;"
+        "    left: 123px;"
+        "    top: 456px;"
+        "    clip: rect(10px, 80px, 70px, 40px);"
+        "    width: 100px;"
+        "    height: 100px;"
+        "  }"
+        "  #abs {"
+        "    position: absolute;"
+        "    left: 654px;"
+        "    top: 321px;"
+        "    width: 10px;"
+        "    heght: 20px"
+        "  }"
+        "</style>"
+        "<div id='clip'><div id='absolute'></div></div>"
+    );
+    LayoutRect localClipRect(40, 10, 40, 60);
+    LayoutRect absoluteClipRect = localClipRect;
+    absoluteClipRect.move(123, 456);
+
+    FrameView* frameView = document().view();
+
+    LayoutObject* clip = document().getElementById("clip")->layoutObject();
+    const ObjectPaintProperties* clipProperties = clip->objectPaintProperties();
+    EXPECT_EQ(frameView->contentClip(), clipProperties->cssClip()->parent());
+    EXPECT_EQ(frameView->scrollTranslation(), clipProperties->cssClip()->localTransformSpace());
+    EXPECT_EQ(FloatRoundedRect(FloatRect(absoluteClipRect)), clipProperties->cssClip()->clipRect());
+// TODO(chrishtr): the old visual rect code is not able to apply CSS clip to fixed-position elements.
+//    CHECK_VISUAL_RECT(clip, frameView->layoutView());
+
+    LayoutObject* absolute = document().getElementById("absolute")->layoutObject();
+    const ObjectPaintProperties* absPosProperties = absolute->objectPaintProperties();
+    EXPECT_EQ(clipProperties->cssClip(), absPosProperties->localBorderBoxProperties()->propertyTreeState.clip);
+    EXPECT_EQ(frameView->preTranslation(), absPosProperties->localBorderBoxProperties()->propertyTreeState.transform->parent());
+    EXPECT_EQ(LayoutPoint(123, 456), absPosProperties->localBorderBoxProperties()->paintOffset);
+    CHECK_VISUAL_RECT(absolute, frameView->layoutView());
+}
+
 TEST_F(PaintPropertyTreeBuilderTest, CSSClipFixedPositionDescendantNonShared)
 {
     // This test is similar to CSSClipFixedPositionDescendant above, except that
     // now we have a parent overflow clip that should be escaped by the fixed descendant.
     setBodyInnerHTML(
         "<style>"
         "  body {"
         "    margin: 0;"
         "  }"
         "  #overflow {"
@@ skipping 557 matching lines @@
     EXPECT_EQ(cProperties, c->layoutObject()->objectPaintProperties());
     EXPECT_EQ(cTransformNode, cProperties->transform());
     EXPECT_EQ(bTransformNode, cTransformNode->parent());
 
     CHECK_VISUAL_RECT(a->layoutObject(), frameView->layoutView());
     CHECK_VISUAL_RECT(b->layoutObject(), frameView->layoutView());
     CHECK_VISUAL_RECT(c->layoutObject(), frameView->layoutView());
 }
 
 } // namespace blink