Clean up LayoutTableSection::calcRowLogicalHeight() a bit.

Clean up LayoutTableSection::calcRowLogicalHeight() a bit.

Since nobody apparently wants to deal with a rowspanned cell unless we're at
its first row, just skip that cell for subsequent rows, instead of having
checks for this everywhere.

Also removed a debug hashmap that was just used to assert that we didn't add
duplicate cells to a vector. Check the vector directly instead.

Committed: https://crrev.com/cc645344ea206b9bf3df064054fec1b7a734d38c
Cr-Commit-Position: refs/heads/master@{#426555}

[mstensho (USE GERRIT), 2016-10-20 12:58:51]

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

[commit-bot: I haz the power, 2016-10-20 12:59:04]

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-20 14:12:00]

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

[commit-bot: I haz the power, 2016-10-20 14:12:01]

Dry run: This issue passed the CQ dry run.

[mstensho (USE GERRIT), 2016-10-20 14:24:56]

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

[eae, 2016-10-20 16:56:47]

LGTM

[mstensho (USE GERRIT), 2016-10-20 18:59:33]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-10-20 18:59:53]

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-20 19:08:13]

Committed patchset #1 (id:1)

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

Description was changed from

==========
Clean up LayoutTableSection::calcRowLogicalHeight() a bit.

Since nobody apparently wants to deal with a rowspanned cell unless we're at
its first row, just skip that cell for subsequent rows, instead of having
checks for this everywhere.

Also removed a debug hashmap that was just used to assert that we didn't add
duplicate cells to a vector. Check the vector directly instead.
==========

to

==========
Clean up LayoutTableSection::calcRowLogicalHeight() a bit.

Since nobody apparently wants to deal with a rowspanned cell unless we're at
its first row, just skip that cell for subsequent rows, instead of having
checks for this everywhere.

Also removed a debug hashmap that was just used to assert that we didn't add
duplicate cells to a vector. Check the vector directly instead.

Committed: https://crrev.com/cc645344ea206b9bf3df064054fec1b7a734d38c
Cr-Commit-Position: refs/heads/master@{#426555}
==========

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

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