Revert of Don't early out recursion into PaintLayerChildren when computing composited bounds.

Revert of Don't early out recursion into PaintLayerChildren when computing composited bounds. (patchset #6 id:100001 of https://codereview.chromium.org/2502833002/ )

Reason for revert:
Speculative revert for https://crbug.com/667144.

A checkfail in PaintController.cpp started happening reliably in between r433228
and r433235. This CL looks closely related.

BUG=667144
Original issue's description:
> Don't early out recursion into PaintLayerChildren when computing composited bounds.
>
> PaintLayer::boundingBoxForCompositing recursively computes the union of the
> bounds of a root PaintLayer and its non-comositing descendants. Previously, in PaintLayer::boundingBoxForCompositing, there was code that would avoid recursing
> into PaintLayer children if the PaintLayer was clipped. This is incorrect if there
> are any PaintLayer descendants that have different clipping ancestors than
> the current PaintLayer.
>
> Also fixed a bug that transform-related properties were not considered containing
> blocks for clipping of fixed-position children.
>
> BUG=618046
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
>
> Committed: https://crrev.com/b33e49953078995f1cd28663a8a8b43012ac2eb0
> Cr-Commit-Position: refs/heads/master@{#433232}

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

[Matt Giuca, 2016-11-21 00:46:44]

The CQ bit was checked by mgiuca@chromium.org

[Matt Giuca, 2016-11-21 00:46:44]

Created Revert of Don't early out recursion into PaintLayerChildren when
computing composited bounds.

[commit-bot: I haz the power, 2016-11-21 00:46:53]

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

[Matt Giuca, 2016-11-21 00:47:13]

The CQ bit was unchecked by mgiuca@chromium.org

[Matt Giuca, 2016-11-21 00:47:25]

Description was changed from

==========
Revert of Don't early out recursion into PaintLayerChildren when computing
composited bounds. (patchset #6 id:100001 of
https://codereview.chromium.org/2502833002/ )

Reason for revert:
Speculative revert for https://crbug.com/667144.

A checkfail in PaintController.cpp started happening reliably in between r433228
and r433235. This CL looks closely related.

Original issue's description:
> Don't early out recursion into PaintLayerChildren when computing composited
bounds.
> 
> PaintLayer::boundingBoxForCompositing recursively computes the union of the
> bounds of a root PaintLayer and its non-comositing descendants. Previously, in
PaintLayer::boundingBoxForCompositing, there was code that would avoid recursing
> into PaintLayer children if the PaintLayer was clipped. This is incorrect if
there
> are any PaintLayer descendants that have different clipping ancestors than
> the current PaintLayer.
> 
> Also fixed a bug that transform-related properties were not considered
containing
> blocks for clipping of fixed-position children.
> 
> BUG=618046
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Committed: https://crrev.com/b33e49953078995f1cd28663a8a8b43012ac2eb0
> Cr-Commit-Position: refs/heads/master@{#433232}

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

to

==========
Revert of Don't early out recursion into PaintLayerChildren when computing
composited bounds. (patchset #6 id:100001 of
https://codereview.chromium.org/2502833002/ )

Reason for revert:
Speculative revert for https://crbug.com/667144.

A checkfail in PaintController.cpp started happening reliably in between r433228
and r433235. This CL looks closely related.

BUG=667144

Original issue's description:
> Don't early out recursion into PaintLayerChildren when computing composited
bounds.
> 
> PaintLayer::boundingBoxForCompositing recursively computes the union of the
> bounds of a root PaintLayer and its non-comositing descendants. Previously, in
PaintLayer::boundingBoxForCompositing, there was code that would avoid recursing
> into PaintLayer children if the PaintLayer was clipped. This is incorrect if
there
> are any PaintLayer descendants that have different clipping ancestors than
> the current PaintLayer.
> 
> Also fixed a bug that transform-related properties were not considered
containing
> blocks for clipping of fixed-position children.
> 
> BUG=618046
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Committed: https://crrev.com/b33e49953078995f1cd28663a8a8b43012ac2eb0
> Cr-Commit-Position: refs/heads/master@{#433232}

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

[Matt Giuca, 2016-11-21 00:47:31]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2016-11-21 00:47:44]

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-11-21 02:19:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479689251184700, "parent_rev":
["de2ba8e0d6567433b019e4b0cfb631554db9cd1f", null], "commit_rev":
["df7c106d176efce892d57a401029a4c4f5d7046b", null]}

[commit-bot: I haz the power, 2016-11-21 02:19:38]

Description was changed from

==========
Revert of Don't early out recursion into PaintLayerChildren when computing
composited bounds. (patchset #6 id:100001 of
https://codereview.chromium.org/2502833002/ )

Reason for revert:
Speculative revert for https://crbug.com/667144.

A checkfail in PaintController.cpp started happening reliably in between r433228
and r433235. This CL looks closely related.

BUG=667144

Original issue's description:
> Don't early out recursion into PaintLayerChildren when computing composited
bounds.
> 
> PaintLayer::boundingBoxForCompositing recursively computes the union of the
> bounds of a root PaintLayer and its non-comositing descendants. Previously, in
PaintLayer::boundingBoxForCompositing, there was code that would avoid recursing
> into PaintLayer children if the PaintLayer was clipped. This is incorrect if
there
> are any PaintLayer descendants that have different clipping ancestors than
> the current PaintLayer.
> 
> Also fixed a bug that transform-related properties were not considered
containing
> blocks for clipping of fixed-position children.
> 
> BUG=618046
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Committed: https://crrev.com/b33e49953078995f1cd28663a8a8b43012ac2eb0
> Cr-Commit-Position: refs/heads/master@{#433232}

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

to

==========
Revert of Don't early out recursion into PaintLayerChildren when computing
composited bounds. (patchset #6 id:100001 of
https://codereview.chromium.org/2502833002/ )

Reason for revert:
Speculative revert for https://crbug.com/667144.

A checkfail in PaintController.cpp started happening reliably in between r433228
and r433235. This CL looks closely related.

BUG=667144

Original issue's description:
> Don't early out recursion into PaintLayerChildren when computing composited
bounds.
> 
> PaintLayer::boundingBoxForCompositing recursively computes the union of the
> bounds of a root PaintLayer and its non-comositing descendants. Previously, in
PaintLayer::boundingBoxForCompositing, there was code that would avoid recursing
> into PaintLayer children if the PaintLayer was clipped. This is incorrect if
there
> are any PaintLayer descendants that have different clipping ancestors than
> the current PaintLayer.
> 
> Also fixed a bug that transform-related properties were not considered
containing
> blocks for clipping of fixed-position children.
> 
> BUG=618046
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Committed: https://crrev.com/b33e49953078995f1cd28663a8a8b43012ac2eb0
> Cr-Commit-Position: refs/heads/master@{#433232}

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

[commit-bot: I haz the power, 2016-11-21 02:19:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-21 02:21:24]

Description was changed from

==========
Revert of Don't early out recursion into PaintLayerChildren when computing
composited bounds. (patchset #6 id:100001 of
https://codereview.chromium.org/2502833002/ )

Reason for revert:
Speculative revert for https://crbug.com/667144.

A checkfail in PaintController.cpp started happening reliably in between r433228
and r433235. This CL looks closely related.

BUG=667144

Original issue's description:
> Don't early out recursion into PaintLayerChildren when computing composited
bounds.
> 
> PaintLayer::boundingBoxForCompositing recursively computes the union of the
> bounds of a root PaintLayer and its non-comositing descendants. Previously, in
PaintLayer::boundingBoxForCompositing, there was code that would avoid recursing
> into PaintLayer children if the PaintLayer was clipped. This is incorrect if
there
> are any PaintLayer descendants that have different clipping ancestors than
> the current PaintLayer.
> 
> Also fixed a bug that transform-related properties were not considered
containing
> blocks for clipping of fixed-position children.
> 
> BUG=618046
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Committed: https://crrev.com/b33e49953078995f1cd28663a8a8b43012ac2eb0
> Cr-Commit-Position: refs/heads/master@{#433232}

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

to

==========
Revert of Don't early out recursion into PaintLayerChildren when computing
composited bounds. (patchset #6 id:100001 of
https://codereview.chromium.org/2502833002/ )

Reason for revert:
Speculative revert for https://crbug.com/667144.

A checkfail in PaintController.cpp started happening reliably in between r433228
and r433235. This CL looks closely related.

BUG=667144

Original issue's description:
> Don't early out recursion into PaintLayerChildren when computing composited
bounds.
>
> PaintLayer::boundingBoxForCompositing recursively computes the union of the
> bounds of a root PaintLayer and its non-comositing descendants. Previously, in
PaintLayer::boundingBoxForCompositing, there was code that would avoid recursing
> into PaintLayer children if the PaintLayer was clipped. This is incorrect if
there
> are any PaintLayer descendants that have different clipping ancestors than
> the current PaintLayer.
>
> Also fixed a bug that transform-related properties were not considered
containing
> blocks for clipping of fixed-position children.
>
> BUG=618046
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
>
> Committed: https://crrev.com/b33e49953078995f1cd28663a8a8b43012ac2eb0
> Cr-Commit-Position: refs/heads/master@{#433232}

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

Committed: https://crrev.com/754b0c5c59d039c70bd6fea540af4b706ea39c77
Cr-Commit-Position: refs/heads/master@{#433468}
==========

[commit-bot: I haz the power, 2016-11-21 02:21:25]

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