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

Revert of Clear pseudoElement from ElementRareData when FirstLetterPseudoElement doesn't have a LayoutObject (patchset #4 id:60001 of https://codereview.chromium.org/2752163003/ )

Reason for revert:
The added test is failing on many platforms due to a mismatch between actual expected (not a crash).

BUG=708967
Original issue's description:
> Clear pseudoElement from ElementRareData when FirstLetterPseudoElement doesn't have a LayoutObject
> 
> This patch edits Element::rebuildPseudoElementLayoutTree() to make sure 
> it mirrors Element::updatePseudoElement() called in 
> Element::recaclStyle() by calling Element::updateFirstLetter() in the 
> case the PseudoElement is firstLetter as is done in 
> Element::updateFirstLetter() to ensure we don't call 
> Element::retachLayoutTree() uncoditionally.
> 
> Adds crash test and expectation as well.
> 
> BUG=700655
> 
> Review-Url: https://codereview.chromium.org/2752163003
> Cr-Commit-Position: refs/heads/master@{#462373}
> Committed: https://chromium.googlesource.com/chromium/src/+/6d202487ff823def4672599f51e9f13880e6b877

TBR=esprehn@chromium.org,nainar@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=700655
Review-Url: https://codereview.chromium.org/2802993002
Cr-Commit-Position: refs/heads/master@{#462413}
Committed: https://chromium.googlesource.com/chromium/src/+/7773939ef302371795ef39138e10225f3d493aa1

[foolip, 2017-04-06 10:17:48]

The CQ bit was checked by foolip@chromium.org

[foolip, 2017-04-06 10:17:48]

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

[commit-bot: I haz the power, 2017-04-06 10:19:40]

Description was changed from

==========
Revert of Clear pseudoElement from ElementRareData when FirstLetterPseudoElement
doesn't have a LayoutObject (patchset #4 id:60001 of
https://codereview.chromium.org/2752163003/ )

Reason for revert:
The added test is failing on many platforms due to a mismatch between actual
expected (not a crash).

BUG=708967

Original issue's description:
> Clear pseudoElement from ElementRareData when FirstLetterPseudoElement doesn't
have a LayoutObject
> 
> This patch edits Element::rebuildPseudoElementLayoutTree() to make sure 
> it mirrors Element::updatePseudoElement() called in 
> Element::recaclStyle() by calling Element::updateFirstLetter() in the 
> case the PseudoElement is firstLetter as is done in 
> Element::updateFirstLetter() to ensure we don't call 
> Element::retachLayoutTree() uncoditionally.
> 
> Adds crash test and expectation as well.
> 
> BUG=700655
> 
> Review-Url: https://codereview.chromium.org/2752163003
> Cr-Commit-Position: refs/heads/master@{#462373}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6d202487ff823def4672599f51e9...

TBR=esprehn@chromium.org,nainar@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=700655
==========

to

==========
Revert of Clear pseudoElement from ElementRareData when FirstLetterPseudoElement
doesn't have a LayoutObject (patchset #4 id:60001 of
https://codereview.chromium.org/2752163003/ )

Reason for revert:
The added test is failing on many platforms due to a mismatch between actual
expected (not a crash).

BUG=708967

Original issue's description:
> Clear pseudoElement from ElementRareData when FirstLetterPseudoElement doesn't
have a LayoutObject
> 
> This patch edits Element::rebuildPseudoElementLayoutTree() to make sure 
> it mirrors Element::updatePseudoElement() called in 
> Element::recaclStyle() by calling Element::updateFirstLetter() in the 
> case the PseudoElement is firstLetter as is done in 
> Element::updateFirstLetter() to ensure we don't call 
> Element::retachLayoutTree() uncoditionally.
> 
> Adds crash test and expectation as well.
> 
> BUG=700655
> 
> Review-Url: https://codereview.chromium.org/2752163003
> Cr-Commit-Position: refs/heads/master@{#462373}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6d202487ff823def4672599f51e9...

TBR=esprehn@chromium.org,nainar@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=700655

Review-Url: https://codereview.chromium.org/2802993002
Cr-Commit-Position: refs/heads/master@{#462413}
Committed:
https://chromium.googlesource.com/chromium/src/+/7773939ef302371795ef39138e10...
==========