Introduce markChildForPaginationRelayoutIfNeeded().

Introduce markChildForPaginationRelayoutIfNeeded().

No behavioral changes intended.

This replaces markForPaginationRelayoutIfNeeded(). Since the method was always
called when it was the container that was being laid out, this is more
"correct", and in the same spirit as e.g.
updateBlockChildDirtyBitsBeforeLayout(), adjustBlockChildForPagination(), and
so on.

This is a preparatory patch to allow for calling
offsetFromLogicalTopOfFirstPage(), which uses LayoutState more heavily.
When using LayoutState, you generally need to be laying out the very same
object as the one LayoutState points to.

BUG=487026
Committed: https://crrev.com/74ff11de78f94ac88173da124c23ca6b6b333642
Cr-Commit-Position: refs/heads/master@{#421815}

[mstensho (USE GERRIT), 2016-09-29 12:08:23]

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

[commit-bot: I haz the power, 2016-09-29 12:08:36]

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-09-29 13:06:43]

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

[mstensho (USE GERRIT), 2016-09-29 13:06:43]

The actual fix for bug 487026 is further down the road, but if you want me to
land a performance test first, take a look at
https://codereview.chromium.org/1695193006/ . There were some concerns about
noise, though.

[commit-bot: I haz the power, 2016-09-29 13:14:38]

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

[commit-bot: I haz the power, 2016-09-29 13:14:39]

Dry run: This issue passed the CQ dry run.

[eae, 2016-09-29 14:30:59]

LGTM

[eae, 2016-09-29 14:31:06]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2016-09-29 14:31:37]

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-09-29 14:38:41]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-29 14:40:28]

Description was changed from

==========
Introduce markChildForPaginationRelayoutIfNeeded().

No behavioral changes intended.

This replaces markForPaginationRelayoutIfNeeded(). Since the method was always
called when it was the container that was being laid out, this is more
"correct", and in the same spirit as e.g.
updateBlockChildDirtyBitsBeforeLayout(), adjustBlockChildForPagination(), and
so on.

This is a preparatory patch to allow for calling
offsetFromLogicalTopOfFirstPage(), which uses LayoutState more heavily.
When using LayoutState, you generally need to be laying out the very same
object as the one LayoutState points to.

BUG=487026
==========

to

==========
Introduce markChildForPaginationRelayoutIfNeeded().

No behavioral changes intended.

This replaces markForPaginationRelayoutIfNeeded(). Since the method was always
called when it was the container that was being laid out, this is more
"correct", and in the same spirit as e.g.
updateBlockChildDirtyBitsBeforeLayout(), adjustBlockChildForPagination(), and
so on.

This is a preparatory patch to allow for calling
offsetFromLogicalTopOfFirstPage(), which uses LayoutState more heavily.
When using LayoutState, you generally need to be laying out the very same
object as the one LayoutState points to.

BUG=487026

Committed: https://crrev.com/74ff11de78f94ac88173da124c23ca6b6b333642
Cr-Commit-Position: refs/heads/master@{#421815}
==========

[commit-bot: I haz the power, 2016-09-29 14:40:29]

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