Use paginationStrut() instead of paginationStrutForRow()

Use paginationStrut() instead of paginationStrutForRow()

There's no change in behaviour. Now that we setPaginationStrut() on the row,
we don't need to call paginationStrutForRow() outside of layout anymore.

BUG=642814
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/1c81ee4bbd257cb08f4aa603abece114d855a18e
Cr-Commit-Position: refs/heads/master@{#422858}

[rhogan, 2016-10-03 18:34:02]

Description was changed from

==========
Use paginationStrut() instead of paginationStrutForRow()

BUG=642814
==========

to

==========
Use paginationStrut() instead of paginationStrutForRow()

BUG=642814
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[rhogan, 2016-10-03 18:34:28]

The CQ bit was checked by robhogan@gmail.com to run a CQ dry run

[rhogan, 2016-10-03 18:34:40]

robhogan@gmail.com changed reviewers:
+ mstensho@opera.com

[commit-bot: I haz the power, 2016-10-03 18:35:05]

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-10-03 18:37:28]

lgtm

[rhogan, 2016-10-03 18:39:41]

The CQ bit was checked by robhogan@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-03 18:39:54]

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-10-03 20:21:00]

On 2016/10/03 18:37:28, mstensho wrote:
> lgtm

Wait! We need a test for this, right?

[rhogan, 2016-10-03 20:58:18]

On 2016/10/03 at 20:21:00, mstensho wrote:
> On 2016/10/03 18:37:28, mstensho wrote:
> > lgtm
> 
> Wait! We need a test for this, right?

Do we? There should be no change in behaviour so I would only expect current
tests to pass.

[mstensho (USE GERRIT), 2016-10-03 21:05:02]

On 2016/10/03 20:58:18, rhogan wrote:
> On 2016/10/03 at 20:21:00, mstensho wrote:
> > On 2016/10/03 18:37:28, mstensho wrote:
> > > lgtm
> > 
> > Wait! We need a test for this, right?
> 
> Do we? There should be no change in behaviour so I would only expect current
> tests to pass.

Oh, I thought that this fixed bug 642814, but if not, please state in the commit
message that it doesn't change behavior.

[commit-bot: I haz the power, 2016-10-03 22:11:31]

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

[commit-bot: I haz the power, 2016-10-03 22:11:32]

Dry run: This issue passed the CQ dry run.

[rhogan, 2016-10-04 17:57:48]

Description was changed from

==========
Use paginationStrut() instead of paginationStrutForRow()

BUG=642814
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Use paginationStrut() instead of paginationStrutForRow()

There's no change in behaviour. Now that we setPaginationStrut() on the row,
we don't need to call paginationStrutForRow() outside of layout anymore.

BUG=642814
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[rhogan, 2016-10-04 17:57:52]

The CQ bit was checked by robhogan@gmail.com

[commit-bot: I haz the power, 2016-10-04 17:58:27]

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-04 18:07:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-04 18:10:02]

Description was changed from

==========
Use paginationStrut() instead of paginationStrutForRow()

There's no change in behaviour. Now that we setPaginationStrut() on the row,
we don't need to call paginationStrutForRow() outside of layout anymore.

BUG=642814
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Use paginationStrut() instead of paginationStrutForRow()

There's no change in behaviour. Now that we setPaginationStrut() on the row,
we don't need to call paginationStrutForRow() outside of layout anymore.

BUG=642814
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/1c81ee4bbd257cb08f4aa603abece114d855a18e
Cr-Commit-Position: refs/heads/master@{#422858}
==========

[commit-bot: I haz the power, 2016-10-04 18:10:03]

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