From d1227a7417922ce26252d55815d0d1e98f0eb070 Mon Sep 17 00:00:00 2001 From: msarett Date: Wed, 18 May 2016 06:23:57 -0700 Subject: Delete SkBitmapRegionCanvas This was an approach we considered for implementing Android's BitmapRegionDecoder. It was useful for testing and comparison, but now is no longer worth maintaining and testing. The approach to subset/scaled decodes (clipped decode, then scale) may be worth reconsidering at some point. BUG=skia:5307 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1990543002 Review-Url: https://codereview.chromium.org/1990543002 --- bench/BitmapRegionDecoderBench.h | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) (limited to 'bench/BitmapRegionDecoderBench.h') diff --git a/bench/BitmapRegionDecoderBench.h b/bench/BitmapRegionDecoderBench.h index 7c331aee31..69588b24c3 100644 --- a/bench/BitmapRegionDecoderBench.h +++ b/bench/BitmapRegionDecoderBench.h @@ -18,16 +18,13 @@ /** * Benchmark Android's BitmapRegionDecoder for a particular colorType, sampleSize, and subset. * - * fStrategy determines which of various implementations is to be used. - * * nanobench.cpp handles creating benchmarks for interesting scaled subsets. We strive to test * on real use cases. */ class BitmapRegionDecoderBench : public Benchmark { public: // Calls encoded->ref() - BitmapRegionDecoderBench(const char* basename, SkData* encoded, - SkBitmapRegionDecoder::Strategy strategy, SkColorType colorType, + BitmapRegionDecoderBench(const char* basename, SkData* encoded, SkColorType colorType, uint32_t sampleSize, const SkIRect& subset); protected: @@ -40,7 +37,6 @@ private: SkString fName; SkAutoTDelete fBRD; SkAutoTUnref fData; - const SkBitmapRegionDecoder::Strategy fStrategy; const SkColorType fColorType; const uint32_t fSampleSize; const SkIRect fSubset; -- cgit v1.2.3