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

Side by Side Diff: src/gpu/GrSWMaskHelper.cpp

Issue 16952006: Replace fixed-size array of effect stages in GrDrawState with two appendable arrays, one for color,… (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: fix comments Created 7 years, 6 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/gpu/GrOvalRenderer.cpp ('k') | src/gpu/GrTextContext.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 2012 Google Inc. 2 * Copyright 2012 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 "GrSWMaskHelper.h" 8 #include "GrSWMaskHelper.h"
9 #include "GrDrawState.h" 9 #include "GrDrawState.h"
10 #include "GrGpu.h" 10 #include "GrGpu.h"
(...skipping 173 matching lines...) Expand 10 before | Expand all | Expand 10 after
184 184
185 void GrSWMaskHelper::DrawToTargetWithPathMask(GrTexture* texture, 185 void GrSWMaskHelper::DrawToTargetWithPathMask(GrTexture* texture,
186 GrDrawTarget* target, 186 GrDrawTarget* target,
187 const GrIRect& rect) { 187 const GrIRect& rect) {
188 GrDrawState* drawState = target->drawState(); 188 GrDrawState* drawState = target->drawState();
189 189
190 GrDrawState::AutoViewMatrixRestore avmr; 190 GrDrawState::AutoViewMatrixRestore avmr;
191 if (!avmr.setIdentity(drawState)) { 191 if (!avmr.setIdentity(drawState)) {
192 return; 192 return;
193 } 193 }
194 GrDrawState::AutoRestoreEffects are(drawState);
194 enum { 195 enum {
195 // the SW path renderer shares this stage with glyph 196 // the SW path renderer shares this stage with glyph
196 // rendering (kGlyphMaskStage in GrTextContext) 197 // rendering (kGlyphMaskStage in GrTextContext)
197 // && edge rendering (kEdgeEffectStage in GrContext) 198 // && edge rendering (kEdgeEffectStage in GrContext)
198 kPathMaskStage = GrPaint::kTotalStages, 199 kPathMaskStage = GrPaint::kTotalStages,
199 }; 200 };
200 201
201 GrRect dstRect = GrRect::MakeLTRB( 202 GrRect dstRect = GrRect::MakeLTRB(
202 SK_Scalar1 * rect.fLeft, 203 SK_Scalar1 * rect.fLeft,
203 SK_Scalar1 * rect.fTop, 204 SK_Scalar1 * rect.fTop,
204 SK_Scalar1 * rect.fRight, 205 SK_Scalar1 * rect.fRight,
205 SK_Scalar1 * rect.fBottom); 206 SK_Scalar1 * rect.fBottom);
206 207
207 // We want to use device coords to compute the texture coordinates. We set o ur matrix to be 208 // We want to use device coords to compute the texture coordinates. We set o ur matrix to be
208 // equal to the view matrix followed by a translation so that the top-left o f the device bounds 209 // equal to the view matrix followed by a translation so that the top-left o f the device bounds
209 // maps to 0,0, and then a scaling matrix to normalized coords. We apply thi s matrix to the 210 // maps to 0,0, and then a scaling matrix to normalized coords. We apply thi s matrix to the
210 // vertex positions rather than local coords. 211 // vertex positions rather than local coords.
211 SkMatrix maskMatrix; 212 SkMatrix maskMatrix;
212 maskMatrix.setIDiv(texture->width(), texture->height()); 213 maskMatrix.setIDiv(texture->width(), texture->height());
213 maskMatrix.preTranslate(SkIntToScalar(-rect.fLeft), SkIntToScalar(-rect.fTop )); 214 maskMatrix.preTranslate(SkIntToScalar(-rect.fLeft), SkIntToScalar(-rect.fTop ));
214 maskMatrix.preConcat(drawState->getViewMatrix()); 215 maskMatrix.preConcat(drawState->getViewMatrix());
215 216
216 GrAssert(!drawState->isStageEnabled(kPathMaskStage)); 217 drawState->addCoverageEffect(
217 drawState->setEffect(kPathMaskStage,
218 GrSimpleTextureEffect::Create(texture, 218 GrSimpleTextureEffect::Create(texture,
219 maskMatrix, 219 maskMatrix,
220 false, 220 false,
221 GrEffect::kPosition_Coord sType))->unref(); 221 GrEffect::kPosition_Coord sType))->unref();
222 222
223 target->drawSimpleRect(dstRect); 223 target->drawSimpleRect(dstRect);
224 drawState->disableStage(kPathMaskStage);
225 } 224 }
OLDNEW
« no previous file with comments | « src/gpu/GrOvalRenderer.cpp ('k') | src/gpu/GrTextContext.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698