Revert of Paint collapsed borders of a table as one display item

Revert of Paint collapsed borders of a table as one display item (patchset #10 id:180001 of https://codereview.chromium.org/2430313004/ )

Reason for revert:
Caused bug 664887. We miss invalidation of collapsed borders when they are changed by layout.

BUG=664887
Original issue's description:
> Paint collapsed borders of a table as one display item
>
> This simplifies paint invalidation logic on collapsed border changes.
>
> About performance:
> With this CL we create a big display item on the table containing all
> collapsed border paintings, instead of many small display items on cells.
> This will cause more painting when the display item is invalidated, but
> saves time to handle many cached display items. Perf try jobs [1] show
> that this CL actually improves performance of one test
> (blink_perf.paint.large-table-background-change-with-invisible-collapsed-borders.html),
> and doesn't affect much of performance of other tests.
> Cluster-telemetry run showed no change or slight progression [2].
>
> [1] https://codereview.chromium.org/2430313004/#ps60001
> [2] https://ct.skia.org/chromium_perf_runs/, run 1433
>
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> BUG=663208
>
> Committed: https://crrev.com/c5cc11e88930c6e3f5e08b330fb14b6b571a0404
> Cr-Commit-Position: refs/heads/master@{#431744}

TBR=chrishtr@chromium.org,wkorman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=663208
Committed: https://crrev.com/b2a470747585456b0c545551d61a7b3f7baffda9
Cr-Commit-Position: refs/heads/master@{#431903}

[Xianzhu, 2016-11-14 17:00:55]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-11-14 17:00:56]

Created Revert of Paint collapsed borders of a table as one display item

[commit-bot: I haz the power, 2016-11-14 17:02:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[chrishtr, 2016-11-14 17:19:51]

lgtm

[commit-bot: I haz the power, 2016-11-14 19:28:23]

Description was changed from

==========
Revert of Paint collapsed borders of a table as one display item (patchset #10
id:180001 of https://codereview.chromium.org/2430313004/ )

Reason for revert:
Caused bug 664887. We miss invalidation of collapsed borders when they are
changed by layout.

BUG=664887

Original issue's description:
> Paint collapsed borders of a table as one display item
> 
> This simplifies paint invalidation logic on collapsed border changes.
> 
> About performance:
> With this CL we create a big display item on the table containing all
> collapsed border paintings, instead of many small display items on cells.
> This will cause more painting when the display item is invalidated, but
> saves time to handle many cached display items. Perf try jobs [1] show
> that this CL actually improves performance of one test
>
(blink_perf.paint.large-table-background-change-with-invisible-collapsed-borders.html),
> and doesn't affect much of performance of other tests.
> Cluster-telemetry run showed no change or slight progression [2].
> 
> [1] https://codereview.chromium.org/2430313004/#ps60001
> [2] https://ct.skia.org/chromium_perf_runs/, run 1433
> 
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> BUG=663208
> 
> Committed: https://crrev.com/c5cc11e88930c6e3f5e08b330fb14b6b571a0404
> Cr-Commit-Position: refs/heads/master@{#431744}

TBR=chrishtr@chromium.org,wkorman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=663208
==========

to

==========
Revert of Paint collapsed borders of a table as one display item (patchset #10
id:180001 of https://codereview.chromium.org/2430313004/ )

Reason for revert:
Caused bug 664887. We miss invalidation of collapsed borders when they are
changed by layout.

BUG=664887

Original issue's description:
> Paint collapsed borders of a table as one display item
> 
> This simplifies paint invalidation logic on collapsed border changes.
> 
> About performance:
> With this CL we create a big display item on the table containing all
> collapsed border paintings, instead of many small display items on cells.
> This will cause more painting when the display item is invalidated, but
> saves time to handle many cached display items. Perf try jobs [1] show
> that this CL actually improves performance of one test
>
(blink_perf.paint.large-table-background-change-with-invisible-collapsed-borders.html),
> and doesn't affect much of performance of other tests.
> Cluster-telemetry run showed no change or slight progression [2].
> 
> [1] https://codereview.chromium.org/2430313004/#ps60001
> [2] https://ct.skia.org/chromium_perf_runs/, run 1433
> 
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> BUG=663208
> 
> Committed: https://crrev.com/c5cc11e88930c6e3f5e08b330fb14b6b571a0404
> Cr-Commit-Position: refs/heads/master@{#431744}

TBR=chrishtr@chromium.org,wkorman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=663208
==========

[commit-bot: I haz the power, 2016-11-14 19:28:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-14 20:10:06]

Description was changed from

==========
Revert of Paint collapsed borders of a table as one display item (patchset #10
id:180001 of https://codereview.chromium.org/2430313004/ )

Reason for revert:
Caused bug 664887. We miss invalidation of collapsed borders when they are
changed by layout.

BUG=664887

Original issue's description:
> Paint collapsed borders of a table as one display item
> 
> This simplifies paint invalidation logic on collapsed border changes.
> 
> About performance:
> With this CL we create a big display item on the table containing all
> collapsed border paintings, instead of many small display items on cells.
> This will cause more painting when the display item is invalidated, but
> saves time to handle many cached display items. Perf try jobs [1] show
> that this CL actually improves performance of one test
>
(blink_perf.paint.large-table-background-change-with-invisible-collapsed-borders.html),
> and doesn't affect much of performance of other tests.
> Cluster-telemetry run showed no change or slight progression [2].
> 
> [1] https://codereview.chromium.org/2430313004/#ps60001
> [2] https://ct.skia.org/chromium_perf_runs/, run 1433
> 
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> BUG=663208
> 
> Committed: https://crrev.com/c5cc11e88930c6e3f5e08b330fb14b6b571a0404
> Cr-Commit-Position: refs/heads/master@{#431744}

TBR=chrishtr@chromium.org,wkorman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=663208
==========

to

==========
Revert of Paint collapsed borders of a table as one display item (patchset #10
id:180001 of https://codereview.chromium.org/2430313004/ )

Reason for revert:
Caused bug 664887. We miss invalidation of collapsed borders when they are
changed by layout.

BUG=664887

Original issue's description:
> Paint collapsed borders of a table as one display item
>
> This simplifies paint invalidation logic on collapsed border changes.
>
> About performance:
> With this CL we create a big display item on the table containing all
> collapsed border paintings, instead of many small display items on cells.
> This will cause more painting when the display item is invalidated, but
> saves time to handle many cached display items. Perf try jobs [1] show
> that this CL actually improves performance of one test
>
(blink_perf.paint.large-table-background-change-with-invisible-collapsed-borders.html),
> and doesn't affect much of performance of other tests.
> Cluster-telemetry run showed no change or slight progression [2].
>
> [1] https://codereview.chromium.org/2430313004/#ps60001
> [2] https://ct.skia.org/chromium_perf_runs/, run 1433
>
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> BUG=663208
>
> Committed: https://crrev.com/c5cc11e88930c6e3f5e08b330fb14b6b571a0404
> Cr-Commit-Position: refs/heads/master@{#431744}

TBR=chrishtr@chromium.org,wkorman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=663208

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

[commit-bot: I haz the power, 2016-11-14 20:10:09]

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