Need to repaint its ::first-line background when a block moves.

third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp

Index: third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp b/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
index 02cf84f9dbd2a1216e737550be00b1e2d3201017..43555eb07f418623c438aaea01165f1d68953aa1 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBlockFlowLine.cpp
@@ -2079,4 +2079,17 @@ void LayoutBlockFlow::invalidateDisplayItemClientsOfFirstLine()
         firstRootBox->invalidateDisplayItemClientsRecursively();
 }
 
+PaintInvalidationReason LayoutBlockFlow::invalidatePaintIfNeeded(PaintInvalidationState& paintInvalidationState, const LayoutBoxModelObject& paintInvalidationContainer)

[Xianzhu, 2015/12/18 17:24:54]

This will miss the LayoutButton case, but I think it's fine to leave this corner
case as-is which will be automatically fixed when we fix crbug.com/226252.

+{
+    PaintInvalidationReason reason = LayoutBlock::invalidatePaintIfNeeded(paintInvalidationState, paintInvalidationContainer);
+    if (reason == PaintInvalidationNone)
+        return reason;
+    RootInlineBox* line = firstRootBox();
+    if (!line || !line->isFirstLineStyle())
+        return reason;
+    // It's the RootInlineBox that paints the ::first-line background.
+    invalidateDisplayItemClient(*line);

[Xianzhu, 2015/12/18 07:01:20]

Will this always invalidate the first line, even if there is no first-line
style?

[mstensho (USE GERRIT), 2015/12/18 10:33:04]

Yeah, looks like it. That wasn't my intention.

[mstensho (USE GERRIT), 2015/12/18 13:11:11]

Fixed it. Not sure if it's worth it, though. Could it generally be more
expensive to figure out that you have no ::first-line that applies, than to just
invalidate it (which would also involve rebaselining a bunch of tests)? What do
you think?

[Xianzhu, 2015/12/18 17:24:54]

Hmm enclosingFirstLineStyleBlock() seems not cheap (especially there is no
first-line style) and it would be tested for all LayoutBlockFlows. Normally a
RootInlineBox's painting is just one drawRect which is cheap. This invalidation
also won't cause more rasterization invalidation. I think we can use this
always-invalidation version.

Patch Set 1 lgtm.

[mstensho (USE GERRIT), 2015/12/21 10:54:10]

Done. Added a comment too.

+    return reason;
+}
+
 }