Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(100)

Side by Side Diff: include/effects/Sk1DPathEffect.h

Issue 1719993002: fix misc asserts and checks found by fuzzer (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: hide the creation of the typeface as well Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « include/core/SkPathEffect.h ('k') | include/effects/SkAlphaThresholdFilter.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2006 The Android Open Source Project 2 * Copyright 2006 The Android Open Source Project
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #ifndef Sk1DPathEffect_DEFINED 8 #ifndef Sk1DPathEffect_DEFINED
9 #define Sk1DPathEffect_DEFINED 9 #define Sk1DPathEffect_DEFINED
10 10
(...skipping 28 matching lines...) Expand all
39 typedef SkPathEffect INHERITED; 39 typedef SkPathEffect INHERITED;
40 }; 40 };
41 41
42 class SK_API SkPath1DPathEffect : public Sk1DPathEffect { 42 class SK_API SkPath1DPathEffect : public Sk1DPathEffect {
43 public: 43 public:
44 enum Style { 44 enum Style {
45 kTranslate_Style, // translate the shape to each position 45 kTranslate_Style, // translate the shape to each position
46 kRotate_Style, // rotate the shape about its center 46 kRotate_Style, // rotate the shape about its center
47 kMorph_Style, // transform each point, and turn lines into curves 47 kMorph_Style, // transform each point, and turn lines into curves
48 48
49 kStyleCount 49 kLastEnum_Style = kMorph_Style,
50 }; 50 };
51 51
52 /** Dash by replicating the specified path. 52 /** Dash by replicating the specified path.
53 @param path The path to replicate (dash) 53 @param path The path to replicate (dash)
54 @param advance The space between instances of path 54 @param advance The space between instances of path
55 @param phase distance (mod advance) along path for its initial position 55 @param phase distance (mod advance) along path for its initial position
56 @param style how to transform path at each point (based on the current 56 @param style how to transform path at each point (based on the current
57 position and tangent) 57 position and tangent)
58 */ 58 */
59 static SkPathEffect* Create(const SkPath& path, SkScalar advance, SkScalar p hase, Style style) { 59 static SkPathEffect* Create(const SkPath& path, SkScalar advance, SkScalar p hase, Style);
60 return new SkPath1DPathEffect(path, advance, phase, style);
61 }
62 60
63 virtual bool filterPath(SkPath*, const SkPath&, 61 virtual bool filterPath(SkPath*, const SkPath&,
64 SkStrokeRec*, const SkRect*) const override; 62 SkStrokeRec*, const SkRect*) const override;
65 63
66 SK_TO_STRING_OVERRIDE() 64 SK_TO_STRING_OVERRIDE()
67 SK_DECLARE_PUBLIC_FLATTENABLE_DESERIALIZATION_PROCS(SkPath1DPathEffect) 65 SK_DECLARE_PUBLIC_FLATTENABLE_DESERIALIZATION_PROCS(SkPath1DPathEffect)
68 66
69 protected: 67 protected:
70 SkPath1DPathEffect(const SkPath& path, SkScalar advance, SkScalar phase, Sty le); 68 SkPath1DPathEffect(const SkPath& path, SkScalar advance, SkScalar phase, Sty le);
71 void flatten(SkWriteBuffer&) const override; 69 void flatten(SkWriteBuffer&) const override;
72 70
73 // overrides from Sk1DPathEffect 71 // overrides from Sk1DPathEffect
74 SkScalar begin(SkScalar contourLength) const override; 72 SkScalar begin(SkScalar contourLength) const override;
75 SkScalar next(SkPath*, SkScalar, SkPathMeasure&) const override; 73 SkScalar next(SkPath*, SkScalar, SkPathMeasure&) const override;
76 74
77 private: 75 private:
78 SkPath fPath; // copied from constructor 76 SkPath fPath; // copied from constructor
79 SkScalar fAdvance; // copied from constructor 77 SkScalar fAdvance; // copied from constructor
80 SkScalar fInitialOffset; // computed from phase 78 SkScalar fInitialOffset; // computed from phase
81 Style fStyle; // copied from constructor 79 Style fStyle; // copied from constructor
82 80
83 typedef Sk1DPathEffect INHERITED; 81 typedef Sk1DPathEffect INHERITED;
84 }; 82 };
85 83
86 #endif 84 #endif
OLDNEW
« no previous file with comments | « include/core/SkPathEffect.h ('k') | include/effects/SkAlphaThresholdFilter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698