Respect constrained height on nested multicol containers.

Respect constrained height on nested multicol containers.

If there's no more space in an inner multicol container (according to e.g. its
height or max-height), don't create any additional fragmentainer groups (i.e.
column rows).

The spec isn't clear here, but this change moves us closer to Edge, and also
eliminates cases where we'd previously end up with pathological numbers of
fragmentainer groups.

Also flipped the logic in hasFragmentainerGroupForColumnAt(), and renamed it to
needsNewFragmentainerGroupAt().

BUG=688158
Review-Url: https://codereview.chromium.org/2725943003
Cr-Commit-Position: refs/heads/master@{#454411}
Committed: https://chromium.googlesource.com/chromium/src/+/7fa349e632a44c152b05ca6a66ade5f2e5b3f139

[mstensho (USE GERRIT), 2017-03-02 14:29:15]

The CQ bit was checked by mstensho@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-02 14:29:39]

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, 2017-03-02 16:26:21]

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

[commit-bot: I haz the power, 2017-03-02 16:26:21]

Dry run: This issue passed the CQ dry run.

[mstensho (USE GERRIT), 2017-03-02 16:47:09]

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

[eae, 2017-03-02 22:26:27]

LGTM

[mstensho (USE GERRIT), 2017-03-02 22:34:36]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2017-03-02 22:35:03]

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, 2017-03-02 22:39:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488494076883500, "parent_rev":
"76e94d0aa96fe99356b5009716739027fa4d0159", "commit_rev":
"7fa349e632a44c152b05ca6a66ade5f2e5b3f139"}

[commit-bot: I haz the power, 2017-03-02 22:40:02]

Description was changed from

==========
Respect constrained height on nested multicol containers.

If there's no more space in an inner multicol container (according to e.g. its
height or max-height), don't create any additional fragmentainer groups (i.e.
column rows).

The spec isn't clear here, but this change moves us closer to Edge, and also
eliminates cases where we'd previously end up with pathological numbers of
fragmentainer groups.

Also flipped the logic in hasFragmentainerGroupForColumnAt(), and renamed it to
needsNewFragmentainerGroupAt().

BUG=688158
==========

to

==========
Respect constrained height on nested multicol containers.

If there's no more space in an inner multicol container (according to e.g. its
height or max-height), don't create any additional fragmentainer groups (i.e.
column rows).

The spec isn't clear here, but this change moves us closer to Edge, and also
eliminates cases where we'd previously end up with pathological numbers of
fragmentainer groups.

Also flipped the logic in hasFragmentainerGroupForColumnAt(), and renamed it to
needsNewFragmentainerGroupAt().

BUG=688158

Review-Url: https://codereview.chromium.org/2725943003
Cr-Commit-Position: refs/heads/master@{#454411}
Committed:
https://chromium.googlesource.com/chromium/src/+/7fa349e632a44c152b05ca6a66ad...
==========

[commit-bot: I haz the power, 2017-03-02 22:40:04]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/7fa349e632a44c152b05ca6a66ad...