Revert of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during painting

Revert of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during painting (patchset #8 id:140001 of https://codereview.chromium.org/2549553002/ )

Reason for revert:
Breaks the following webkit_unit_tests:
TableCellPainterTest.CollapsedBorderAndOverflow
TablePainterTest.CollapsedBorderInterestRectChange
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgrounds/0
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgrounds/2
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgroundsDynamic/2
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgroundsDynamic/0
on WebKit Win x64 Builder (dbg)
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win%20x64%20Builder%20%28dbg%29/builds/99618

Original issue's description:
> Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during painting
>
> Checks that nothing should be painted if
> paintedOutputOfObjectHasNoEffectRegardlessOfSize() is true, to
> catch under-invalidations caused by it.
>
> The verification also works during the first paint, so the existing
> layout tests have enough coverage for the function.
>
> Also fixed some check failures:
> - LayoutDetailsMarker: was actual under-invalidation.
> - LayoutView: as display item client of frame scroll corner. Was not an
>   actual under-invalidation because scroll controls are invalidated
>   separately, but change code to avoid the check failure. This is not
>   applicable to rootLayerScrolling.
> - LayoutSVGBlock (LayoutSVGText and LayoutSVGForeignObject): now assume
>   there is always SVG effects.
>
> Objects painted in SkPictureBuilder are not checked. One example is
> LayoutSVGResourceClipper which doesn't have any painting on the backing,
> but may output paint operations in SkPictureBuilder for the clipped
> object.
>
> BUG=669327
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
>
> Committed: https://crrev.com/31a69618a214eb36bad90baa1a4f4d6bc15d008e
> Cr-Commit-Position: refs/heads/master@{#436455}

TBR=chrishtr@chromium.org,pdr@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=669327
Committed: https://crrev.com/f6231f793a5e42804598473806f3f43046ac1fbc
Cr-Commit-Position: refs/heads/master@{#436476}

[Xianzhu, 2016-12-06 01:01:21]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-12-06 01:01:22]

Created Revert of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during
painting

[commit-bot: I haz the power, 2016-12-06 01:02:20]

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-12-06 01:03:09]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480986081913190, "parent_rev":
"888cac8774457eff3a6df16964c7eeb17148e83a", "commit_rev":
"8134abc06455808f223f72cf29a657af070a1bf8"}

[commit-bot: I haz the power, 2016-12-06 01:03:36]

Description was changed from

