Revert of Call Element::rebuildLayoutTree from Document::updateStyle directly

Revert of Call Element::rebuildLayoutTree from Document::updateStyle directly (patchset #33 id:640001 of https://codereview.chromium.org/2473743003/ )

Reason for revert:
Reason for revert is: https://bugs.chromium.org/p/chromium/issues/detail?id=695950


Original issue's description:
> Call Element::rebuildLayoutTree from Document::updateStyle directly
> 
> This patch changes the callstack in Document::updateStyle to call
> Element::rebuildLayoutTree after Element::recalcStyle instead of
> from it.
> 
> It also adds helper functions needed to perform Layout Tree Construction:
> 1. ContainerNode::rebuildDescendantLayoutTree()
> 2. Element::rebuildShadowRootLayoutTree()
> 3. Element::reattachPseudoElementLayoutTree(PseudoId pseudoId)
> 4. ShadowRoot::rebuildLayoutTree()
> 
> It also changes the setting/getting of the (Child)NeedsStyleRecalc
> and (Child)NeedsReattachLayoutTree bits to match this pattern.
> 
> BUG=595137
> 
> Review-Url: https://codereview.chromium.org/2473743003
> Cr-Commit-Position: refs/heads/master@{#451987}
> Committed: https://chromium.googlesource.com/chromium/src/+/8b6c7b7ce5a36d8eda49dd6e60c5e7c8f5c7277f

TBR=esprehn@chromium.org,rune@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=595137
Review-Url: https://codereview.chromium.org/2729453002
Cr-Commit-Position: refs/heads/master@{#453853}
Committed: https://chromium.googlesource.com/chromium/src/+/7ff96ffaec996e51a13ac4b9f026d96f81910348

[nainar, 2017-03-01 03:39:57]

The CQ bit was checked by nainar@chromium.org

[nainar, 2017-03-01 03:39:57]

Created Revert of Call Element::rebuildLayoutTree from Document::updateStyle
directly

[dstockwell, 2017-03-01 03:40:22]

dstockwell@chromium.org changed reviewers:
+ dstockwell@chromium.org

[dstockwell, 2017-03-01 03:40:22]

lgtm

[commit-bot: I haz the power, 2017-03-01 03:40:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-01 05:22:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488339597367830, "parent_rev":
"2e8976ab56dd366d516525fb2ebef0755ebe5b2a", "commit_rev":
"7ff96ffaec996e51a13ac4b9f026d96f81910348"}

[commit-bot: I haz the power, 2017-03-01 05:23:33]

Description was changed from

==========
Revert of Call Element::rebuildLayoutTree from Document::updateStyle directly
(patchset #33 id:640001 of https://codereview.chromium.org/2473743003/ )

Reason for revert:
Reason for revert is:
https://bugs.chromium.org/p/chromium/issues/detail?id=695950


Original issue's description:
> Call Element::rebuildLayoutTree from Document::updateStyle directly
> 
> This patch changes the callstack in Document::updateStyle to call
> Element::rebuildLayoutTree after Element::recalcStyle instead of
> from it.
> 
> It also adds helper functions needed to perform Layout Tree Construction:
> 1. ContainerNode::rebuildDescendantLayoutTree()
> 2. Element::rebuildShadowRootLayoutTree()
> 3. Element::reattachPseudoElementLayoutTree(PseudoId pseudoId)
> 4. ShadowRoot::rebuildLayoutTree()
> 
> It also changes the setting/getting of the (Child)NeedsStyleRecalc
> and (Child)NeedsReattachLayoutTree bits to match this pattern.
> 
> BUG=595137
> 
> Review-Url: https://codereview.chromium.org/2473743003
> Cr-Commit-Position: refs/heads/master@{#451987}
> Committed:
https://chromium.googlesource.com/chromium/src/+/8b6c7b7ce5a36d8eda49dd6e60c5...

TBR=esprehn@chromium.org,rune@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=595137
==========

to

==========
Revert of Call Element::rebuildLayoutTree from Document::updateStyle directly
(patchset #33 id:640001 of https://codereview.chromium.org/2473743003/ )

Reason for revert:
Reason for revert is:
https://bugs.chromium.org/p/chromium/issues/detail?id=695950


Original issue's description:
> Call Element::rebuildLayoutTree from Document::updateStyle directly
> 
> This patch changes the callstack in Document::updateStyle to call
> Element::rebuildLayoutTree after Element::recalcStyle instead of
> from it.
> 
> It also adds helper functions needed to perform Layout Tree Construction:
> 1. ContainerNode::rebuildDescendantLayoutTree()
> 2. Element::rebuildShadowRootLayoutTree()
> 3. Element::reattachPseudoElementLayoutTree(PseudoId pseudoId)
> 4. ShadowRoot::rebuildLayoutTree()
> 
> It also changes the setting/getting of the (Child)NeedsStyleRecalc
> and (Child)NeedsReattachLayoutTree bits to match this pattern.
> 
> BUG=595137
> 
> Review-Url: https://codereview.chromium.org/2473743003
> Cr-Commit-Position: refs/heads/master@{#451987}
> Committed:
https://chromium.googlesource.com/chromium/src/+/8b6c7b7ce5a36d8eda49dd6e60c5...

TBR=esprehn@chromium.org,rune@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=595137

Review-Url: https://codereview.chromium.org/2729453002
Cr-Commit-Position: refs/heads/master@{#453853}
Committed:
https://chromium.googlesource.com/chromium/src/+/7ff96ffaec996e51a13ac4b9f026...
==========

[commit-bot: I haz the power, 2017-03-01 05:23:34]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/7ff96ffaec996e51a13ac4b9f026...