Invalidate first line display item clients when first line style changes

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  *           (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 1826 matching lines @@
         style->inheritFrom(*pseudoStyle);
         setStyle(style.release());
         return;
     }
 
     setStyle(pseudoStyle);
 }
 
 void LayoutObject::firstLineStyleDidChange(const ComputedStyle& oldStyle, const ComputedStyle& newStyle)
 {
-    StyleDifference diff = oldStyle.visualInvalidationDiff(newStyle);
-    if (diff.hasDifference()) {
-        // TODO(rune@opera.com): We should use the diff to determine whether a repaint vs. layout
-        // is needed, but for now just assume a layout will be required. The diff code
-        // in LayoutObject::setStyle would need to be factored out so that it could be reused.
-        setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation(LayoutInvalidationReason::StyleChange);
+    StyleDifference diff = adjustStyleDifference(oldStyle.visualInvalidationDiff(newStyle));

[chrishtr, 2015/11/24 23:21:56]

Is the change in this file strictly necessary?

[Xianzhu, 2015/11/24 23:26:58]

I think yes.

The above line ensures that StyleDifference::textDecorationOrColorChanged etc.
to be translated to needsPaintInvalidation and/or needsLayout.

+
+    if (diff.needsPaintInvalidation()) {
+        setShouldDoFullPaintInvalidation();
+        if (isLayoutBlockFlow())
+            toLayoutBlockFlow(this)->invalidateDisplayItemClientsOfFirstLine();
     }
+    if (diff.needsLayout())
+        setNeedsLayoutAndPrefWidthsRecalc(LayoutInvalidationReason::StyleChange);
 }
 
 void LayoutObject::markContainingBlocksForOverflowRecalc()
 {
     for (LayoutBlock* container = containingBlock(); container && !container->childNeedsOverflowRecalcAfterStyleChange(); container = container->containingBlock())
         container->setChildNeedsOverflowRecalcAfterStyleChange();
 }
 
 void LayoutObject::setNeedsOverflowRecalcAfterStyleChange()
 {
@@ skipping 1640 matching lines @@
         const blink::LayoutObject* root = object1;
         while (root->parent())
             root = root->parent();
         root->showLayoutTreeAndMark(object1, "*", object2, "-", 0);
     } else {
         fprintf(stderr, "Cannot showLayoutTree. Root is (nil)\n");
     }
 }
 
 #endif