commit 5e611227e94309d4fb81069384a094cf6f93876b
author Morten Stenshorne <mstensho@opera.com> Mon Sep 04 15:21:08 2017
committer Commit Bot <commit-bot@chromium.org> Mon Sep 04 15:21:08 2017
tree e8c31ebf200a8ec7436977d675841053d1f13831
parent 9d9b6f21fda90dde8d63d8131ec16dfd3e06dc3e

[LayoutNG] Proper overflow legacy write-back inside multicol.

Only calculate overflow when at the last fragment. It's only then that we can
be sure that all children have been laid out. This used to trigger DCHECK
failures in legacy layout.

Also don't let a line break fool us into believing that the block fragmented
(by creating unfinished break tokens).

Re-enable and update unit test OverflowedBlock, now that overflow no longer
causes DCHECK failures in legacy layout.

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I9a104c3a7392db66c251e6f8da8ea8c7e16ba357
Bug: 757767
Reviewed-on: https://chromium-review.googlesource.com/645968
Commit-Queue: Morten Stenshorne <mstensho@opera.com>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#499503}