aboutsummaryrefslogtreecommitdiffhomepage
path: root/bench/DashBench.cpp
diff options
context:
space:
mode:
authorGravatar reed <reed@google.com>2016-03-18 10:17:27 -0700
committerGravatar Commit bot <commit-bot@chromium.org>2016-03-18 10:17:27 -0700
commitf28ad894272018fd2855e3f77ea1236ea0cce1c0 (patch)
tree25dfb6f13e3bbbde1c9d7af7b41ea39b2df78dd4 /bench/DashBench.cpp
parent9fbee18f691a0afed1e38a851048ce06063505ed (diff)
Revert of switch patheffects over to sk_sp (patchset #5 id:80001 of https://codereview.chromium.org/1813553005/ )
Reason for revert: some build breaks, possibly related to paint having to know what a patheffect is Original issue's description: > switch patheffects over to sk_sp > > BUG=skia: > GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1813553005 > > Committed: https://skia.googlesource.com/skia/+/9fbee18f691a0afed1e38a851048ce06063505ed TBR=caryclark@google.com # 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/1817543002
Diffstat (limited to 'bench/DashBench.cpp')
-rw-r--r--bench/DashBench.cpp25
1 files changed, 14 insertions, 11 deletions
diff --git a/bench/DashBench.cpp b/bench/DashBench.cpp
index 43db54c092..e3d8364013 100644
--- a/bench/DashBench.cpp
+++ b/bench/DashBench.cpp
@@ -72,7 +72,9 @@ protected:
SkPath path;
this->makePath(&path);
- paint.setPathEffect(SkDashPathEffect::Make(fIntervals.begin(), fIntervals.count(), 0));
+ SkAutoTUnref<SkPathEffect> effect(SkDashPathEffect::Create(fIntervals.begin(),
+ fIntervals.count(), 0));
+ paint.setPathEffect(effect);
if (fDoClip) {
SkRect r = path.getBounds();
@@ -177,7 +179,7 @@ static void make_cubic(SkPath* path) {
class MakeDashBench : public Benchmark {
SkString fName;
SkPath fPath;
- sk_sp<SkPathEffect> fPE;
+ SkAutoTUnref<SkPathEffect> fPE;
public:
MakeDashBench(void (*proc)(SkPath*), const char name[]) {
@@ -185,7 +187,7 @@ public:
proc(&fPath);
SkScalar vals[] = { SkIntToScalar(4), SkIntToScalar(4) };
- fPE = SkDashPathEffect::Make(vals, 2, 0);
+ fPE.reset(SkDashPathEffect::Create(vals, 2, 0));
}
protected:
@@ -214,7 +216,7 @@ class DashLineBench : public Benchmark {
SkString fName;
SkScalar fStrokeWidth;
bool fIsRound;
- sk_sp<SkPathEffect> fPE;
+ SkAutoTUnref<SkPathEffect> fPE;
public:
DashLineBench(SkScalar width, bool isRound) {
@@ -223,7 +225,7 @@ public:
fIsRound = isRound;
SkScalar vals[] = { SK_Scalar1, SK_Scalar1 };
- fPE = SkDashPathEffect::Make(vals, 2, 0);
+ fPE.reset(SkDashPathEffect::Create(vals, 2, 0));
}
protected:
@@ -252,7 +254,7 @@ class DrawPointsDashingBench : public Benchmark {
int fStrokeWidth;
bool fDoAA;
- sk_sp<SkPathEffect> fPathEffect;
+ SkAutoTUnref<SkPathEffect> fPathEffect;
public:
DrawPointsDashingBench(int dashLength, int strokeWidth, bool doAA)
@@ -262,7 +264,7 @@ public:
fDoAA = doAA;
SkScalar vals[] = { SkIntToScalar(dashLength), SkIntToScalar(dashLength) };
- fPathEffect = SkDashPathEffect::Make(vals, 2, SK_Scalar1);
+ fPathEffect.reset(SkDashPathEffect::Create(vals, 2, SK_Scalar1));
}
protected:
@@ -299,7 +301,7 @@ class GiantDashBench : public Benchmark {
SkString fName;
SkScalar fStrokeWidth;
SkPoint fPts[2];
- sk_sp<SkPathEffect> fPathEffect;
+ SkAutoTUnref<SkPathEffect> fPathEffect;
public:
enum LineType {
@@ -322,7 +324,8 @@ public:
// deliberately pick intervals that won't be caught by asPoints(), so
// we can test the filterPath code-path.
const SkScalar intervals[] = { 20, 10, 10, 10 };
- fPathEffect = SkDashPathEffect::Make(intervals, SK_ARRAY_COUNT(intervals), 0);
+ fPathEffect.reset(SkDashPathEffect::Create(intervals,
+ SK_ARRAY_COUNT(intervals), 0));
SkScalar cx = 640 / 2; // center X
SkScalar cy = 480 / 2; // center Y
@@ -378,7 +381,7 @@ class DashGridBench : public Benchmark {
int fStrokeWidth;
bool fDoAA;
- sk_sp<SkPathEffect> fPathEffect;
+ SkAutoTUnref<SkPathEffect> fPathEffect;
public:
DashGridBench(int dashLength, int strokeWidth, bool doAA) {
@@ -387,7 +390,7 @@ public:
fDoAA = doAA;
SkScalar vals[] = { SkIntToScalar(dashLength), SkIntToScalar(dashLength) };
- fPathEffect = SkDashPathEffect::Make(vals, 2, SK_Scalar1);
+ fPathEffect.reset(SkDashPathEffect::Create(vals, 2, SK_Scalar1));
}
protected: