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

Unified Diff: src/gpu/GrClipMaskManager.cpp

Issue 680543002: Add some more volatile paths. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Rebase to ToT Created 6 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 side-by-side diff with in-line comments
Download patch
« src/core/SkCanvas.cpp ('K') | « src/core/SkCanvas.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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);
}
}
« src/core/SkCanvas.cpp ('K') | « src/core/SkCanvas.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698