switch maskfilters to sk_sp

src/animator/SkDrawPaint.cpp

Index: src/animator/SkDrawPaint.cpp
diff --git a/src/animator/SkDrawPaint.cpp b/src/animator/SkDrawPaint.cpp
index fc1a4132440da91ff0830027ae3a6f6b50030461..1336ea2dc429b0039b0ff1fcea3f745bef96b9e4 100644
--- a/src/animator/SkDrawPaint.cpp
+++ b/src/animator/SkDrawPaint.cpp
@@ -229,7 +229,7 @@ void SkDrawPaint::setupPaint(SkPaint* paint) const {
     if (maskFilter == nullptr)
         paint->setMaskFilter(nullptr);
     else if (maskFilter != (SkDrawMaskFilter*) -1)
-        SkSafeUnref(paint->setMaskFilter(maskFilter->getMaskFilter()));
+        paint->setMaskFilter(sk_sp<SkMaskFilter>(maskFilter->getMaskFilter()));

[tomhudson, 2016/04/04 14:18:58]

getMaskFilter() is always returning a pointer that needs to be adopted like
this? (e.g. one already ref'd?)

[reed1, 2016/04/04 16:34:47]

I think so. I think the old code was leaking.

     if (pathEffect == nullptr)
         paint->setPathEffect(nullptr);
     else if (pathEffect != (SkDrawPathEffect*) -1)