Remove pageLogicalHeightChanged() from LayoutState.

Remove pageLogicalHeightChanged() from LayoutState.

It was only used from insertFloatingObject(), and in a bogus manner at that.
Added a TODO instead. We haven't even positioned the float at this point, so
attempting layout for pagination here is essentially bad.

We make sure to relayout correctly for pagination when we get to
positionNewFloats() later, anyway.

Committed: https://crrev.com/78b7de043288de59a3fd7d741c4a1061fc601029
Cr-Commit-Position: refs/heads/master@{#429533}

[mstensho (USE GERRIT), 2016-11-02 22:45:36]

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

[commit-bot: I haz the power, 2016-11-02 22:46:35]

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-02 22:47:14]

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

[mstensho (USE GERRIT), 2016-11-02 22:56:51]

Description was changed from

==========
Remove pageLogicalHeightChanged() from LayoutState.

It was only used from insertFloatingObject(), and in a bogus manner at that.
Added a TODO instead. We haven't even positioned the float at this point, so
attempting layout for pagination here is essentially bad.
==========

to

==========
Remove pageLogicalHeightChanged() from LayoutState.

It was only used from insertFloatingObject(), and in a bogus manner at that.
Added a TODO instead. We haven't even positioned the float at this point, so
attempting layout for pagination here is essentially bad.

We make sure to relayout correctly for pagination when we get to
positionNewFloats() later, anyway.
==========

[commit-bot: I haz the power, 2016-11-03 00:22:19]

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

[commit-bot: I haz the power, 2016-11-03 00:22:20]

Dry run: This issue passed the CQ dry run.

[szager1, 2016-11-03 01:18:12]

lgtm

[mstensho (USE GERRIT), 2016-11-03 06:01:52]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-11-03 06:02:21]

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-03 06:07:45]

Description was changed from

==========
Remove pageLogicalHeightChanged() from LayoutState.

It was only used from insertFloatingObject(), and in a bogus manner at that.
Added a TODO instead. We haven't even positioned the float at this point, so
attempting layout for pagination here is essentially bad.

We make sure to relayout correctly for pagination when we get to
positionNewFloats() later, anyway.
==========

to

==========
Remove pageLogicalHeightChanged() from LayoutState.

It was only used from insertFloatingObject(), and in a bogus manner at that.
Added a TODO instead. We haven't even positioned the float at this point, so
attempting layout for pagination here is essentially bad.

We make sure to relayout correctly for pagination when we get to
positionNewFloats() later, anyway.
==========

[commit-bot: I haz the power, 2016-11-03 06:07:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-03 06:10:56]

Description was changed from

==========
Remove pageLogicalHeightChanged() from LayoutState.

It was only used from insertFloatingObject(), and in a bogus manner at that.
Added a TODO instead. We haven't even positioned the float at this point, so
attempting layout for pagination here is essentially bad.

We make sure to relayout correctly for pagination when we get to
positionNewFloats() later, anyway.
==========

to

==========
Remove pageLogicalHeightChanged() from LayoutState.

It was only used from insertFloatingObject(), and in a bogus manner at that.
Added a TODO instead. We haven't even positioned the float at this point, so
attempting layout for pagination here is essentially bad.

We make sure to relayout correctly for pagination when we get to
positionNewFloats() later, anyway.

Committed: https://crrev.com/78b7de043288de59a3fd7d741c4a1061fc601029
Cr-Commit-Position: refs/heads/master@{#429533}
==========

[commit-bot: I haz the power, 2016-11-03 06:10:57]

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