Force subtree paint property updates on local border box changes

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/html/HTMLIFrameElement.h"
 #include "core/layout/LayoutTestHelper.h"
 #include "core/layout/LayoutTreeAsText.h"
 #include "core/layout/api/LayoutViewItem.h"
 #include "core/paint/ObjectPaintProperties.h"
 #include "core/paint/PaintPropertyTreePrinter.h"
@@ skipping 3213 matching lines @@
 TEST_P(PaintPropertyTreeBuilderTest, NoPaintPropertyUpdateOnBackgroundChange) {
   setBodyInnerHTML("<div id='div' style='background-color: blue'>DIV</div>");
   auto* div = document().getElementById("div");
 
   document().view()->updateAllLifecyclePhases();
   div->setAttribute(HTMLNames::styleAttr, "background-color: green");
   document().view()->updateLifecycleToLayoutClean();
   EXPECT_FALSE(div->layoutObject()->needsPaintPropertyUpdate());
 }
 
+TEST_P(PaintPropertyTreeBuilderTest, AbsoluteLocalBorderBoxChanges) {
+  setBodyInnerHTML(
+      "<style>"
+      "  body { margin: 0 }"
+      "  #scroller { width: 3px; height: 5px; overflow: scroll; }"
+      "  #child { top: 0; left: 0; }"
+      "</style>"
+      "<div id='scroller'>"
+      "  <div id='child'>"
+      "    <div id='grandChild'></div>"
+      "  </div>"
+      "  <div style='height: 100px;'></div>"
+      "</div>");
+  document().view()->updateAllLifecyclePhases();
+  auto* scroller = document().getElementById("scroller");
+  auto* scrollerProperties = scroller->layoutObject()->paintProperties();
+  auto* child = document().getElementById("child");
+  auto* childProperties = child->layoutObject()->paintProperties();
+  auto* childLocalBorderBox = childProperties->localBorderBoxProperties();
+  EXPECT_EQ(childLocalBorderBox->propertyTreeState.scroll(),
+            scrollerProperties->scroll());
+  EXPECT_EQ(LayoutPoint(0, 0), childLocalBorderBox->paintOffset);
+  auto* grandChild = document().getElementById("grandChild");
+  auto* grandChildProperties = grandChild->layoutObject()->paintProperties();
+  auto* grandChildLocalBorderBox =
+      grandChildProperties->localBorderBoxProperties();
+  EXPECT_EQ(LayoutPoint(0, 0), grandChildLocalBorderBox->paintOffset);
+  EXPECT_EQ(
+      FloatRect(0, 0, 3, 5),
+      grandChildLocalBorderBox->propertyTreeState.clip()->clipRect().rect());
+
+  // Update the scroller's transform value but ensure the child's paint offset
+  // is unchanged so subtree updates are not forced by paint offset changes.
+  scroller->setScrollTop(7);
+  child->setAttribute(HTMLNames::styleAttr, "position: absolute");
+  document().view()->updateAllLifecyclePhases();
+  EXPECT_EQ(LayoutPoint(0, 0), childLocalBorderBox->paintOffset);
+  EXPECT_EQ(LayoutPoint(0, 0), grandChildLocalBorderBox->paintOffset);
+  EXPECT_NE(childLocalBorderBox->propertyTreeState.scroll(),
+            scrollerProperties->scroll());
+  EXPECT_TRUE(childLocalBorderBox->propertyTreeState.scroll()->isRoot());
+  EXPECT_EQ(
+      FloatRect(0, 0, 800, 600),
+      grandChildLocalBorderBox->propertyTreeState.clip()->clipRect().rect());
+}
+
+TEST_P(PaintPropertyTreeBuilderTest, FixedLocalBorderBoxChanges) {
+  setBodyInnerHTML(
+      "<style>"
+      "  body { margin: 0 }"
+      "  #scroller { width: 3px; height: 5px; overflow: scroll; }"
+      "  #child { top: 0; left: 0; }"
+      "</style>"
+      "<div id='scroller'>"
+      "  <div id='child'>"
+      "    <div id='grandChild'></div>"
+      "  </div>"
+      "  <div style='height: 100px;'></div>"
+      "</div>");
+  document().view()->updateAllLifecyclePhases();
+  auto* scroller = document().getElementById("scroller");
+  auto* scrollerProperties = scroller->layoutObject()->paintProperties();
+  auto* child = document().getElementById("child");
+  auto* childProperties = child->layoutObject()->paintProperties();
+  auto* childLocalBorderBox = childProperties->localBorderBoxProperties();
+  EXPECT_EQ(childLocalBorderBox->propertyTreeState.scroll(),
+            scrollerProperties->scroll());
+  EXPECT_EQ(LayoutPoint(0, 0), childLocalBorderBox->paintOffset);
+  auto* grandChild = document().getElementById("grandChild");
+  auto* grandChildProperties = grandChild->layoutObject()->paintProperties();
+  auto* grandChildLocalBorderBox =
+      grandChildProperties->localBorderBoxProperties();
+  EXPECT_EQ(LayoutPoint(0, 0), grandChildLocalBorderBox->paintOffset);
+  EXPECT_EQ(
+      FloatRect(0, 0, 3, 5),
+      grandChildLocalBorderBox->propertyTreeState.clip()->clipRect().rect());
+
+  // Update the scroller's transform value but ensure the child's paint offset
+  // is unchanged so subtree updates are not forced by paint offset changes.
+  scroller->setScrollTop(7);
+  child->setAttribute(HTMLNames::styleAttr, "position: fixed");
+  document().view()->updateAllLifecyclePhases();
+  EXPECT_EQ(LayoutPoint(0, 0), childLocalBorderBox->paintOffset);
+  EXPECT_EQ(LayoutPoint(0, 0), grandChildLocalBorderBox->paintOffset);
+  EXPECT_NE(childLocalBorderBox->propertyTreeState.scroll(),
+            scrollerProperties->scroll());
+  EXPECT_TRUE(childLocalBorderBox->propertyTreeState.scroll()->isRoot());
+  EXPECT_EQ(
+      FloatRect(0, 0, 800, 600),
+      grandChildLocalBorderBox->propertyTreeState.clip()->clipRect().rect());
+}
+
 }  // namespace blink