Better isPageLogicalHeightKnown() implementation.

Better isPageLogicalHeightKnown() implementation.

Need to consult the flow thread, if we have one. We may run into situations
where fragmentainer groups in the first column set have got their height
calculated, while later column sets still haven't calculated it [1]. So
checking if flow thread offset 0 is in a fragmentainer of known height isn't
good enough.

Also moved the implementation from LayoutBlock to LayoutBox, since it's pretty
coincidental that we currently don't need this particular method outside of
LayoutBlock.

[1] LayoutMultiColumnSet::recalculateColumnHeight() may reset the column
heights if it detects that the column set has been moved since previous layout
pass.

BUG=670902
Committed: https://crrev.com/95342a4986af56783818b4a7f578f4b3b2bf5f2b
Cr-Commit-Position: refs/heads/master@{#437063}

[mstensho (USE GERRIT), 2016-12-06 14:13:58]

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

[commit-bot: I haz the power, 2016-12-06 14:14:09]

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, 2016-12-06 15:54:44]

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

[commit-bot: I haz the power, 2016-12-06 15:54:45]

Dry run: This issue passed the CQ dry run.

[mstensho (USE GERRIT), 2016-12-06 16:40:54]

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

[szager1, 2016-12-07 18:31:50]

lgtm

[mstensho (USE GERRIT), 2016-12-07 19:10:32]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-12-07 19:10:57]

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-12-07 21:19:47]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481137832054230, "parent_rev":
"8152d3db9baf3e88b0ec73242315e7b7a4f64714", "commit_rev":
"2cdb8c33168434a61aec213e59ead29390f1154d"}

[commit-bot: I haz the power, 2016-12-07 21:20:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-07 21:23:54]

Description was changed from

==========
Better isPageLogicalHeightKnown() implementation.

Need to consult the flow thread, if we have one. We may run into situations
where fragmentainer groups in the first column set have got their height
calculated, while later column sets still haven't calculated it [1]. So
checking if flow thread offset 0 is in a fragmentainer of known height isn't
good enough.

Also moved the implementation from LayoutBlock to LayoutBox, since it's pretty
coincidental that we currently don't need this particular method outside of
LayoutBlock.

[1] LayoutMultiColumnSet::recalculateColumnHeight() may reset the column
heights if it detects that the column set has been moved since previous layout
pass.

BUG=670902
==========

to

==========
Better isPageLogicalHeightKnown() implementation.

Need to consult the flow thread, if we have one. We may run into situations
where fragmentainer groups in the first column set have got their height
calculated, while later column sets still haven't calculated it [1]. So
checking if flow thread offset 0 is in a fragmentainer of known height isn't
good enough.

Also moved the implementation from LayoutBlock to LayoutBox, since it's pretty
coincidental that we currently don't need this particular method outside of
LayoutBlock.

[1] LayoutMultiColumnSet::recalculateColumnHeight() may reset the column
heights if it detects that the column set has been moved since previous layout
pass.

BUG=670902

Committed: https://crrev.com/95342a4986af56783818b4a7f578f4b3b2bf5f2b
Cr-Commit-Position: refs/heads/master@{#437063}
==========

[commit-bot: I haz the power, 2016-12-07 21:23:55]

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