Add flag for tracking descendant paint property updates

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

Index: third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.cpp
diff --git a/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.cpp b/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.cpp
index ca46d9b7ae0f0643ff5ef6bdb0ac29013deb050d..3ec2f9a97e57666fa8ed9691e1d2de7a70dad87f 100644
--- a/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.cpp
+++ b/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.cpp
@@ -2615,6 +2615,7 @@ TEST_P(PaintPropertyTreeBuilderTest,
   // TODO(pdr): The main thread scrolling setting should invalidate properties.
   document().view()->setNeedsPaintPropertyUpdate();
   overflowA->layoutObject()->setNeedsPaintPropertyUpdate();
+  overflowA->layoutObject()->setAllAncestorsNeedPaintPropertyUpdate();

[Xianzhu, 2016/11/21 19:41:34]

Isn't the above call redundant?

[pdr., 2016/11/22 02:55:35]

This is not needed yet so I have removed it from this patch. It will be needed
in a future patch that prunes the prepaint tree walk though, because this test
will fail (see comment above for details).

   document().view()->updateAllLifecyclePhases();
 
   EXPECT_TRUE(frameScroll()->threadedScrollingDisabled());
@@ -2889,4 +2890,46 @@ TEST_P(PaintPropertyTreeBuilderTest, FilterReparentClips) {
   EXPECT_EQ(filterProperties->effect(), childPaintState.effect());
 }
 
+TEST_P(PaintPropertyTreeBuilderTest, DescendantNeedsUpdateAcrossFrames) {
+  setBodyInnerHTML(
+      "<style>body { margin: 0; }</style>"
+      "<div id='divWithTransform' style='transform: translate3d(1px,2px,3px);'>"
+      "  <iframe style='border: 7px solid black'></iframe>"
+      "</div>");
+  setChildFrameHTML(
+      "<style>body { margin: 0; }</style><div id='transform' style='transform: "
+      "translate3d(4px, 5px, 6px); width: 100px; height: 200px'></div>");
+
+  FrameView* frameView = document().view();
+  frameView->updateAllLifecyclePhases();
+
+  LayoutObject* divWithTransform =
+      document().getElementById("divWithTransform")->layoutObject();
+  LayoutObject* childLayoutView = childDocument().layoutView();
+  LayoutObject* innerDivWithTransform =
+      childDocument().getElementById("transform")->layoutObject();
+
+  // Initially, no objects should need a descendant update.
+  EXPECT_FALSE(document().layoutView()->descendantNeedsPaintPropertyUpdate());
+  EXPECT_FALSE(divWithTransform->descendantNeedsPaintPropertyUpdate());
+  EXPECT_FALSE(childLayoutView->descendantNeedsPaintPropertyUpdate());
+  EXPECT_FALSE(innerDivWithTransform->descendantNeedsPaintPropertyUpdate());
+
+  // Marking the child div as needing a paint property update should propagate
+  // up the tree and across frames.
+  innerDivWithTransform->setNeedsPaintPropertyUpdate();
+  EXPECT_TRUE(document().layoutView()->descendantNeedsPaintPropertyUpdate());
+  EXPECT_TRUE(divWithTransform->descendantNeedsPaintPropertyUpdate());
+  EXPECT_TRUE(childLayoutView->descendantNeedsPaintPropertyUpdate());
+  EXPECT_TRUE(innerDivWithTransform->needsPaintPropertyUpdate());
+  EXPECT_FALSE(innerDivWithTransform->descendantNeedsPaintPropertyUpdate());
+
+  // After a lifecycle update, no nodes should need a descendant update.
+  frameView->updateAllLifecyclePhases();
+  EXPECT_FALSE(document().layoutView()->descendantNeedsPaintPropertyUpdate());
+  EXPECT_FALSE(divWithTransform->descendantNeedsPaintPropertyUpdate());
+  EXPECT_FALSE(childLayoutView->descendantNeedsPaintPropertyUpdate());
+  EXPECT_FALSE(innerDivWithTransform->descendantNeedsPaintPropertyUpdate());
+}
+
 }  // namespace blink