Provide a dedicated getter for the offset to a repeatable THEAD.

Provide a dedicated getter for the offset to a repeatable THEAD.

pageLogicalOffset() is otherwise only used as an optimization during layout,
and the way we're optimizing for fragmenting is soon going to change.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/5fb3c583c02dcdeaabeb999d96797bb7e307f03e
Cr-Commit-Position: refs/heads/master@{#428203}

[mstensho (USE GERRIT), 2016-10-27 13:14:38]

Description was changed from

==========
Provide a dedicated getter for the offset to a repeatable THEAD.

pageLogicalOffset() is otherwise only used as an optimization during layout,
and the way we're optimizing for fragmenting is soon going to change.
==========

to

==========
Provide a dedicated getter for the offset to a repeatable THEAD.

pageLogicalOffset() is otherwise only used as an optimization during layout,
and the way we're optimizing for fragmenting is soon going to change.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[mstensho (USE GERRIT), 2016-10-27 13:14:44]

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

[commit-bot: I haz the power, 2016-10-27 13:15:07]

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-10-27 15:05:16]

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

[commit-bot: I haz the power, 2016-10-27 15:05:16]

Dry run: This issue passed the CQ dry run.

[mstensho (USE GERRIT), 2016-10-27 15:05:36]

mstensho@opera.com changed reviewers:
+ dgrogan@chromium.org, eae@chromium.org, robhogan@gmail.com

[eae, 2016-10-27 17:05:10]

LGTM w/nit

https://codereview.chromium.org/2455733003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/LayoutTable.h (right):

https://codereview.chromium.org/2455733003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutTable.h:420: LayoutUnit
offsetToFirstRepeatableHeader() const {
How about blockOffsetToFirstRepeatableHeader to make it clear which direction
the offset is in?

[mstensho (USE GERRIT), 2016-10-27 20:25:47]

https://codereview.chromium.org/2455733003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/LayoutTable.h (right):

https://codereview.chromium.org/2455733003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutTable.h:420: LayoutUnit
offsetToFirstRepeatableHeader() const {
On 2016/10/27 17:05:10, eae wrote:
> How about blockOffsetToFirstRepeatableHeader to make it clear which direction
> the offset is in?

Done.

[mstensho (USE GERRIT), 2016-10-27 20:25:51]

The CQ bit was checked by mstensho@opera.com

[mstensho (USE GERRIT), 2016-10-27 20:25:51]

The patchset sent to the CQ was uploaded after l-g-t-m from eae@chromium.org
Link to the patchset: https://codereview.chromium.org/2455733003/#ps20001
(title: "code review")

[commit-bot: I haz the power, 2016-10-27 20:26:34]

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-10-27 23:23:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-27 23:26:37]

Description was changed from

==========
Provide a dedicated getter for the offset to a repeatable THEAD.

pageLogicalOffset() is otherwise only used as an optimization during layout,
and the way we're optimizing for fragmenting is soon going to change.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Provide a dedicated getter for the offset to a repeatable THEAD.

pageLogicalOffset() is otherwise only used as an optimization during layout,
and the way we're optimizing for fragmenting is soon going to change.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/5fb3c583c02dcdeaabeb999d96797bb7e307f03e
Cr-Commit-Position: refs/heads/master@{#428203}
==========

[commit-bot: I haz the power, 2016-10-27 23:26:38]

Patchset 2 (id:??) landed as
https://crrev.com/5fb3c583c02dcdeaabeb999d96797bb7e307f03e
Cr-Commit-Position: refs/heads/master@{#428203}