Clean up Ganesh path render a little bit

Clean up Ganesh path render a little bit

I'm not sure if this is an improvement but this is an attempt to start moving the handling of pathEffects up the call stack (ideally to SkGpuDevice).

Note that the duplicate path sniffing in GrDrawContext::drawPath is still useful since it can be invoked if a path has to be explicitly drawn for a mask filter.

Committed: https://skia.googlesource.com/skia/+/0e7029eaef917d314475dcc9df9df174af88de52

[robertphillips, 2015-11-25 20:16:18]

Description was changed from

==========
Clean up Ganesh path render a little bit
==========

to

==========
Clean up Ganesh path render a little bit

I'm not sure if this is an improvement but this is an attempt to start moving
the handling of pathEffects up the call stack (ideally to SkGpuDevice).

Note that the duplicate path sniffing in GrDrawContext::drawPath is still useful
since it can be invoked if a path has to be explicitly drawn for a mask filter.
==========

[robertphillips, 2015-11-25 20:16:18]

robertphillips@google.com changed reviewers:
+ jvanverth@google.com

[jvanverth1, 2015-11-25 20:32:46]

lgtm

[robertphillips, 2015-11-25 20:37:42]

The CQ bit was checked by robertphillips@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-25 20:37:50]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1472333003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1472333003/20001

[robertphillips, 2015-11-25 20:51:57]

The CQ bit was checked by robertphillips@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-25 20:52:09]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1472333003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1472333003/40001

[commit-bot: I haz the power, 2015-11-25 22:36:04]

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

[commit-bot: I haz the power, 2015-11-25 22:36:05]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2015-11-30 13:31:19]

The CQ bit was checked by robertphillips@google.com

[robertphillips, 2015-11-30 13:31:19]

The patchset sent to the CQ was uploaded after l-g-t-m from jvanverth@google.com
Link to the patchset: https://codereview.chromium.org/1472333003/#ps40001
(title: "Ooops")

[commit-bot: I haz the power, 2015-11-30 13:31:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1472333003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1472333003/40001

[commit-bot: I haz the power, 2015-11-30 13:45:09]

Description was changed from

==========
Clean up Ganesh path render a little bit

I'm not sure if this is an improvement but this is an attempt to start moving
the handling of pathEffects up the call stack (ideally to SkGpuDevice).

Note that the duplicate path sniffing in GrDrawContext::drawPath is still useful
since it can be invoked if a path has to be explicitly drawn for a mask filter.
==========

to

==========
Clean up Ganesh path render a little bit

I'm not sure if this is an improvement but this is an attempt to start moving
the handling of pathEffects up the call stack (ideally to SkGpuDevice).

Note that the duplicate path sniffing in GrDrawContext::drawPath is still useful
since it can be invoked if a path has to be explicitly drawn for a mask filter.

Committed:
https://skia.googlesource.com/skia/+/0e7029eaef917d314475dcc9df9df174af88de52
==========

[commit-bot: I haz the power, 2015-11-30 13:45:10]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/0e7029eaef917d314475dcc9df9df174af88de52