Fix traverseNonCompositingDescendantsInPaintOrder for float-under-inline cases

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

Index: third_party/WebKit/Source/core/paint/ObjectPaintInvalidatorTest.cpp
diff --git a/third_party/WebKit/Source/core/paint/ObjectPaintInvalidatorTest.cpp b/third_party/WebKit/Source/core/paint/ObjectPaintInvalidatorTest.cpp
index 08c615c314e75e7323c2130c1dcf20e67e5d9cc4..64b9242819a4721aeb7b88e046ae4de9f20eeb0e 100644
--- a/third_party/WebKit/Source/core/paint/ObjectPaintInvalidatorTest.cpp
+++ b/third_party/WebKit/Source/core/paint/ObjectPaintInvalidatorTest.cpp
@@ -6,6 +6,7 @@
 
 #include "core/layout/LayoutObject.h"
 #include "core/layout/LayoutTestHelper.h"
+#include "core/paint/PaintLayer.h"
 #include "platform/json/JSONValues.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ -64,4 +65,158 @@ TEST_F(ObjectPaintInvalidatorTest,
             s);
 }
 
+TEST_F(ObjectPaintInvalidatorTest, TraverseFloatUnderCompositedInline) {
+  if (RuntimeEnabledFeatures::slimmingPaintV2Enabled())
+    return;
+
+  enableCompositing();
+  setBodyInnerHTML(
+      "<div id='compositedContainer' style='position: relative;"
+      "    will-change: transform'>"
+      "  <div id='containingBlock' style='position: relative; z-index: 0'>"
+      "    <div style='backface-visibility: hidden'></div>"
+      "    <span id='span'"
+      "        style='clip-path: polygon(0px 15px, 0px 54px, 100px 0px)'>"
+      "      <div id='target' style='float: right'></div>"
+      "    </span>"
+      "  </div>"
+      "</div>");
+
+  auto* target = getLayoutObjectByElementId("target");
+  auto* containingBlock = getLayoutObjectByElementId("containingBlock");
+  auto* containingBlockLayer = toLayoutBoxModelObject(containingBlock)->layer();
+  auto* compositedContainer = getLayoutObjectByElementId("compositedContainer");
+  auto* compositedContainerLayer =
+      toLayoutBoxModelObject(compositedContainer)->layer();
+  auto* span = getLayoutObjectByElementId("span");
+
+  // Thought |target| is under |span| which is a composited stacking context,
+  // |span| is not the paint invalidation container of |target|.
+  EXPECT_TRUE(span->isPaintInvalidationContainer());
+  EXPECT_TRUE(span->styleRef().isStackingContext());
+  EXPECT_EQ(compositedContainer, &target->containerForPaintInvalidation());
+  EXPECT_EQ(containingBlockLayer, target->paintingLayer());
+
+  // Traversing from target should mark needsRepaint on correct layers.
+  EXPECT_FALSE(containingBlockLayer->needsRepaint());
+  EXPECT_FALSE(compositedContainerLayer->needsRepaint());
+  ObjectPaintInvalidator(*target)
+      .invalidateDisplayItemClientsIncludingNonCompositingDescendants(
+          PaintInvalidationSubtree);
+  EXPECT_TRUE(containingBlockLayer->needsRepaint());
+  EXPECT_TRUE(compositedContainerLayer->needsRepaint());
+
+  document().view()->updateAllLifecyclePhases();
+
+  // Traversing from span should mark needsRepaint on correct layers for target.
+  EXPECT_FALSE(containingBlockLayer->needsRepaint());
+  EXPECT_FALSE(compositedContainerLayer->needsRepaint());
+  ObjectPaintInvalidator(*span)
+      .invalidateDisplayItemClientsIncludingNonCompositingDescendants(
+          PaintInvalidationSubtree);
+  EXPECT_TRUE(containingBlockLayer->needsRepaint());
+  EXPECT_TRUE(compositedContainerLayer->needsRepaint());
+
+  document().view()->updateAllLifecyclePhases();
+
+  // Traversing from compositedContainer should reach target.
+  document().view()->setTracksPaintInvalidations(true);
+  EXPECT_FALSE(containingBlockLayer->needsRepaint());
+  EXPECT_FALSE(compositedContainerLayer->needsRepaint());
+  ObjectPaintInvalidator(*compositedContainer)
+      .invalidateDisplayItemClientsIncludingNonCompositingDescendants(
+          PaintInvalidationSubtree);
+  EXPECT_TRUE(containingBlockLayer->needsRepaint());
+  EXPECT_TRUE(compositedContainerLayer->needsRepaint());
+
+  std::unique_ptr<JSONArray> invalidations =
+      document().view()->trackedObjectPaintInvalidationsAsJSON();
+  document().view()->setTracksPaintInvalidations(false);
+
+  ASSERT_EQ(5u, invalidations->size());
+  String s;
+  JSONObject::cast(invalidations->at(0))->get("object")->asString(&s);
+  EXPECT_EQ(compositedContainer->debugName(), s);
+  JSONObject::cast(invalidations->at(1))->get("object")->asString(&s);
+  EXPECT_EQ(containingBlock->debugName(), s);
+  // This is the anonymous block enclosing the span.
+  JSONObject::cast(invalidations->at(2))->get("object")->asString(&s);
+  EXPECT_EQ("LayoutBlockFlow (anonymous)", s);
+  JSONObject::cast(invalidations->at(3))->get("object")->asString(&s);
+  EXPECT_EQ(target->debugName(), s);
+  // This is the text node after the span.
+  JSONObject::cast(invalidations->at(4))->get("object")->asString(&s);
+  EXPECT_EQ("LayoutText #text", s);
+}
+
+TEST_F(ObjectPaintInvalidatorTest,
+       TraverseFloatUnderMultiLevelCompositedInlines) {
+  if (RuntimeEnabledFeatures::slimmingPaintV2Enabled())
+    return;
+
+  enableCompositing();
+  setBodyInnerHTML(
+      "<div id='compositedContainer' style='position: relative;"
+      "    will-change: transform'>"
+      "  <div id='containingBlock' style='position: relative; z-index: 0'>"
+      "    <div style='backface-visibility: hidden'></div>"
+      "    <span id='span'"
+      "        style='clip-path: polygon(0px 15px, 0px 54px, 100px 0px)'>"
+      "      <div style='display: inline-block'>"
+      "        <div style='backface-visibility: hidden'></div>"
+      "        <span id='innerSpan'"
+      "            style='clip-path: polygon(0px 15px, 0px 54px, 100px 0px)'>"
+      "          <div id='target' style='float: right'></div>"
+      "        </span>"
+      "      </div>"
+      "    </span>"
+      "  </div>"
+      "</div>");
+
+  auto* target = getLayoutObjectByElementId("target");
+  auto* containingBlock = getLayoutObjectByElementId("containingBlock");
+  auto* containingBlockLayer = toLayoutBoxModelObject(containingBlock)->layer();
+  auto* compositedContainer = getLayoutObjectByElementId("compositedContainer");
+  auto* compositedContainerLayer =
+      toLayoutBoxModelObject(compositedContainer)->layer();
+  auto* span = getLayoutObjectByElementId("span");
+  auto* innerSpan = getLayoutObjectByElementId("innerSpan");
+
+  EXPECT_TRUE(span->isPaintInvalidationContainer());
+  EXPECT_TRUE(span->styleRef().isStackingContext());
+  EXPECT_TRUE(innerSpan->isPaintInvalidationContainer());
+  EXPECT_TRUE(innerSpan->styleRef().isStackingContext());
+
+  // Traversing from compositedContainer should reach target.
+  document().view()->setTracksPaintInvalidations(true);
+  EXPECT_FALSE(containingBlockLayer->needsRepaint());
+  EXPECT_FALSE(compositedContainerLayer->needsRepaint());
+  ObjectPaintInvalidator(*compositedContainer)
+      .invalidateDisplayItemClientsIncludingNonCompositingDescendants(
+          PaintInvalidationSubtree);
+  EXPECT_TRUE(containingBlockLayer->needsRepaint());
+  EXPECT_TRUE(compositedContainerLayer->needsRepaint());
+
+  std::unique_ptr<JSONArray> invalidations =
+      document().view()->trackedObjectPaintInvalidationsAsJSON();
+  document().view()->setTracksPaintInvalidations(false);
+
+  ASSERT_EQ(6u, invalidations->size());
+  String s;
+  JSONObject::cast(invalidations->at(0))->get("object")->asString(&s);
+  EXPECT_EQ(compositedContainer->debugName(), s);
+  JSONObject::cast(invalidations->at(1))->get("object")->asString(&s);
+  EXPECT_EQ(containingBlock->debugName(), s);
+  // This is the anonymous block enclosing the span.
+  JSONObject::cast(invalidations->at(2))->get("object")->asString(&s);
+  EXPECT_EQ("LayoutBlockFlow (anonymous)", s);
+  JSONObject::cast(invalidations->at(3))->get("object")->asString(&s);
+  EXPECT_EQ("RootInlineBox", s);
+  JSONObject::cast(invalidations->at(4))->get("object")->asString(&s);
+  EXPECT_EQ(target->debugName(), s);
+  // This is the text node after the span.
+  JSONObject::cast(invalidations->at(5))->get("object")->asString(&s);
+  EXPECT_EQ("LayoutText #text", s);
+}
+
 }  // namespace blink