Reland of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during painting

Reland of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during painting (patchset #1 id:1 of https://codereview.chromium.org/2550333002/ )

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

Original issue's description:
> 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}

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/8f64b7c27bc9a8757977498ee748c641bfe7247e
Cr-Commit-Position: refs/heads/master@{#436518}

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

The CQ bit was checked by wangxianzhu@chromium.org

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

Created Reland of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during
painting

[commit-bot: I haz the power, 2016-12-06 03:57:47]

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 03:58:26]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480996647532130, "parent_rev":
"5d9da744d0febfd5fcfe0f108eedf4aa38e942a8", "commit_rev":
"b20c753d610cf8d6b8e032e81f3ab5c16bb77c23"}

[commit-bot: I haz the power, 2016-12-06 03:58:49]

Description was changed from

==========
Reland of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during
painting (patchset #1 id:1 of https://codereview.chromium.org/2550333002/ )

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

Original issue's description:
> 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}

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

==========
Reland of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during
painting (patchset #1 id:1 of https://codereview.chromium.org/2550333002/ )

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

Original issue's description:
> 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}

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 03:58:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-06 04:02:33]

Description was changed from

==========
Reland of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during
painting (patchset #1 id:1 of https://codereview.chromium.org/2550333002/ )

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

Original issue's description:
> 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}

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

==========
Reland of Verify paintedOutputOfObjectHasNoEffectRegardlessOfSize during
painting (patchset #1 id:1 of https://codereview.chromium.org/2550333002/ )

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

Original issue's description:
> 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}

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/8f64b7c27bc9a8757977498ee748c641bfe7247e
Cr-Commit-Position: refs/heads/master@{#436518}
==========

[commit-bot: I haz the power, 2016-12-06 04:02:35]

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

[haraken, 2016-12-06 07:06:29]

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

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

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