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

Unified Diff: src/gpu/GrSoftwarePathRenderer.cpp

Issue 418073002: Alter Default & Software path renderers to better handle AA hairline paths (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Update to ToT Created 6 years, 5 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
« no previous file with comments | « src/gpu/GrDefaultPathRenderer.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/gpu/GrSoftwarePathRenderer.cpp
diff --git a/src/gpu/GrSoftwarePathRenderer.cpp b/src/gpu/GrSoftwarePathRenderer.cpp
index d0936d6b28a37427b31d0b66cda1dc7138dfccdd..bca2c66b37f82614f44660b21fed9d72cb97f00d 100644
--- a/src/gpu/GrSoftwarePathRenderer.cpp
+++ b/src/gpu/GrSoftwarePathRenderer.cpp
@@ -15,13 +15,7 @@ bool GrSoftwarePathRenderer::canDrawPath(const SkPath&,
const SkStrokeRec&,
const GrDrawTarget*,
bool antiAlias) const {
- if (!antiAlias || NULL == fContext) {
- // TODO: We could allow the SW path to also handle non-AA paths but
- // this would mean that GrDefaultPathRenderer would never be called
- // (since it appears after the SW renderer in the path renderer
- // chain). Some testing would need to be done r.e. performance
- // and consistency of the resulting images before removing
- // the "!antiAlias" clause from the above test
+ if (NULL == fContext) {
return false;
}
« no previous file with comments | « src/gpu/GrDefaultPathRenderer.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698