If an object's containing block is in a flow thread, so is the object.

If an object's containing block is in a flow thread, so is the object.

Remove harmful condition in LayoutState that the object not be out-of-flow.

Boring details:

In simplified layout of an absolutely positioned object inside a multicol
container we'd fail to realize that we were paginated, and therefore wouldn't
insert pagination struts. This was only problematic for simplified layout. In
normal non-simplified layout, we'd pass a non-zero page logical height to
LayoutState() when entering the flow thread, and, even if the LayoutState of
the absolutely positioned descendant would have no flow thread associated with
it, it would still become m_paginated, thanks to the non-zero page logical
height. Which was enough to get the machinery to insert struts.

BUG=589004
Committed: https://crrev.com/47237f1292938078afe25cc723cf1ccfc48c92cc
Cr-Commit-Position: refs/heads/master@{#433220}

[mstensho (USE GERRIT), 2016-11-18 12:36:19]

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

[commit-bot: I haz the power, 2016-11-18 12:36:26]

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-11-18 14:23:24]

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

[commit-bot: I haz the power, 2016-11-18 14:58:57]

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

[commit-bot: I haz the power, 2016-11-18 14:58:58]

Dry run: This issue passed the CQ dry run.

[eae, 2016-11-18 16:45:12]

LGTM

[eae, 2016-11-18 16:45:51]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2016-11-18 16:47:00]

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-11-18 17:00:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-18 17:03:35]

Description was changed from

==========
If an object's containing block is in a flow thread, so is the object.

Remove harmful condition in LayoutState that the object not be out-of-flow.

Boring details:

In simplified layout of an absolutely positioned object inside a multicol
container we'd fail to realize that we were paginated, and therefore wouldn't
insert pagination struts. This was only problematic for simplified layout. In
normal non-simplified layout, we'd pass a non-zero page logical height to
LayoutState() when entering the flow thread, and, even if the LayoutState of
the absolutely positioned descendant would have no flow thread associated with
it, it would still become m_paginated, thanks to the non-zero page logical
height. Which was enough to get the machinery to insert struts.

BUG=589004
==========

to

==========
If an object's containing block is in a flow thread, so is the object.

Remove harmful condition in LayoutState that the object not be out-of-flow.

Boring details:

In simplified layout of an absolutely positioned object inside a multicol
container we'd fail to realize that we were paginated, and therefore wouldn't
insert pagination struts. This was only problematic for simplified layout. In
normal non-simplified layout, we'd pass a non-zero page logical height to
LayoutState() when entering the flow thread, and, even if the LayoutState of
the absolutely positioned descendant would have no flow thread associated with
it, it would still become m_paginated, thanks to the non-zero page logical
height. Which was enough to get the machinery to insert struts.

BUG=589004

Committed: https://crrev.com/47237f1292938078afe25cc723cf1ccfc48c92cc
Cr-Commit-Position: refs/heads/master@{#433220}
==========

[commit-bot: I haz the power, 2016-11-18 17:03:36]

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