Factory methods for heap-allocated SkImageFilter objects.

bench/MatrixConvolutionBench.cpp

Index: bench/MatrixConvolutionBench.cpp
diff --git a/bench/MatrixConvolutionBench.cpp b/bench/MatrixConvolutionBench.cpp
index 1acc0cb01a3c31c92101570f5e9df14660d5fe89..82529737c1bbd344f1206ec279ecdfd92e5469e4 100644
--- a/bench/MatrixConvolutionBench.cpp
+++ b/bench/MatrixConvolutionBench.cpp
@@ -23,7 +23,7 @@ public:
         };
         SkScalar gain = 0.3f, bias = SkIntToScalar(100);
         SkIPoint target = SkIPoint::Make(1, 1);

[Stephen White, 2014/03/10 20:58:48]

Nit: should probably rename this to kernelOffset too.

[Dominik Grewe, 2014/03/12 10:05:57]

Thanks for spotting this! I've fixed it (and two other places) in a follow-up
patch: https://codereview.chromium.org/197013003

-        fFilter = new SkMatrixConvolutionImageFilter(kernelSize, kernel, gain, bias, target, tileMode, convolveAlpha);
+        fFilter = SkMatrixConvolutionImageFilter::Create(kernelSize, kernel, gain, bias, target, tileMode, convolveAlpha);
     }
 
     ~MatrixConvolutionBench() {