Set logical top and height of table rows and cells in the first layout pass.

Set logical top and height of table rows and cells in the first layout pass.

This gives the fragmentation machinery an opportunity to insert breaks at the
right places. We previously assumed that all cells were at the top of their
table section, so break insertion was completely bogus. While we'd get a second
chance to break correctly in the second layout pass, this doesn't always work
too well. There's currently some code in layoutRows() in LayoutTableSection
that attempts to adjust the row height when we change where we break inside a
table cell, but it doesn't re-align cells vertically after this adjustment.
That code must die, and this CL is a preparatory step.

BUG=534751
Committed: https://crrev.com/0f36142505fcc4bc9b2f3518649befd2b5cbc63f
Cr-Commit-Position: refs/heads/master@{#426015}

[mstensho (USE GERRIT), 2016-10-18 15:10:35]

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

[commit-bot: I haz the power, 2016-10-18 15:10:51]

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-18 16:30:46]

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

[commit-bot: I haz the power, 2016-10-18 16:30:47]

Dry run: This issue passed the CQ dry run.

[mstensho (USE GERRIT), 2016-10-18 17:11:14]

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

[eae, 2016-10-18 18:04:31]

LGTM

[eae, 2016-10-18 18:05:29]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2016-10-18 18:47: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-18 18:57:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-21 13:02:00]

Description was changed from

==========
Set logical top and height of table rows and cells in the first layout pass.

This gives the fragmentation machinery an opportunity to insert breaks at the
right places. We previously assumed that all cells were at the top of their
table section, so break insertion was completely bogus. While we'd get a second
chance to break correctly in the second layout pass, this doesn't always work
too well. There's currently some code in layoutRows() in LayoutTableSection
that attempts to adjust the row height when we change where we break inside a
table cell, but it doesn't re-align cells vertically after this adjustment.
That code must die, and this CL is a preparatory step.

BUG=534751
==========

to

==========
Set logical top and height of table rows and cells in the first layout pass.

This gives the fragmentation machinery an opportunity to insert breaks at the
right places. We previously assumed that all cells were at the top of their
table section, so break insertion was completely bogus. While we'd get a second
chance to break correctly in the second layout pass, this doesn't always work
too well. There's currently some code in layoutRows() in LayoutTableSection
that attempts to adjust the row height when we change where we break inside a
table cell, but it doesn't re-align cells vertically after this adjustment.
That code must die, and this CL is a preparatory step.

BUG=534751

Committed: https://crrev.com/0f36142505fcc4bc9b2f3518649befd2b5cbc63f
Cr-Commit-Position: refs/heads/master@{#426015}
==========

[commit-bot: I haz the power, 2016-10-21 13:02:01]

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