diff options
author | 2016-03-22 07:35:17 -0700 | |
---|---|---|
committer | 2016-03-22 07:35:17 -0700 | |
commit | 1eb81db650d31f50be67b12d60c4f9e7dd08432f (patch) | |
tree | 58a5fd565f06a398192be170009fefa2a12a1ee2 /bench/ColorCubeBench.cpp | |
parent | ee451cf09de31267c93b64e1fdac7c823cee7e85 (diff) |
Revert of switch colorfilters to sk_sp (patchset #11 id:200001 of https://codereview.chromium.org/1822623002/ )
Reason for revert:
need to fix unguarded makeWithFilter
Original issue's description:
> switch colorfilters to sk_sp
>
> BUG=skia:
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1822623002
>
> Committed: https://skia.googlesource.com/skia/+/f809d7687a4fb7b88b651b046da2bc0035d6aa09
TBR=fmalita@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1820303002
Diffstat (limited to 'bench/ColorCubeBench.cpp')
-rw-r--r-- | bench/ColorCubeBench.cpp | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/bench/ColorCubeBench.cpp b/bench/ColorCubeBench.cpp index 6a7774012d..9f6bd001ff 100644 --- a/bench/ColorCubeBench.cpp +++ b/bench/ColorCubeBench.cpp @@ -13,14 +13,20 @@ class ColorCubeBench : public Benchmark { SkISize fSize; int fCubeDimension; - sk_sp<SkData> fCubeData; + SkData* fCubeData; SkBitmap fBitmap; public: - ColorCubeBench() : fCubeDimension(0) { + ColorCubeBench() + : fCubeDimension(0) + , fCubeData(nullptr) { fSize = SkISize::Make(2880, 1800); // 2014 Macbook Pro resolution } + ~ColorCubeBench() { + SkSafeUnref(fCubeData); + } + protected: const char* onGetName() override { return "colorcube"; @@ -65,7 +71,7 @@ private: void makeCubeData() { fCubeDimension = 32; - fCubeData = SkData::MakeUninitialized(sizeof(SkColor) * + fCubeData = SkData::NewUninitialized(sizeof(SkColor) * fCubeDimension * fCubeDimension * fCubeDimension); SkColor* pixels = (SkColor*)(fCubeData->writable_data()); SkAutoTMalloc<uint8_t> lutMemory(fCubeDimension); @@ -89,7 +95,9 @@ private: void test(int loops, SkCanvas* canvas) { SkPaint paint; for (int i = 0; i < loops; i++) { - paint.setColorFilter(SkColorCubeFilter::Make(fCubeData, fCubeDimension)); + SkAutoTUnref<SkColorFilter> colorCube( + SkColorCubeFilter::Create(fCubeData, fCubeDimension)); + paint.setColorFilter(colorCube); canvas->drawBitmap(fBitmap, 0, 0, &paint); } } |