Use child processors to implement compose color filter.

src/gpu/SkGr.cpp

Index: src/gpu/SkGr.cpp
diff --git a/src/gpu/SkGr.cpp b/src/gpu/SkGr.cpp
index e0892d8ab974e88db4034b429cba09727ab8cb93..cabac8601d1df50b5b28d20a1a9e6bbb3499cb5d 100644
--- a/src/gpu/SkGr.cpp
+++ b/src/gpu/SkGr.cpp
@@ -812,12 +812,10 @@ static inline bool skpaint_to_grpaint_impl(GrContext* context,
         if (applyColorFilterToPaintColor) {
             grPaint->setColor(SkColorToPremulGrColor(colorFilter->filterColor(skPaint.getColor())));
         } else {
-            SkTDArray<const GrFragmentProcessor*> array;
-            if (colorFilter->asFragmentProcessors(context, grPaint->getProcessorDataManager(),
-                                                  &array)) {
-                for (int i = 0; i < array.count(); ++i) {
-                    grPaint->addColorFragmentProcessor(array[i])->unref();
-                }
+            SkAutoTUnref<const GrFragmentProcessor> cfFP(colorFilter->asFragmentProcessor(context,
+                grPaint->getProcessorDataManager()));

[egdaniel, 2015/09/29 13:23:52]

Is there any cleaner way to alight this so that it's obvious the second param is
part of asFragProc(...)

[bsalomon, 2015/09/29 13:31:43]

Done.

+            if (cfFP) {
+                grPaint->addColorFragmentProcessor(cfFP);
             } else {
                 return false;
             }