Revert of Move Layout Tree Construction code into Element::rebuildLayoutTree()

Revert of Move Layout Tree Construction code into Element::rebuildLayoutTree() (patchset #12 id:240001 of https://codereview.chromium.org/2398293003/ )

Reason for revert:
Suspected to cause crbug.com/658322.

Original issue's description:
> Move Layout Tree Construction code into Element::rebuildLayoutTree()
>
> This patch uses the two dirty bits (NeedsReattachLayoutTree and
> ChildNeedsReattachLayoutTree) on Node and adds the relevant
> getters/setters for them.
>
> It makes rebuildLayoutTree() public to be accessed in Document.cpp.
>
> It also moves reattachWhitespaceSiblingsIfNeeded() to rebuildLayoutTree() and
> uses dirtyBits to call either reattachLayoutTree() on node itself or call
> rebuildLayoutTree() on children nodes.
>
> Also added some comments explaining some design decisions. Will be removed
> upon completion of separation.
>
> BUG=595137
>
> Committed: https://crrev.com/09b7a8cf0647abca732b3718987a629d12a67fee
> Cr-Commit-Position: refs/heads/master@{#426353}

TBR=esprehn@chromium.org,bugsnash@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=595137
Committed: https://crrev.com/352b045421e6e2ceeba083eeebd27f64adf41194
Cr-Commit-Position: refs/heads/master@{#426991}

[nainar, 2016-10-22 06:14:16]

The CQ bit was checked by nainar@chromium.org

[nainar, 2016-10-22 06:14:17]

Created Revert of Move Layout Tree Construction code into
Element::rebuildLayoutTree()

[commit-bot: I haz the power, 2016-10-22 06:14:27]

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, 2016-10-22 06:16:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-22 06:16:57]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[nainar, 2016-10-22 08:22:03]

The CQ bit was checked by nainar@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-22 08:22:11]

Dry run: 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, 2016-10-22 08:24:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-22 08:24:41]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[nainar, 2016-10-22 08:46:10]

The CQ bit was checked by nainar@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-22 08:46:26]

Dry run: 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, 2016-10-22 10:08:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-22 10:08:13]

Dry run: This issue passed the CQ dry run.

[nainar, 2016-10-22 12:58:16]

The CQ bit was checked by nainar@chromium.org

[commit-bot: I haz the power, 2016-10-22 12:58:20]

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, 2016-10-22 13:07:14]

Committed patchset #2 (id:200001)

[commit-bot: I haz the power, 2016-10-22 13:09:22]

Description was changed from

==========
Revert of Move Layout Tree Construction code into Element::rebuildLayoutTree()
(patchset #12 id:240001 of https://codereview.chromium.org/2398293003/ )

Reason for revert:
Suspected to cause crbug.com/658322.

Original issue's description:
> Move Layout Tree Construction code into Element::rebuildLayoutTree()
> 
> This patch uses the two dirty bits (NeedsReattachLayoutTree and
> ChildNeedsReattachLayoutTree) on Node and adds the relevant
> getters/setters for them.
> 
> It makes rebuildLayoutTree() public to be accessed in Document.cpp.
> 
> It also moves reattachWhitespaceSiblingsIfNeeded() to rebuildLayoutTree() and
> uses dirtyBits to call either reattachLayoutTree() on node itself or call
> rebuildLayoutTree() on children nodes.
> 
> Also added some comments explaining some design decisions. Will be removed
> upon completion of separation.
> 
> BUG=595137
> 
> Committed: https://crrev.com/09b7a8cf0647abca732b3718987a629d12a67fee
> Cr-Commit-Position: refs/heads/master@{#426353}

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

to

==========
Revert of Move Layout Tree Construction code into Element::rebuildLayoutTree()
(patchset #12 id:240001 of https://codereview.chromium.org/2398293003/ )

Reason for revert:
Suspected to cause crbug.com/658322.

Original issue's description:
> Move Layout Tree Construction code into Element::rebuildLayoutTree()
>
> This patch uses the two dirty bits (NeedsReattachLayoutTree and
> ChildNeedsReattachLayoutTree) on Node and adds the relevant
> getters/setters for them.
>
> It makes rebuildLayoutTree() public to be accessed in Document.cpp.
>
> It also moves reattachWhitespaceSiblingsIfNeeded() to rebuildLayoutTree() and
> uses dirtyBits to call either reattachLayoutTree() on node itself or call
> rebuildLayoutTree() on children nodes.
>
> Also added some comments explaining some design decisions. Will be removed
> upon completion of separation.
>
> BUG=595137
>
> Committed: https://crrev.com/09b7a8cf0647abca732b3718987a629d12a67fee
> Cr-Commit-Position: refs/heads/master@{#426353}

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

Committed: https://crrev.com/352b045421e6e2ceeba083eeebd27f64adf41194
Cr-Commit-Position: refs/heads/master@{#426991}
==========

[commit-bot: I haz the power, 2016-10-22 13:09:23]

Patchset 2 (id:??) landed as
https://crrev.com/352b045421e6e2ceeba083eeebd27f64adf41194
Cr-Commit-Position: refs/heads/master@{#426991}