Append fragmentainer groups in outer multicols first.

Append fragmentainer groups in outer multicols first.

We need to evaluate the need for extra fragmentainer groups in ancestral
multicol containers first, so that the inner ones can calculate their column
height restrictions correctly.

BUG=572771
Committed: https://crrev.com/5b4232e508495a6986fa2908946d47b045727aa2
Cr-Commit-Position: refs/heads/master@{#388017}

[mstensho (USE GERRIT), 2016-04-18 20:16:18]

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

[leviw_travelin_and_unemployed, 2016-04-18 20:54:19]

The CQ bit was checked by leviw@chromium.org

[leviw_travelin_and_unemployed, 2016-04-18 20:54:19]

lgtm

I'm looking forward to when you have to have quadruply nested multicols in your
tests.

[commit-bot: I haz the power, 2016-04-18 20:54:52]

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

[mstensho (USE GERRIT), 2016-04-18 21:01:08]

On 2016/04/18 20:54:19, leviw wrote:
> lgtm
> 
> I'm looking forward to when you have to have quadruply nested multicols in
your
> tests.

I'll see what I can do. In the meantime, there's always
LayoutTests/fast/multicol/5-levels-of-nesting-crash.html :)

[commit-bot: I haz the power, 2016-04-18 21:01:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-18 21:02:38]

Description was changed from

==========
Append fragmentainer groups in outer multicols first.

We need to evaluate the need for extra fragmentainer groups in ancestral
multicol containers first, so that the inner ones can calculate their column
height restrictions correctly.

BUG=572771
==========

to

==========
Append fragmentainer groups in outer multicols first.

We need to evaluate the need for extra fragmentainer groups in ancestral
multicol containers first, so that the inner ones can calculate their column
height restrictions correctly.

BUG=572771

Committed: https://crrev.com/5b4232e508495a6986fa2908946d47b045727aa2
Cr-Commit-Position: refs/heads/master@{#388017}
==========

[commit-bot: I haz the power, 2016-04-18 21:02:39]

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

[mstensho (USE GERRIT), 2016-04-19 06:49:33]

Seems to have fixed bug 604532 too.