Add out-of-flow objects under the inline in a continuation chain, when possible.

Add out-of-flow objects under the inline in a continuation chain, when possible.

The same goes for floating objects. Only when a floating or out-of-flow
positioned object is to be added between two block-level children should we add
it to the anonymous block box holding the block-level children. If the new
child is to be added before a block-level child, and this beforeChild is the
first block-level child, we should rather make the new child the last child of
the preceding inline, rather than the first child of the anonymous block
containing block-level children.

Also cleaned up and documented the code somewhat.

BUG=689643
Review-Url: https://codereview.chromium.org/2698243002
Cr-Commit-Position: refs/heads/master@{#451525}
Committed: https://chromium.googlesource.com/chromium/src/+/77ec19ddefb32688cd18277ed7412531b536401a

[mstensho (USE GERRIT), 2017-02-16 10:42:03]

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

[commit-bot: I haz the power, 2017-02-16 10:45:05]

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-02-16 12:46:40]

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

[commit-bot: I haz the power, 2017-02-16 12:46:41]

Dry run: This issue passed the CQ dry run.

[mstensho (USE GERRIT), 2017-02-16 12:49:08]

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

[mstensho (USE GERRIT), 2017-02-16 12:49:08]

See patch set #1 for actual changes. The rest is just cleanup.

[eae, 2017-02-17 19:44:42]

LGTM

[eae, 2017-02-17 19:44:48]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2017-02-17 19:45:49]

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-02-17 20:36:53]

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

[commit-bot: I haz the power, 2017-02-17 20:36:54]

Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[mstensho (USE GERRIT), 2017-02-19 17:36:36]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2017-02-19 17:36:50]

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-02-19 18:59:08]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487525796194920,
"parent_rev": "5939cf034a8d276d3f08b16a85503905a4c0580a", "commit_rev":
"77ec19ddefb32688cd18277ed7412531b536401a"}

[commit-bot: I haz the power, 2017-02-19 18:59:45]

Description was changed from

==========
Add out-of-flow objects under the inline in a continuation chain, when possible.

The same goes for floating objects. Only when a floating or out-of-flow
positioned object is to be added between two block-level children should we add
it to the anonymous block box holding the block-level children. If the new
child is to be added before a block-level child, and this beforeChild is the
first block-level child, we should rather make the new child the last child of
the preceding inline, rather than the first child of the anonymous block
containing block-level children.

Also cleaned up and documented the code somewhat.

BUG=689643
==========

to

==========
Add out-of-flow objects under the inline in a continuation chain, when possible.

The same goes for floating objects. Only when a floating or out-of-flow
positioned object is to be added between two block-level children should we add
it to the anonymous block box holding the block-level children. If the new
child is to be added before a block-level child, and this beforeChild is the
first block-level child, we should rather make the new child the last child of
the preceding inline, rather than the first child of the anonymous block
containing block-level children.

Also cleaned up and documented the code somewhat.

BUG=689643

Review-Url: https://codereview.chromium.org/2698243002
Cr-Commit-Position: refs/heads/master@{#451525}
Committed:
https://chromium.googlesource.com/chromium/src/+/77ec19ddefb32688cd18277ed741...
==========

[commit-bot: I haz the power, 2017-02-19 18:59:46]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/77ec19ddefb32688cd18277ed741...