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

Side by Side Diff: include/gpu/GrCoordTransform.h

Issue 226183018: SkNonCopyable should be used with private inheritance. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: name resolution Created 6 years, 8 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/gpu/GrContextFactory.h ('k') | include/gpu/GrEffectUnitTest.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 2013 Google Inc. 2 * Copyright 2013 Google Inc.
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 GrCoordTransform_DEFINED 8 #ifndef GrCoordTransform_DEFINED
9 #define GrCoordTransform_DEFINED 9 #define GrCoordTransform_DEFINED
10 10
(...skipping 21 matching lines...) Expand all
32 * usually not the coordinates a GrEffect wants. 32 * usually not the coordinates a GrEffect wants.
33 */ 33 */
34 kPosition_GrCoordSet 34 kPosition_GrCoordSet
35 }; 35 };
36 36
37 /** 37 /**
38 * A class representing a linear transformation from one of the built-in coordin ate sets (local or 38 * A class representing a linear transformation from one of the built-in coordin ate sets (local or
39 * position). GrEffects just define these transformations, and the framework doe s the rest of the 39 * position). GrEffects just define these transformations, and the framework doe s the rest of the
40 * work to make the transformed coordinates available in their fragment shader. 40 * work to make the transformed coordinates available in their fragment shader.
41 */ 41 */
42 class GrCoordTransform : public SkNoncopyable { 42 class GrCoordTransform : SkNoncopyable {
43 public: 43 public:
44 GrCoordTransform() { SkDEBUGCODE(fInEffect = false); } 44 GrCoordTransform() { SkDEBUGCODE(fInEffect = false); }
45 45
46 /** 46 /**
47 * Create a transformation that maps [0, 1] to a texture's boundaries. 47 * Create a transformation that maps [0, 1] to a texture's boundaries.
48 */ 48 */
49 GrCoordTransform(GrCoordSet sourceCoords, const GrTexture* texture) { 49 GrCoordTransform(GrCoordSet sourceCoords, const GrTexture* texture) {
50 SkDEBUGCODE(fInEffect = false); 50 SkDEBUGCODE(fInEffect = false);
51 this->reset(sourceCoords, texture); 51 this->reset(sourceCoords, texture);
52 } 52 }
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
110 110
111 #ifdef SK_DEBUG 111 #ifdef SK_DEBUG
112 public: 112 public:
113 void setInEffect() const { fInEffect = true; } 113 void setInEffect() const { fInEffect = true; }
114 private: 114 private:
115 mutable bool fInEffect; 115 mutable bool fInEffect;
116 #endif 116 #endif
117 }; 117 };
118 118
119 #endif 119 #endif
OLDNEW
« no previous file with comments | « include/gpu/GrContextFactory.h ('k') | include/gpu/GrEffectUnitTest.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698