diff options
author | reed <reed@google.com> | 2016-03-18 10:17:27 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-03-18 10:17:27 -0700 |
commit | f28ad894272018fd2855e3f77ea1236ea0cce1c0 (patch) | |
tree | 25dfb6f13e3bbbde1c9d7af7b41ea39b2df78dd4 /bench/DashBench.cpp | |
parent | 9fbee18f691a0afed1e38a851048ce06063505ed (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.cpp | 25 |
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: |