Revert of Made ElementRareData store ComputedStyle on LayoutObject if possible.

Revert of Made ElementRareData store ComputedStyle on LayoutObject if possible. (patchset #5 id:100001 of https://codereview.chromium.org/2293713002/ )

Reason for revert:
Storage of ComputedStyle on LayoutObject causes invalid LayoutObject state. Also no longer required for consistency as Squad project requires redesign and will no longer store ComputedStyle on Node's data union.

Original issue's description:
> Made ElementRareData store ComputedStyle on LayoutObject if possible.
>
> Made ElementRareData store ComputedStyle on LayoutObject of it's base
> class NodeRareDataBase if it has a LayoutObject.
> Otherwise it is stored on the ComputedStyle member of ElementRareData.
>
> The reason for this is to make it consistent with storage of
> ComputedStyle on Node, which stores a union that can be one of:
> 1) a ComputedStyle
> 2) a LayoutObject (ComputedStyle can be stored on this)
> 3) a NodeRareData (LayoutObject can be stored on this which stores
> ComputedStyle or if no LayoutObject ComputedStyle can be stored
> directly on ElementRareData)
>
> This patch encapsulates the logic of where to store the ComputedStyle
> on the ElementRareData so that it doesn't need to be done at the call
> site (e.g. in new method Node::setComputedStyle from
> https://codereview.chromium.org/2001453002)
>
> The clearComputedStyle method is special and only clears the
> ComputedStyle member as its current use is not intended to clear a
> LayoutObject's ComputedStyle.
>
> BUG=595137
>
> Committed: https://crrev.com/c542d3744b905a2a00f5ac2dd2047a6a323ec181
> Cr-Commit-Position: refs/heads/master@{#418453}

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

[Bugs Nash, 2016-09-28 02:15:58]

The CQ bit was checked by bugsnash@chromium.org

[Bugs Nash, 2016-09-28 02:15:59]

Created Revert of Made ElementRareData store ComputedStyle on LayoutObject if
possible.

[commit-bot: I haz the power, 2016-09-28 02:16:24]

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-09-28 02:16:24]

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

[commit-bot: I haz the power, 2016-09-28 02:16:25]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[nainar, 2016-09-28 02:18:05]

The CQ bit was checked by nainar@chromium.org

[nainar, 2016-09-28 02:18:06]

lgtm

[commit-bot: I haz the power, 2016-09-28 02:18:23]

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-09-28 02:20:21]

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

[commit-bot: I haz the power, 2016-09-28 02:20:22]

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

[esprehn, 2016-09-28 02:24:21]

lgtm

[Bugs Nash, 2016-09-28 02:35:49]

The CQ bit was checked by bugsnash@chromium.org

[Bugs Nash, 2016-09-28 02:35:49]

The patchset sent to the CQ was uploaded after l-g-t-m from nainar@chromium.org,
esprehn@chromium.org
Link to the patchset: https://codereview.chromium.org/2378573002/#ps160001
(title: "Fixed conflicts (nainar)")

[commit-bot: I haz the power, 2016-09-28 02:36:09]

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-09-28 04:34:59]

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

[commit-bot: I haz the power, 2016-09-28 04:35:01]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[esprehn, 2016-09-28 07:20:48]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2016-09-28 07:21:01]

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-09-28 08:18:01]

Committed patchset #2 (id:160001)

[commit-bot: I haz the power, 2016-09-28 08:20:07]

Description was changed from

==========
Revert of Made ElementRareData store ComputedStyle on LayoutObject if possible.
(patchset #5 id:100001 of https://codereview.chromium.org/2293713002/ )

Reason for revert:
Storage of ComputedStyle on LayoutObject causes invalid LayoutObject state. Also
no longer required for consistency as Squad project requires redesign and will
no longer store ComputedStyle on Node's data union.

Original issue's description:
> Made ElementRareData store ComputedStyle on LayoutObject if possible.
> 
> Made ElementRareData store ComputedStyle on LayoutObject of it's base
> class NodeRareDataBase if it has a LayoutObject.
> Otherwise it is stored on the ComputedStyle member of ElementRareData.
> 
> The reason for this is to make it consistent with storage of
> ComputedStyle on Node, which stores a union that can be one of:
> 1) a ComputedStyle
> 2) a LayoutObject (ComputedStyle can be stored on this)
> 3) a NodeRareData (LayoutObject can be stored on this which stores
> ComputedStyle or if no LayoutObject ComputedStyle can be stored
> directly on ElementRareData)
> 
> This patch encapsulates the logic of where to store the ComputedStyle
> on the ElementRareData so that it doesn't need to be done at the call
> site (e.g. in new method Node::setComputedStyle from
> https://codereview.chromium.org/2001453002)
> 
> The clearComputedStyle method is special and only clears the
> ComputedStyle member as its current use is not intended to clear a
> LayoutObject's ComputedStyle.
> 
> BUG=595137
> 
> Committed: https://crrev.com/c542d3744b905a2a00f5ac2dd2047a6a323ec181
> Cr-Commit-Position: refs/heads/master@{#418453}

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

to

==========
Revert of Made ElementRareData store ComputedStyle on LayoutObject if possible.
(patchset #5 id:100001 of https://codereview.chromium.org/2293713002/ )

Reason for revert:
Storage of ComputedStyle on LayoutObject causes invalid LayoutObject state. Also
no longer required for consistency as Squad project requires redesign and will
no longer store ComputedStyle on Node's data union.

Original issue's description:
> Made ElementRareData store ComputedStyle on LayoutObject if possible.
>
> Made ElementRareData store ComputedStyle on LayoutObject of it's base
> class NodeRareDataBase if it has a LayoutObject.
> Otherwise it is stored on the ComputedStyle member of ElementRareData.
>
> The reason for this is to make it consistent with storage of
> ComputedStyle on Node, which stores a union that can be one of:
> 1) a ComputedStyle
> 2) a LayoutObject (ComputedStyle can be stored on this)
> 3) a NodeRareData (LayoutObject can be stored on this which stores
> ComputedStyle or if no LayoutObject ComputedStyle can be stored
> directly on ElementRareData)
>
> This patch encapsulates the logic of where to store the ComputedStyle
> on the ElementRareData so that it doesn't need to be done at the call
> site (e.g. in new method Node::setComputedStyle from
> https://codereview.chromium.org/2001453002)
>
> The clearComputedStyle method is special and only clears the
> ComputedStyle member as its current use is not intended to clear a
> LayoutObject's ComputedStyle.
>
> BUG=595137
>
> Committed: https://crrev.com/c542d3744b905a2a00f5ac2dd2047a6a323ec181
> Cr-Commit-Position: refs/heads/master@{#418453}

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

Committed: https://crrev.com/ced6b481f3ebe3886ec5f0dd25e9815a2f8b96c8
Cr-Commit-Position: refs/heads/master@{#421478}
==========

[commit-bot: I haz the power, 2016-09-28 08:20:08]

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