Improve LayoutMultiColumnFlowThread::IsPageLogicalHeightKnown().

Improve LayoutMultiColumnFlowThread::IsPageLogicalHeightKnown().

Simply returning whether the last column set has a known height isn't really
sufficient. All column sets need to have known heights.

However, instead of asking each column set every time we call
LayoutMultiColumnFlowThread::IsPageLogicalHeightKnown(), use a flag.

BUG=709792, 717939
Review-Url: https://codereview.chromium.org/2877703003
Cr-Commit-Position: refs/heads/master@{#471231}
Committed: https://chromium.googlesource.com/chromium/src/+/079f4500332799e2e3e03eb15243b1b8d5342fe6

[mstensho (USE GERRIT), 2017-05-11 13:13:09]

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

[commit-bot: I haz the power, 2017-05-11 13:13:26]

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-05-11 14:28:17]

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

[commit-bot: I haz the power, 2017-05-11 14:28:18]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[mstensho (USE GERRIT), 2017-05-11 15:29:16]

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

[eae, 2017-05-11 16:47:47]

LGTM

[mstensho (USE GERRIT), 2017-05-11 16:51:15]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2017-05-11 16:51: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-05-11 18:55:09]

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

[commit-bot: I haz the power, 2017-05-11 18:55:15]

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

[mstensho (USE GERRIT), 2017-05-11 20:24:22]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2017-05-11 20:25:14]

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-05-11 23:25:57]

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

[commit-bot: I haz the power, 2017-05-11 23:25:58]

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

[mstensho (USE GERRIT), 2017-05-12 05:17:04]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2017-05-12 05:17:24]

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-05-12 05:57:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494566224100400, "parent_rev":
"75b8f2f4aaf8ce5af39150d2c77b17014e163e14", "commit_rev":
"079f4500332799e2e3e03eb15243b1b8d5342fe6"}

[commit-bot: I haz the power, 2017-05-12 05:58:07]

Description was changed from

==========
Improve LayoutMultiColumnFlowThread::IsPageLogicalHeightKnown().

Simply returning whether the last column set has a known height isn't really
sufficient. All column sets need to have known heights.

However, instead of asking each column set every time we call
LayoutMultiColumnFlowThread::IsPageLogicalHeightKnown(), use a flag.

BUG=709792,717939
==========

to

==========
Improve LayoutMultiColumnFlowThread::IsPageLogicalHeightKnown().

Simply returning whether the last column set has a known height isn't really
sufficient. All column sets need to have known heights.

However, instead of asking each column set every time we call
LayoutMultiColumnFlowThread::IsPageLogicalHeightKnown(), use a flag.

BUG=709792,717939

Review-Url: https://codereview.chromium.org/2877703003
Cr-Commit-Position: refs/heads/master@{#471231}
Committed:
https://chromium.googlesource.com/chromium/src/+/079f4500332799e2e3e03eb15243...
==========

[commit-bot: I haz the power, 2017-05-12 05:58:13]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/079f4500332799e2e3e03eb15243...