Implement C path functions

Implement C path functions

Mainly just to familiarize myself with this code.

Committed: https://skia.googlesource.com/skia/+/a624e87e8e10fd9dc3a34e498dd30804ceee5228

[robertphillips, 2014-10-07 18:19:58]

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

[robertphillips, 2014-10-07 18:20:10]

https://codereview.chromium.org/633183002/diff/1/experimental/sk_surface.cpp
File experimental/sk_surface.cpp (right):

https://codereview.chromium.org/633183002/diff/1/experimental/sk_surface.cpp#...
experimental/sk_surface.cpp:190: 
It seems a bit funky to call this delete but unref inside.

[robertphillips, 2014-10-08 11:55:36]

ping

[reed1, 2014-10-08 12:12:46]

lgtm w/ questions

https://codereview.chromium.org/633183002/diff/1/experimental/sk_surface.cpp
File experimental/sk_surface.cpp (right):

https://codereview.chromium.org/633183002/diff/1/experimental/sk_surface.cpp#...
experimental/sk_surface.cpp:34: static const SkPath* as_path(const sk_path_t*
cpath) {
do we need this one? if so, can it take/return a ref (see const paint)?

https://codereview.chromium.org/633183002/diff/1/experimental/sk_surface.cpp#...
experimental/sk_surface.cpp:190: 
On 2014/10/07 18:20:09, robertphillips wrote:
> It seems a bit funky to call this delete but unref inside.

Yea, trying to be conservative in exposing when something is refcounted under
the hood -- we may have to expose that, but just trying out a more conservative
model.

[robertphillips, 2014-10-08 12:55:52]

The CQ bit was checked by robertphillips@google.com

[robertphillips, 2014-10-08 12:56:18]

https://codereview.chromium.org/633183002/diff/1/experimental/sk_surface.cpp
File experimental/sk_surface.cpp (right):

https://codereview.chromium.org/633183002/diff/1/experimental/sk_surface.cpp#...
experimental/sk_surface.cpp:34: static const SkPath* as_path(const sk_path_t*
cpath) {
Apparently it is not needed, I've removed it.

[commit-bot: I haz the power, 2014-10-08 12:56:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/633183002/20001

[commit-bot: I haz the power, 2014-10-08 13:04:42]

Committed patchset #2 (id:20001) as a624e87e8e10fd9dc3a34e498dd30804ceee5228