Multicol: Min-height should have no effect if content is taller and height is auto.

Multicol: Min-height should have no effect if content is taller and height is auto.

Calling computeLogicalHeight() before layout is somewhat problematic, since we
don't know the content/intrinsic height at that point. Specifying that height
as 0 would return the value of min-height (if specified) and make the multicol
code believe that height is constrained, and prevent columns from getting any
taller than that. In other words, min-height was more or less treated as
max-height in multicol.

Since we only want to know if we have a non-auto height that could constrain
the column heights, we can just skip computeLogicalHeight() if height is
indefinite.

BUG=65731
R=jchaffraix@chromium.org,leviw@chromium.org

Committed: https://crrev.com/79a9f512a11425ca0e55243dc10628f45484feb9
git-svn-id: svn://svn.chromium.org/blink/trunk@202162 bbb929c8-8fbe-4397-9dbb-9b2b20218538

[mstensho (USE GERRIT), 2015-09-11 12:04:43]

pling

[leviw_travelin_and_unemployed, 2015-09-11 15:13:17]

The CQ bit was checked by leviw@chromium.org

[leviw_travelin_and_unemployed, 2015-09-11 15:13:17]

lgtm

plong

I'm gonna slowly be working my way through my backlogs here. Apologies for the
delay!

[commit-bot: I haz the power, 2015-09-11 15:13:26]

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

[commit-bot: I haz the power, 2015-09-11 16:24:27]

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

[commit-bot: I haz the power, 2015-09-11 16:24:28]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[mstensho (USE GERRIT), 2015-09-11 17:29:19]

On 2015/09/11 15:13:17, leviw OOO thru 9-9 )( wrote:
> lgtm
> 
> plong
> 
> I'm gonna slowly be working my way through my backlogs here. Apologies for the
> delay!

Thanks for reviewing. Welcome back from the desert!

[mstensho (USE GERRIT), 2015-09-11 17:45:44]

The CQ bit was checked by mstensho@opera.com

[mstensho (USE GERRIT), 2015-09-11 17:45:45]

The patchset sent to the CQ was uploaded after l-g-t-m from leviw@chromium.org
Link to the patchset: https://codereview.chromium.org/1322513003/#ps40001
(title: "Use block instead of inline in test, so that we don't have to make
assumptions about internal leadi…")

[commit-bot: I haz the power, 2015-09-11 17:45:56]

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

[commit-bot: I haz the power, 2015-09-11 18:51:11]

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

[commit-bot: I haz the power, 2015-09-11 18:51:12]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[mstensho (USE GERRIT), 2015-09-11 18:56:01]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2015-09-11 18:56:09]

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

[commit-bot: I haz the power, 2015-09-11 20:26:31]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=202162

[commit-bot: I haz the power, 2015-09-23 12:26:06]

Patchset 3 (id:??) landed as
https://crrev.com/79a9f512a11425ca0e55243dc10628f45484feb9