==========
Revert of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during
painting (patchset #8 id:140001 of https://codereview.chromium.org/2549553002/ )

Reason for revert:
Breaks the following webkit_unit_tests:
TableCellPainterTest.CollapsedBorderAndOverflow
TablePainterTest.CollapsedBorderInterestRectChange
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgrounds/0
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgrounds/2
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgroundsDynamic/2
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgroundsDynamic/0
on WebKit Win x64 Builder (dbg)
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win%...

Original issue's description:
> Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during painting
> 
> Checks that nothing should be painted if
> paintedOutputOfObjectHasNoEffectRegardlessOfSize() is true, to
> catch under-invalidations caused by it.
> 
> The verification also works during the first paint, so the existing
> layout tests have enough coverage for the function.
> 
> Also fixed some check failures:
> - LayoutDetailsMarker: was actual under-invalidation.
> - LayoutView: as display item client of frame scroll corner. Was not an
>   actual under-invalidation because scroll controls are invalidated
>   separately, but change code to avoid the check failure. This is not
>   applicable to rootLayerScrolling.
> - LayoutSVGBlock (LayoutSVGText and LayoutSVGForeignObject): now assume
>   there is always SVG effects.
> 
> Objects painted in SkPictureBuilder are not checked. One example is
> LayoutSVGResourceClipper which doesn't have any painting on the backing,
> but may output paint operations in SkPictureBuilder for the clipped
> object.
> 
> BUG=669327
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Committed: https://crrev.com/31a69618a214eb36bad90baa1a4f4d6bc15d008e
> Cr-Commit-Position: refs/heads/master@{#436455}

TBR=chrishtr@chromium.org,pdr@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=669327
==========

to

==========
Revert of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during
painting (patchset #8 id:140001 of https://codereview.chromium.org/2549553002/ )

Reason for revert:
Breaks the following webkit_unit_tests:
TableCellPainterTest.CollapsedBorderAndOverflow
TablePainterTest.CollapsedBorderInterestRectChange
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgrounds/0
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgrounds/2
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgroundsDynamic/2
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgroundsDynamic/0
on WebKit Win x64 Builder (dbg)
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win%...

Original issue's description:
> Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during painting
> 
> Checks that nothing should be painted if
> paintedOutputOfObjectHasNoEffectRegardlessOfSize() is true, to
> catch under-invalidations caused by it.
> 
> The verification also works during the first paint, so the existing
> layout tests have enough coverage for the function.
> 
> Also fixed some check failures:
> - LayoutDetailsMarker: was actual under-invalidation.
> - LayoutView: as display item client of frame scroll corner. Was not an
>   actual under-invalidation because scroll controls are invalidated
>   separately, but change code to avoid the check failure. This is not
>   applicable to rootLayerScrolling.
> - LayoutSVGBlock (LayoutSVGText and LayoutSVGForeignObject): now assume
>   there is always SVG effects.
> 
> Objects painted in SkPictureBuilder are not checked. One example is
> LayoutSVGResourceClipper which doesn't have any painting on the backing,
> but may output paint operations in SkPictureBuilder for the clipped
> object.
> 
> BUG=669327
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Committed: https://crrev.com/31a69618a214eb36bad90baa1a4f4d6bc15d008e
> Cr-Commit-Position: refs/heads/master@{#436455}

TBR=chrishtr@chromium.org,pdr@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=669327
==========

[commit-bot: I haz the power, 2016-12-06 01:03:37]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-06 01:08:08]

Description was changed from

==========
Revert of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during
painting (patchset #8 id:140001 of https://codereview.chromium.org/2549553002/ )

Reason for revert:
Breaks the following webkit_unit_tests:
TableCellPainterTest.CollapsedBorderAndOverflow
TablePainterTest.CollapsedBorderInterestRectChange
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgrounds/0
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgrounds/2
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgroundsDynamic/2
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgroundsDynamic/0
on WebKit Win x64 Builder (dbg)
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win%...

Original issue's description:
> Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during painting
> 
> Checks that nothing should be painted if
> paintedOutputOfObjectHasNoEffectRegardlessOfSize() is true, to
> catch under-invalidations caused by it.
> 
> The verification also works during the first paint, so the existing
> layout tests have enough coverage for the function.
> 
> Also fixed some check failures:
> - LayoutDetailsMarker: was actual under-invalidation.
> - LayoutView: as display item client of frame scroll corner. Was not an
>   actual under-invalidation because scroll controls are invalidated
>   separately, but change code to avoid the check failure. This is not
>   applicable to rootLayerScrolling.
> - LayoutSVGBlock (LayoutSVGText and LayoutSVGForeignObject): now assume
>   there is always SVG effects.
> 
> Objects painted in SkPictureBuilder are not checked. One example is
> LayoutSVGResourceClipper which doesn't have any painting on the backing,
> but may output paint operations in SkPictureBuilder for the clipped
> object.
> 
> BUG=669327
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Committed: https://crrev.com/31a69618a214eb36bad90baa1a4f4d6bc15d008e
> Cr-Commit-Position: refs/heads/master@{#436455}

TBR=chrishtr@chromium.org,pdr@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=669327
==========

to

==========
Revert of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during
painting (patchset #8 id:140001 of https://codereview.chromium.org/2549553002/ )

Reason for revert:
Breaks the following webkit_unit_tests:
TableCellPainterTest.CollapsedBorderAndOverflow
TablePainterTest.CollapsedBorderInterestRectChange
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgrounds/0
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgrounds/2
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgroundsDynamic/2
All/PaintLayerPainterTest.TableCollapsedBorderNeedsPaintPhaseDescendantBlockBackgroundsDynamic/0
on WebKit Win x64 Builder (dbg)
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win%...

Original issue's description:
> Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during painting
>
> Checks that nothing should be painted if
> paintedOutputOfObjectHasNoEffectRegardlessOfSize() is true, to
> catch under-invalidations caused by it.
>
> The verification also works during the first paint, so the existing
> layout tests have enough coverage for the function.
>
> Also fixed some check failures:
> - LayoutDetailsMarker: was actual under-invalidation.
> - LayoutView: as display item client of frame scroll corner. Was not an
>   actual under-invalidation because scroll controls are invalidated
>   separately, but change code to avoid the check failure. This is not
>   applicable to rootLayerScrolling.
> - LayoutSVGBlock (LayoutSVGText and LayoutSVGForeignObject): now assume
>   there is always SVG effects.
>
> Objects painted in SkPictureBuilder are not checked. One example is
> LayoutSVGResourceClipper which doesn't have any painting on the backing,
> but may output paint operations in SkPictureBuilder for the clipped
> object.
>
> BUG=669327
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
>
> Committed: https://crrev.com/31a69618a214eb36bad90baa1a4f4d6bc15d008e
> Cr-Commit-Position: refs/heads/master@{#436455}

TBR=chrishtr@chromium.org,pdr@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=669327

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

[commit-bot: I haz the power, 2016-12-06 01:08:09]

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

[Xianzhu, 2016-12-06 03:57:27]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2554763002/ by wangxianzhu@chromium.org.

The reason for reverting is: The failures were because of conflict with
https://codereview.chromium.org/2502353003/ which
has been reverted..

[haraken, 2016-12-06 06:58:42]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2555683002/ by haraken@chromium.org.

The reason for reverting is: A lot of css tests started failing after this CL:

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.11%20%28...
.