Invalidate first line display item clients when first line style changes

third_party/WebKit/Source/core/layout/LayoutObject.h

Index: third_party/WebKit/Source/core/layout/LayoutObject.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutObject.h b/third_party/WebKit/Source/core/layout/LayoutObject.h
index 452d1633e847ccc7eaea35cea054db101f70c4d0..e92b4631f7517012aa6cf7f05fa76bb5d0486512 100644
--- a/third_party/WebKit/Source/core/layout/LayoutObject.h
+++ b/third_party/WebKit/Source/core/layout/LayoutObject.h
@@ -341,10 +341,6 @@ public:
         return hasBoxDecorationBackground() || style()->hasVisualOverflowingEffect();
     }
 
-    // Obtains the nearest enclosing block (including this block) that contributes a first-line style to our inline
-    // children.
-    virtual LayoutBlock* firstLineBlock() const;
-
     // LayoutObject tree manipulation
     //////////////////////////////////////////
     virtual bool canHaveChildren() const { return virtualChildren(); }
@@ -750,6 +746,14 @@ public:
     virtual LayoutMultiColumnSpannerPlaceholder* spannerPlaceholder() const { return nullptr; }
     bool isColumnSpanAll() const { return style()->columnSpan() == ColumnSpanAll && spannerPlaceholder(); }
 
+    // We include isLayoutButton in this check because buttons are implemented
+    // using flex box but should still support first-line|first-letter.
+    // The flex box and grid specs require that flex box and grid do not
+    // support first-line|first-letter, though.
+    // TODO(layout-team): Remove when buttons are implemented with align-items instead

[mstensho (USE GERRIT), 2015/12/03 12:08:10]

A reference to crbug.com/226252 here? Not sure if "layout-team" is right here.
How about TODO(cbiesinger)? The FIXME was added by him in
b981273f245e746020a7afa4580e2a85580c4e32

[Xianzhu, 2015/12/03 17:38:22]

Done.

+    // of flex box.
+    bool canHaveFirstLineOrFirstLetterStyle() const { return isLayoutBlockFlow() || isLayoutButton(); }
+
     // This function returns the containing block of the object.
     // Due to CSS being inconsistent, a containing block can be a relatively
     // positioned inline, thus we can't return a LayoutBlock from this function.