Refactor ClipPathHelper in PaintLayerPainter.cpp for reuse

Refactor ClipPathHelper in PaintLayerPainter.cpp for reuse

Separate out client concerns and move the helper class to a new file and
rename it to ClipPathClipper. This will allow sharing this code between
the PaintLayerPainter and SVGPaintContext code-paths.

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

Committed: https://crrev.com/66574fefebe275a0d8346e741730dfbb638721c5
Cr-Commit-Position: refs/heads/master@{#413518}

[fs, 2016-08-22 14:27:28]

Description was changed from

==========
Refactor ClipPathHelper in PaintLayerPainter.cpp for reuse

Separate out client concerns and move the helper class to a new file and
rename it to ClipPathClipper. This will allow sharing this code between
the PaintLayerPainter and SVGPaintContext code-paths.

BUG=633028
==========

to

==========
Refactor ClipPathHelper in PaintLayerPainter.cpp for reuse

Separate out client concerns and move the helper class to a new file and
rename it to ClipPathClipper. This will allow sharing this code between
the PaintLayerPainter and SVGPaintContext code-paths.

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

[fs, 2016-08-22 14:27:40]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-22 14:34:31]

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

[fs, 2016-08-22 15:50:14]

fs@opera.com changed reviewers:
+ chrishtr@chromium.org, schenney@chromium.org

[Stephen Chennney, 2016-08-22 16:02:58]

Fix the comment then LGTM.

https://codereview.chromium.org/2265123002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/paint/PaintLayerPainter.cpp (right):

https://codereview.chromium.org/2265123002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/paint/PaintLayerPainter.cpp:258: // "object
bounding box" the offset is already accounted for in the rootRelativeBounds.
This comment belongs in the new location.

[commit-bot: I haz the power, 2016-08-22 16:04:35]

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

[commit-bot: I haz the power, 2016-08-22 16:04:36]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[fs, 2016-08-22 16:23:32]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[fs, 2016-08-22 16:23:54]

https://codereview.chromium.org/2265123002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/paint/PaintLayerPainter.cpp (right):

https://codereview.chromium.org/2265123002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/paint/PaintLayerPainter.cpp:258: // "object
bounding box" the offset is already accounted for in the rootRelativeBounds.
On 2016/08/22 at 16:02:58, Stephen Chennney wrote:
> This comment belongs in the new location.

Moved (w/ minor edits.)

[commit-bot: I haz the power, 2016-08-22 16:24:02]

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

[fs, 2016-08-22 19:57:38]

The CQ bit was unchecked by fs@opera.com

[fs, 2016-08-22 19:57:45]

The CQ bit was checked by fs@opera.com

[fs, 2016-08-22 19:57:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
schenney@chromium.org
Link to the patchset: https://codereview.chromium.org/2265123002/#ps20001
(title: "Move comment")

[commit-bot: I haz the power, 2016-08-22 19:58:24]

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-08-22 20:04:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-22 20:06:08]

Description was changed from

==========
Refactor ClipPathHelper in PaintLayerPainter.cpp for reuse

Separate out client concerns and move the helper class to a new file and
rename it to ClipPathClipper. This will allow sharing this code between
the PaintLayerPainter and SVGPaintContext code-paths.

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

to

==========
Refactor ClipPathHelper in PaintLayerPainter.cpp for reuse

Separate out client concerns and move the helper class to a new file and
rename it to ClipPathClipper. This will allow sharing this code between
the PaintLayerPainter and SVGPaintContext code-paths.

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

Committed: https://crrev.com/66574fefebe275a0d8346e741730dfbb638721c5
Cr-Commit-Position: refs/heads/master@{#413518}
==========

[commit-bot: I haz the power, 2016-08-22 20:06:09]

Patchset 2 (id:??) landed as
https://crrev.com/66574fefebe275a0d8346e741730dfbb638721c5
Cr-Commit-Position: refs/heads/master@{#413518}