[css-tables] Fix caret position inside empty cell

[css-tables] Fix caret position inside empty cell

Override hasLineIfEmpty() in LayoutTableCell to consider
that it has an empty line if it's editable.

This fixes the position of the caret in empty cells,
that was painted lower than expected
(and then modified when you start to edit the cell).

BUG=421716
TEST=editing/caret/caret-in-empty-cell.html

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

[Manuel Rego, 2016-12-29 10:41:39]

rego@igalia.com changed reviewers:
+ dgrogan@chromium.org, eae@chromium.org, yosin@chromium.org, yutak@chromium.org

[Manuel Rego, 2016-12-29 10:49:59]

BTW, some tests are going to fail, I'm running the tryjobs to get the new
baselines.

[eae, 2017-01-02 18:34:42]

LGTM

[Manuel Rego, 2017-01-03 08:27:39]

The CQ bit was checked by rego@igalia.com

[commit-bot: I haz the power, 2017-01-03 08:27:46]

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, 2017-01-03 10:15:27]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483432059941940,
"parent_rev": "659fd993500e60cfcbaa2af7cfbadbd557a90eb6", "commit_rev":
"4b7bbcfeb90b9a3dcb431dd16d734de02a976b20"}

[commit-bot: I haz the power, 2017-01-03 10:15:58]

Description was changed from

==========
[css-tables] Fix caret position inside empty cell

Override hasLineIfEmpty() in LayoutTableCell to consider
that it has an empty line if it's editable.

This fixes the position of the caret in empty cells,
that was painted lower than expected
(and then modified when you start to edit the cell).

BUG=421716
TEST=editing/caret/caret-in-empty-cell.html
==========

to

==========
[css-tables] Fix caret position inside empty cell

Override hasLineIfEmpty() in LayoutTableCell to consider
that it has an empty line if it's editable.

This fixes the position of the caret in empty cells,
that was painted lower than expected
(and then modified when you start to edit the cell).

BUG=421716
TEST=editing/caret/caret-in-empty-cell.html

Review-Url: https://codereview.chromium.org/2601923003
==========

[commit-bot: I haz the power, 2017-01-03 10:15:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-03 10:18:11]

Description was changed from

==========
[css-tables] Fix caret position inside empty cell

Override hasLineIfEmpty() in LayoutTableCell to consider
that it has an empty line if it's editable.

This fixes the position of the caret in empty cells,
that was painted lower than expected
(and then modified when you start to edit the cell).

BUG=421716
TEST=editing/caret/caret-in-empty-cell.html

Review-Url: https://codereview.chromium.org/2601923003
==========

to

==========
[css-tables] Fix caret position inside empty cell

Override hasLineIfEmpty() in LayoutTableCell to consider
that it has an empty line if it's editable.

This fixes the position of the caret in empty cells,
that was painted lower than expected
(and then modified when you start to edit the cell).

BUG=421716
TEST=editing/caret/caret-in-empty-cell.html

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

[commit-bot: I haz the power, 2017-01-03 10:18:12]

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