diff options
author | 2014-10-27 07:38:01 -0700 | |
---|---|---|
committer | 2014-10-27 07:38:01 -0700 | |
commit | 3a0cfeb96185934c0a36f1313f21b96c57ca6341 (patch) | |
tree | 2083c3d65e1f64f3565ebdddcdbb0440e607ec83 /src/gpu/GrGpu.cpp | |
parent | 1c9acd980b0a8f82a1851c857ea3cd446fe0453e (diff) |
Revert of Clip mask manager sets stencil on draw type (patchset #3 id:40001 of https://codereview.chromium.org/676983003/)
Reason for revert:
seems to cause a gm issue on windows.
Original issue's description:
> Clip mask manager sets stencil on draw type
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/7afb5aa201e4b59397cbd8480e121d7501a227e7
TBR=bsalomon@google.com,joshualitt@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/678843003
Diffstat (limited to 'src/gpu/GrGpu.cpp')
-rw-r--r-- | src/gpu/GrGpu.cpp | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/src/gpu/GrGpu.cpp b/src/gpu/GrGpu.cpp index d8ffe6745d..65100489f4 100644 --- a/src/gpu/GrGpu.cpp +++ b/src/gpu/GrGpu.cpp @@ -298,12 +298,10 @@ const GrIndexBuffer* GrGpu::getQuadIndexBuffer() const { //////////////////////////////////////////////////////////////////////////////// -bool GrGpu::setupClipAndFlushState(DrawType type, - const GrDeviceCoordTexture* dstCopy, +bool GrGpu::setupClipAndFlushState(DrawType type, const GrDeviceCoordTexture* dstCopy, GrDrawState::AutoRestoreEffects* are, - GrDrawState::AutoRestoreStencil* ars, const SkRect* devBounds) { - if (!fClipMaskManager.setupClipping(this->getClip(), are, ars, devBounds)) { + if (!fClipMaskManager.setupClipping(this->getClip(), are, devBounds)) { return false; } @@ -346,9 +344,8 @@ void GrGpu::geometrySourceWillPop(const GeometrySrcState& restoredState) { void GrGpu::onDraw(const DrawInfo& info) { this->handleDirtyContext(); GrDrawState::AutoRestoreEffects are; - GrDrawState::AutoRestoreStencil asr; if (!this->setupClipAndFlushState(PrimTypeToDrawType(info.primitiveType()), - info.getDstCopy(), &are, &asr, info.getDevBounds())) { + info.getDstCopy(), &are, info.getDevBounds())) { return; } this->onGpuDraw(info); @@ -358,8 +355,7 @@ void GrGpu::onStencilPath(const GrPath* path, SkPath::FillType fill) { this->handleDirtyContext(); GrDrawState::AutoRestoreEffects are; - GrDrawState::AutoRestoreStencil asr; - if (!this->setupClipAndFlushState(kStencilPath_DrawType, NULL, &are, &asr, NULL)) { + if (!this->setupClipAndFlushState(kStencilPath_DrawType, NULL, &are, NULL)) { return; } @@ -374,8 +370,7 @@ void GrGpu::onDrawPath(const GrPath* path, SkPath::FillType fill, drawState()->setDefaultVertexAttribs(); GrDrawState::AutoRestoreEffects are; - GrDrawState::AutoRestoreStencil asr; - if (!this->setupClipAndFlushState(kDrawPath_DrawType, dstCopy, &are, &asr, NULL)) { + if (!this->setupClipAndFlushState(kDrawPath_DrawType, dstCopy, &are, NULL)) { return; } @@ -391,8 +386,7 @@ void GrGpu::onDrawPaths(const GrPathRange* pathRange, drawState()->setDefaultVertexAttribs(); GrDrawState::AutoRestoreEffects are; - GrDrawState::AutoRestoreStencil asr; - if (!this->setupClipAndFlushState(kDrawPaths_DrawType, dstCopy, &are, &asr, NULL)) { + if (!this->setupClipAndFlushState(kDrawPaths_DrawType, dstCopy, &are, NULL)) { return; } |