tree 04a5e0e069115f58baedad1c3e106c9b42b01613
parent 89515a5249920f95e54d38f8b33fbc383ae16782
author Kevin Lubick <kjlubick@google.com> 1533655812 -0400
committer Kevin Lubick <kjlubick@google.com> 1533660118 +0000

[PathKit] Made some APIs return null if things failed

This makes our calls to emscripten::val a bit more consistent.

Adds in the macro SkPathOrVal to self-document where it's really
a SkPath we are returning, but C++ doesn't realize SkPath and
emscripten::val::null() can be the same type.  Casting SkPath
via emscripten::val() is basically a no-op, since Emscripten bind
seems to be doing it under the hood anyway.

No functional changes, except when there would be a failure,
methods will return null instead of an empty SkPath.

Bug: skia:8216
Change-Id: I1fff620d5aa50ec4a57f76e706d8d005ea26605f
Reviewed-on: https://skia-review.googlesource.com/145728
Reviewed-by: Cary Clark <caryclark@google.com>
Reviewed-by: Florin Malita <fmalita@chromium.org>