From 20efb41dbbbe1f9fa9cd8f3fd8bd84840b79dbbb Mon Sep 17 00:00:00 2001 From: mtklein Date: Tue, 12 Apr 2016 15:52:52 -0700 Subject: distinguish distinct matrixconvolution benchmarks Today they all show as "matrixconvolution", and we probably only log one. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1886523002 Review URL: https://codereview.chromium.org/1886523002 --- bench/MatrixConvolutionBench.cpp | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) (limited to 'bench/MatrixConvolutionBench.cpp') diff --git a/bench/MatrixConvolutionBench.cpp b/bench/MatrixConvolutionBench.cpp index da61891e73..3a43de863d 100644 --- a/bench/MatrixConvolutionBench.cpp +++ b/bench/MatrixConvolutionBench.cpp @@ -11,10 +11,21 @@ #include "SkRandom.h" #include "SkString.h" +static const char* name(SkMatrixConvolutionImageFilter::TileMode mode) { + switch (mode) { + case SkMatrixConvolutionImageFilter::kClamp_TileMode: return "clamp"; + case SkMatrixConvolutionImageFilter::kRepeat_TileMode: return "repeat"; + case SkMatrixConvolutionImageFilter::kClampToBlack_TileMode: return "clampToBlack"; + } + return "oops"; +} + class MatrixConvolutionBench : public Benchmark { public: MatrixConvolutionBench(SkMatrixConvolutionImageFilter::TileMode tileMode, bool convolveAlpha) - : fName("matrixconvolution") { + : fName(SkStringPrintf("matrixconvolution_%s%s", + name(tileMode), + convolveAlpha ? "" : "_noConvolveAlpha")) { SkISize kernelSize = SkISize::Make(3, 3); SkScalar kernel[9] = { SkIntToScalar( 1), SkIntToScalar( 1), SkIntToScalar( 1), @@ -23,7 +34,7 @@ public: }; SkScalar gain = 0.3f, bias = SkIntToScalar(100); SkIPoint kernelOffset = SkIPoint::Make(1, 1); - fFilter = SkMatrixConvolutionImageFilter::Make(kernelSize, kernel, gain, bias, + fFilter = SkMatrixConvolutionImageFilter::Make(kernelSize, kernel, gain, bias, kernelOffset, tileMode, convolveAlpha, nullptr); } -- cgit v1.2.3