GitLab

There was some old code that wasn't needed anymore, and it was causing
trouble for nested multicol. We need to break before a multicol
container if it doesn't fit. The column layout algorithm always tries to
limit the block size of an inner multicol container, so that it will fit
in the outer fragmentation context. If that's not possible, though, we
need to break before it and retry in the next outer fragmentainer.

So for multicol container children we needed the code path that we
previously only ran for child.IsMonolithic(), but we can in fact always
do that (if it doesn't fit, and we're allowed to break, let's break!),
so just delete the unnecessary code, and fix 5 tests. :)

Bug: 829028
Change-Id: I93b6b10c8b427055e69f094b460c13fa68a49fa6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2575031
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#834585}