Paint table row backgound behind visibility:hidden cell

Paint table row backgound behind visibility:hidden cell

PaintContainerBackgroundBehindCell should check the visibility of
background_object not layout_table_cell_.

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

[cathiechentx, 2017-06-06 10:12:34]

Description was changed from

==========
Paint table row backgound behind visibility:hidden cell

PaintContainerBackgroundBehindCell should check the visibility of
background_object not layout_table_cell_.

BUG=714583
==========

to

==========
Paint table row backgound behind visibility:hidden cell

PaintContainerBackgroundBehindCell should check the visibility of
background_object not layout_table_cell_.

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

[cathiechentx, 2017-06-06 10:15:30]

Description was changed from

==========
Paint table row backgound behind visibility:hidden cell

PaintContainerBackgroundBehindCell should check the visibility of
background_object not layout_table_cell_.

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

to

==========
Paint table row backgound behind visibility:hidden cell

PaintContainerBackgroundBehindCell should check the visibility of
background_object not layout_table_cell_.

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

[cathiechentx, 2017-06-06 10:15:30]

cathiechen@tencent.com changed reviewers:
+ jamesxwei@tencent.com, wangxianzhu@chromium.org

[cathiechentx, 2017-06-06 10:20:31]

Hi xianzhu@, 

Could you please take a look at this?
Thanks! :)

[Xianzhu, 2017-06-06 15:54:44]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2017-06-06 15:54:45]

lgtm

LGTM! Thanks!

[commit-bot: I haz the power, 2017-06-06 15:55:03]

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-06-06 17:05:29]

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

[commit-bot: I haz the power, 2017-06-06 17:05:30]

Try jobs failed on following builders:
  linux_layout_tests_slimming_paint_v2 on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_layout_te...)

[cathiechentx, 2017-06-07 13:31:20]

Failed on: fast/table/invisible-cell-background.html

Looks like this is a historic issue.

This issue is introduced by
https://chromium.googlesource.com/chromium/src/+/10dbbd8cd3b5e9f3d5acd8f5b566...
which uploaded a test case (fast/table/invisible-cell-background.html) to make
sure "Don't paint any background if the cell is not visible."

Safari and Edge have the same behavior as blink. (Don't paint any background if
the cell is not visible.)

Firefox is different but weird too.

But according to the rule of visibility, we should paint parent background if
the cell isn't visible.