Separate method for calculating logical height based on CSS properties.

Separate method for calculating logical height based on CSS properties.

Move it out of LayoutTable::layout(), since that method is more than crowded
enough as it is.

Had to make convertStyleLogicalHeightToComputedHeight() a const method, since I
decided to make the new method const. Constified
convertStyleLogicalWidthToComputedWidth() as well, for the sake of consistency.

Committed: https://crrev.com/10041638092c86dd7c5bcc37ba71ed3f30f0e306
Cr-Commit-Position: refs/heads/master@{#425757}

[mstensho (USE GERRIT), 2016-10-17 08:01:37]

mstensho@opera.com changed reviewers:
+ dgrogan@chromium.org, eae@chromium.org

[eae, 2016-10-17 16:53:31]

Nice cleanup! LGTM

[mstensho (USE GERRIT), 2016-10-17 17:03:52]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-10-17 17:04:26]

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-17 19:53:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-17 19:57:23]

Description was changed from

==========
Separate method for calculating logical height based on CSS properties.

Move it out of LayoutTable::layout(), since that method is more than crowded
enough as it is.

Had to make convertStyleLogicalHeightToComputedHeight() a const method, since I
decided to make the new method const. Constified
convertStyleLogicalWidthToComputedWidth() as well, for the sake of consistency.
==========

to

==========
Separate method for calculating logical height based on CSS properties.

Move it out of LayoutTable::layout(), since that method is more than crowded
enough as it is.

Had to make convertStyleLogicalHeightToComputedHeight() a const method, since I
decided to make the new method const. Constified
convertStyleLogicalWidthToComputedWidth() as well, for the sake of consistency.

Committed: https://crrev.com/10041638092c86dd7c5bcc37ba71ed3f30f0e306
Cr-Commit-Position: refs/heads/master@{#425757}
==========

[commit-bot: I haz the power, 2016-10-17 19:57:24]

Patchset 1 (id:??) landed as
https://crrev.com/10041638092c86dd7c5bcc37ba71ed3f30f0e306
Cr-Commit-Position: refs/heads/master@{#425757}