Introduce maxColumnLogicalHeight() and clean up a little.

Introduce maxColumnLogicalHeight() and clean up a little.

Looks like MultiColumnFragmentainerGroup::calculateMaxColumnHeight() isn't
going to be the only place where we need this functionality.

No functional changes intended.

R=leviw@chromium.org

Committed: https://crrev.com/1a8e73e16131664aae15e07536df7fc42e0d0542
Cr-Commit-Position: refs/heads/master@{#376075}

[leviw_travelin_and_unemployed, 2016-02-18 00:48:10]

The CQ bit was checked by leviw@chromium.org

[leviw_travelin_and_unemployed, 2016-02-18 00:48:11]

lgtm

[commit-bot: I haz the power, 2016-02-18 00:49:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1696713003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1696713003/1

[commit-bot: I haz the power, 2016-02-18 02:41:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-18 02:42:46]

Description was changed from

==========
Introduce maxColumnLogicalHeight() and clean up a little.

Looks like MultiColumnFragmentainerGroup::calculateMaxColumnHeight() isn't
going to be the only place where we need this functionality.

No functional changes intended.

R=leviw@chromium.org
==========

to

==========
Introduce maxColumnLogicalHeight() and clean up a little.

Looks like MultiColumnFragmentainerGroup::calculateMaxColumnHeight() isn't
going to be the only place where we need this functionality.

No functional changes intended.

R=leviw@chromium.org

Committed: https://crrev.com/1a8e73e16131664aae15e07536df7fc42e0d0542
Cr-Commit-Position: refs/heads/master@{#376075}
==========

[commit-bot: I haz the power, 2016-02-18 02:42:47]

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