Move painting code from RenderTableCell to TableCellPainter.

Move painting code from RenderTableCell to TableCellPainter.

BUG=412088
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185329

[chrishtr, 2014-11-12 21:46:58]

chrishtr@chromium.org changed reviewers:
+ pdr@chromium.org, schenney@chromium.org, trchen@chromium.org

[chrishtr, 2014-11-13 20:52:28]

chrishtr@chromium.org changed reviewers:
+ leviw@chromium.org

[chrishtr, 2014-11-13 20:52:29]

Ping?

[leviw_travelin_and_unemployed, 2014-11-13 21:05:52]

You've got a handful of truly epic line lengths in there that I'd give some
wrapping love to. Otherwise, lgtm.

https://codereview.chromium.org/724483002/diff/20001/Source/core/paint/TableC...
File Source/core/paint/TableCellPainter.cpp (right):

https://codereview.chromium.org/724483002/diff/20001/Source/core/paint/TableC...
Source/core/paint/TableCellPainter.cpp:19: return
styleForCellFlow->isLeftToRightDirection() ?
m_renderTableCell.section()->cachedCollapsedBorder(&m_renderTableCell, CBSStart)
: m_renderTableCell.section()->cachedCollapsedBorder(&m_renderTableCell,
CBSEnd);
Maybe wrap this? These are long even for me :p

[chrishtr, 2014-11-13 21:31:23]

https://codereview.chromium.org/724483002/diff/20001/Source/core/paint/TableC...
File Source/core/paint/TableCellPainter.cpp (right):

https://codereview.chromium.org/724483002/diff/20001/Source/core/paint/TableC...
Source/core/paint/TableCellPainter.cpp:19: return
styleForCellFlow->isLeftToRightDirection() ?
m_renderTableCell.section()->cachedCollapsedBorder(&m_renderTableCell, CBSStart)
: m_renderTableCell.section()->cachedCollapsedBorder(&m_renderTableCell,
CBSEnd);
On 2014/11/13 21:05:52, leviw wrote:
> Maybe wrap this? These are long even for me :p

Done.

[chrishtr, 2014-11-13 21:43:44]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2014-11-13 21:44:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/724483002/40001

[commit-bot: I haz the power, 2014-11-13 23:06:39]

Committed patchset #3 (id:40001) as 185329