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

Side by Side Diff: src/effects/SkDisplacementMapEffect.cpp

Issue 23021015: Initial error handling code (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: Adapting code to sk_once changes Created 7 years, 2 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 | Annotate | Revision Log
« no previous file with comments | « src/effects/SkColorMatrixFilter.cpp ('k') | src/effects/SkDropShadowImageFilter.cpp » ('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 #include "SkDisplacementMapEffect.h" 8 #include "SkDisplacementMapEffect.h"
9 #include "SkFlattenableBuffers.h" 9 #include "SkFlattenableBuffers.h"
10 #include "SkUnPreMultiply.h" 10 #include "SkUnPreMultiply.h"
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
119 case SkDisplacementMapEffect::kA_ChannelSelectorType: 119 case SkDisplacementMapEffect::kA_ChannelSelectorType:
120 computeDisplacement<SkDisplacementMapEffect::kA_ChannelSelectorType>( 120 computeDisplacement<SkDisplacementMapEffect::kA_ChannelSelectorType>(
121 yChannelSelector, scale, dst, displ, src, bounds); 121 yChannelSelector, scale, dst, displ, src, bounds);
122 break; 122 break;
123 case SkDisplacementMapEffect::kUnknown_ChannelSelectorType: 123 case SkDisplacementMapEffect::kUnknown_ChannelSelectorType:
124 default: 124 default:
125 SkDEBUGFAIL("Unknown X channel selector"); 125 SkDEBUGFAIL("Unknown X channel selector");
126 } 126 }
127 } 127 }
128 128
129 bool channel_selector_type_is_valid(SkDisplacementMapEffect::ChannelSelectorType cst) {
130 switch (cst) {
131 case SkDisplacementMapEffect::kUnknown_ChannelSelectorType:
132 case SkDisplacementMapEffect::kR_ChannelSelectorType:
133 case SkDisplacementMapEffect::kG_ChannelSelectorType:
134 case SkDisplacementMapEffect::kB_ChannelSelectorType:
135 case SkDisplacementMapEffect::kA_ChannelSelectorType:
136 return true;
137 default:
138 break;
139 }
140 return false;
141 }
142
129 } // end namespace 143 } // end namespace
130 144
131 /////////////////////////////////////////////////////////////////////////////// 145 ///////////////////////////////////////////////////////////////////////////////
132 146
133 SkDisplacementMapEffect::SkDisplacementMapEffect(ChannelSelectorType xChannelSel ector, 147 SkDisplacementMapEffect::SkDisplacementMapEffect(ChannelSelectorType xChannelSel ector,
134 ChannelSelectorType yChannelSel ector, 148 ChannelSelectorType yChannelSel ector,
135 SkScalar scale, 149 SkScalar scale,
136 SkImageFilter* displacement, 150 SkImageFilter* displacement,
137 SkImageFilter* color, 151 SkImageFilter* color,
138 const CropRect* cropRect) 152 const CropRect* cropRect)
139 : INHERITED(displacement, color, cropRect) 153 : INHERITED(displacement, color, cropRect)
140 , fXChannelSelector(xChannelSelector) 154 , fXChannelSelector(xChannelSelector)
141 , fYChannelSelector(yChannelSelector) 155 , fYChannelSelector(yChannelSelector)
142 , fScale(scale) 156 , fScale(scale)
143 { 157 {
144 } 158 }
145 159
146 SkDisplacementMapEffect::~SkDisplacementMapEffect() { 160 SkDisplacementMapEffect::~SkDisplacementMapEffect() {
147 } 161 }
148 162
149 SkDisplacementMapEffect::SkDisplacementMapEffect(SkFlattenableReadBuffer& buffer ) 163 SkDisplacementMapEffect::SkDisplacementMapEffect(SkFlattenableReadBuffer& buffer )
150 : INHERITED(buffer) 164 : INHERITED(buffer)
151 { 165 {
152 fXChannelSelector = (SkDisplacementMapEffect::ChannelSelectorType) buffer.re adInt(); 166 fXChannelSelector = (SkDisplacementMapEffect::ChannelSelectorType) buffer.re adInt();
153 fYChannelSelector = (SkDisplacementMapEffect::ChannelSelectorType) buffer.re adInt(); 167 fYChannelSelector = (SkDisplacementMapEffect::ChannelSelectorType) buffer.re adInt();
154 fScale = buffer.readScalar(); 168 fScale = buffer.readScalar();
169 buffer.validate(channel_selector_type_is_valid(fXChannelSelector) &&
170 channel_selector_type_is_valid(fYChannelSelector) &&
171 SkScalarIsFinite(fScale));
155 } 172 }
156 173
157 void SkDisplacementMapEffect::flatten(SkFlattenableWriteBuffer& buffer) const { 174 void SkDisplacementMapEffect::flatten(SkFlattenableWriteBuffer& buffer) const {
158 this->INHERITED::flatten(buffer); 175 this->INHERITED::flatten(buffer);
159 buffer.writeInt((int) fXChannelSelector); 176 buffer.writeInt((int) fXChannelSelector);
160 buffer.writeInt((int) fYChannelSelector); 177 buffer.writeInt((int) fYChannelSelector);
161 buffer.writeScalar(fScale); 178 buffer.writeScalar(fScale);
162 } 179 }
163 180
164 bool SkDisplacementMapEffect::onFilterImage(Proxy* proxy, 181 bool SkDisplacementMapEffect::onFilterImage(Proxy* proxy,
(...skipping 342 matching lines...) Expand 10 before | Expand all | Expand 10 after
507 const GrGLCaps&) { 524 const GrGLCaps&) {
508 const GrDisplacementMapEffect& displacementMap = 525 const GrDisplacementMapEffect& displacementMap =
509 drawEffect.castEffect<GrDisplacementMapEffect>(); 526 drawEffect.castEffect<GrDisplacementMapEffect>();
510 527
511 EffectKey xKey = displacementMap.xChannelSelector(); 528 EffectKey xKey = displacementMap.xChannelSelector();
512 EffectKey yKey = displacementMap.yChannelSelector() << SkDisplacementMapEffe ct::kKeyBits; 529 EffectKey yKey = displacementMap.yChannelSelector() << SkDisplacementMapEffe ct::kKeyBits;
513 530
514 return xKey | yKey; 531 return xKey | yKey;
515 } 532 }
516 #endif 533 #endif
OLDNEW
« no previous file with comments | « src/effects/SkColorMatrixFilter.cpp ('k') | src/effects/SkDropShadowImageFilter.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698