Clear pseudoElement from ElementRareData when FirstLetterPseudoElement doesn't have a LayoutObject

third_party/WebKit/Source/core/dom/Element.cpp

Index: third_party/WebKit/Source/core/dom/Element.cpp
diff --git a/third_party/WebKit/Source/core/dom/Element.cpp b/third_party/WebKit/Source/core/dom/Element.cpp
index cde1c106f18ac2c7486d7dc8bb4f209158ab2c4d..289bda48179fd9e5245de153c5f91aa143aa1404 100644
--- a/third_party/WebKit/Source/core/dom/Element.cpp
+++ b/third_party/WebKit/Source/core/dom/Element.cpp
@@ -2062,8 +2062,11 @@ void Element::rebuildShadowRootLayoutTree() {
 
 void Element::reattachPseudoElementLayoutTree(PseudoId pseudoId) {
   if (PseudoElement* element = pseudoElement(pseudoId)) {
-    if (element->needsReattachLayoutTree() ||

[nainar, 2017/03/17 23:47:40]

Change as per discussed by esprehn@ and I in IRL: 

In the case of FirstLetter we need to mirror Element::updateFirstLetter(). We
have to delete the old first letter here to set the original text back on the
text layoutObject. If we dispose the old first letter after creating the new one
we will get incorrect results because we set the first letter back.

In the codepath before I landed codereview.chromium.org/2473743003 we would have
called updateFirstLetter but now we aren't. Calling reattachLayoutTree
unconditionally leads to bad state hence the call not having the correct
layoutObject information. This causes us to hit the DCHECK seen in the bug. 

Logging here for the record as I am OOO and will probably forget.

-        element->childNeedsReattachLayoutTree())
+    if (pseudoId == PseudoIdFirstLetter &&
+        !FirstLetterPseudoElement::firstLetterTextLayoutObject(*element))
+      elementRareData()->setPseudoElement(pseudoId, nullptr);
+    else if (element->needsReattachLayoutTree() ||

[esprehn, 2017/04/06 00:08:59]

I think we often add braces when things start to get cramped like this.

[nainar, 2017/04/06 00:20:45]

Done.

+             element->childNeedsReattachLayoutTree())
       element->rebuildLayoutTree();
   } else {
     createPseudoElementIfNeeded(pseudoId);