Remove infinite clip in PaintArtifactCompositor.

Remove infinite clip in PaintArtifactCompositor.

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

Committed: https://crrev.com/b1e93efba353876e43c8b2007cdba51c033785cd
Cr-Commit-Position: refs/heads/master@{#440985}

[chrishtr, 2016-12-29 02:16:58]

Description was changed from

==========
none


none


none

BUG=
==========

to

==========
none


none


none

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

[chrishtr, 2016-12-29 02:17:38]

Description was changed from

==========
none


none


none

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

to

==========
Remove infinite clip in PaintArtifactCompositor.

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

[chrishtr, 2016-12-29 02:17:41]

The CQ bit was checked by chrishtr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-29 02:17:52]

Dry run: 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-29 03:38:32]

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

[commit-bot: I haz the power, 2016-12-29 03:38:33]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[chrishtr, 2016-12-29 17:07:19]

The CQ bit was checked by chrishtr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-29 17:07:40]

Dry run: 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-29 18:52:55]

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

[commit-bot: I haz the power, 2016-12-29 18:52:56]

Dry run: This issue passed the CQ dry run.

[chrishtr, 2016-12-29 19:13:18]

chrishtr@chromium.org changed reviewers:
+ wkorman@chromium.org

[wkorman, 2016-12-29 19:20:40]

lgtm

https://codereview.chromium.org/2604083002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h (right):

https://codereview.chromium.org/2604083002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h:130:
FloatRect localToAncestorClipRect(
nit: this is only unit tested indirectly, may be worth adding a few specific
test(s) as we expose it as a public method, but not a big deal.

[chrishtr, 2016-12-29 19:34:21]

https://codereview.chromium.org/2604083002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h (right):

https://codereview.chromium.org/2604083002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h:130:
FloatRect localToAncestorClipRect(
On 2016/12/29 at 19:20:40, wkorman wrote:
> nit: this is only unit tested indirectly, may be worth adding a few specific
test(s) as we expose it as a public method, but not a big deal.

Good idea. Done.

[chrishtr, 2016-12-29 19:34:24]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2016-12-29 19:34:25]

The patchset sent to the CQ was uploaded after l-g-t-m from wkorman@chromium.org
Link to the patchset: https://codereview.chromium.org/2604083002/#ps40001
(title: "none")

[commit-bot: I haz the power, 2016-12-29 19:34:38]

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-29 21:07:34]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1483040064847690,
"parent_rev": "0469896a7d0603cf0c40ad476fd37271ae07ce48", "commit_rev":
"acaab15699a8163743ef6c3e4648c5bdd0e8b77d"}

[commit-bot: I haz the power, 2016-12-29 21:08:03]

Description was changed from

==========
Remove infinite clip in PaintArtifactCompositor.

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

to

==========
Remove infinite clip in PaintArtifactCompositor.

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

Review-Url: https://codereview.chromium.org/2604083002
==========

[commit-bot: I haz the power, 2016-12-29 21:08:04]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2017-01-02 15:53:07]

Description was changed from

==========
Remove infinite clip in PaintArtifactCompositor.

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

Review-Url: https://codereview.chromium.org/2604083002
==========

to

==========
Remove infinite clip in PaintArtifactCompositor.

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

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

[commit-bot: I haz the power, 2017-01-02 15:53:08]

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