Chromium Code Reviews| Index: src/gpu/GrClipMaskManager.cpp |
| diff --git a/src/gpu/GrClipMaskManager.cpp b/src/gpu/GrClipMaskManager.cpp |
| index 349d51e311586103bf14c9f67f08e33a4a1b7a36..73b5dfe2c5ec7b18b3effcf367ec4a6658c21a21 100644 |
| --- a/src/gpu/GrClipMaskManager.cpp |
| +++ b/src/gpu/GrClipMaskManager.cpp |
| @@ -97,6 +97,7 @@ bool GrClipMaskManager::useSWOnlyPath(const GrReducedClip::ElementList& elements |
| if (Element::kRect_Type != element->getType()) { |
| SkPath path; |
| element->asPath(&path); |
| + path.setIsVolatile(true); |
|
bsalomon
2014/10/24 17:29:18
Is it not already volatile since it comes from the
jvanverth1
2014/10/24 17:36:30
I'm not sure when it would get set as volatile in
|
| if (path_needs_SW_renderer(this->getContext(), fGpu, path, stroke, element->isAA())) { |
| return true; |
| } |
| @@ -446,6 +447,7 @@ bool GrClipMaskManager::canStencilAndDrawElement(GrTexture* target, |
| SkASSERT(Element::kEmpty_Type != element->getType()); |
| SkPath path; |
| element->asPath(&path); |
| + path.setIsVolatile(true); |
| if (path.isInverseFillType()) { |
| path.toggleInverseFillType(); |
| } |
| @@ -766,6 +768,7 @@ bool GrClipMaskManager::createStencilClipMask(int32_t elementsGenID, |
| fillInverted = false; |
| } else { |
| element->asPath(&clipPath); |
| + clipPath.setIsVolatile(true); |
| fillInverted = clipPath.isInverseFillType(); |
| if (fillInverted) { |
| clipPath.toggleInverseFillType(); |
| @@ -1080,6 +1083,7 @@ GrTexture* GrClipMaskManager::createSoftwareClipMask(int32_t elementsGenID, |
| SkPath clipPath; |
| element->asPath(&clipPath); |
| + clipPath.setIsVolatile(true); |
| clipPath.toggleInverseFillType(); |
| helper.draw(clipPath, stroke, SkRegion::kReplace_Op, element->isAA(), 0x00); |
| @@ -1093,6 +1097,7 @@ GrTexture* GrClipMaskManager::createSoftwareClipMask(int32_t elementsGenID, |
| } else { |
| SkPath path; |
| element->asPath(&path); |
| + path.setIsVolatile(true); |
| helper.draw(path, stroke, op, element->isAA(), 0xFF); |
| } |
| } |