Need to consider the need for a soft break even when inserting a forced break.

Need to consider the need for a soft break even when inserting a forced break.

A forced break will just take us to the next fragmentainer. However, the next
fragmentainer may not be tall enough to fit any part of the block we're laying
out, which means that we may have to skip to a fragmentainer further ahead -
one that is tall enough. This situation may arise in nested multicol, because
then we may get inner fragmentainers of variable height.

Another reason why we need this is that a forced break on an object is inserted
before we apply clearance. After we have applied clearance, we may have ended
up at a position where there's not enough space left to fit any part of the
block we're laying out. So, again, we may need to skip to the next
fragmentainer.

BUG=647475
Committed: https://crrev.com/4545acb6ba2bd97d53d4641b90ee1d96fb53f98d
Cr-Commit-Position: refs/heads/master@{#420041}

[mstensho (USE GERRIT), 2016-09-21 11:50:50]

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

[commit-bot: I haz the power, 2016-09-21 11:51:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[mstensho (USE GERRIT), 2016-09-21 12:56:45]

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

[commit-bot: I haz the power, 2016-09-21 13:09:22]

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

[commit-bot: I haz the power, 2016-09-21 13:09:23]

Dry run: This issue passed the CQ dry run.

[eae, 2016-09-21 13:22:28]

OK, LGTM

[mstensho (USE GERRIT), 2016-09-21 13:43:44]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-09-21 13:44: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, 2016-09-21 13:49:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-21 13:52:15]

Description was changed from

==========
Need to consider the need for a soft break even when inserting a forced break.

A forced break will just take us to the next fragmentainer. However, the next
fragmentainer may not be tall enough to fit any part of the block we're laying
out, which means that we may have to skip to a fragmentainer further ahead -
one that is tall enough. This situation may arise in nested multicol, because
then we may get inner fragmentainers of variable height.

Another reason why we need this is that a forced break on an object is inserted
before we apply clearance. After we have applied clearance, we may have ended
up at a position where there's not enough space left to fit any part of the
block we're laying out. So, again, we may need to skip to the next
fragmentainer.

BUG=647475
==========

to

==========
Need to consider the need for a soft break even when inserting a forced break.

A forced break will just take us to the next fragmentainer. However, the next
fragmentainer may not be tall enough to fit any part of the block we're laying
out, which means that we may have to skip to a fragmentainer further ahead -
one that is tall enough. This situation may arise in nested multicol, because
then we may get inner fragmentainers of variable height.

Another reason why we need this is that a forced break on an object is inserted
before we apply clearance. After we have applied clearance, we may have ended
up at a position where there's not enough space left to fit any part of the
block we're laying out. So, again, we may need to skip to the next
fragmentainer.

BUG=647475

Committed: https://crrev.com/4545acb6ba2bd97d53d4641b90ee1d96fb53f98d
Cr-Commit-Position: refs/heads/master@{#420041}
==========

[commit-bot: I haz the power, 2016-09-21 13:52:16]

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

[mstensho (USE GERRIT), 2016-09-22 13:08:22]

This turned out to also fix bug 604609.