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

Unified Diff: src/gpu/SkGpuDevice.cpp

Issue 1271243004: Remove SHADER_AA_FILL_RECT flag (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 4 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/GrDrawContext.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/gpu/SkGpuDevice.cpp
diff --git a/src/gpu/SkGpuDevice.cpp b/src/gpu/SkGpuDevice.cpp
index 6358181d408818811db4b55b347e153c368011de..cd257b360a97a93ae2c56e6a25acf4b259898bc6 100644
--- a/src/gpu/SkGpuDevice.cpp
+++ b/src/gpu/SkGpuDevice.cpp
@@ -488,25 +488,14 @@ void SkGpuDevice::drawRect(const SkDraw& draw, const SkRect& rect,
bool usePath = doStroke && width > 0 &&
(paint.getStrokeJoin() == SkPaint::kRound_Join ||
(paint.getStrokeJoin() == SkPaint::kBevel_Join && rect.isEmpty()));
- // another two reasons we might need to call drawPath...
- if (paint.getMaskFilter()) {
+ // a few other reasons we might need to call drawPath...
+ if (paint.getMaskFilter() ||
+ paint.getStyle() == SkPaint::kStrokeAndFill_Style) { // we can't both stroke and fill rects
usePath = true;
}
if (!usePath && paint.isAntiAlias() && !draw.fMatrix->rectStaysRect()) {
-#ifdef SHADER_AA_FILL_RECT
- if (doStroke) {
-#endif
- usePath = true;
-#ifdef SHADER_AA_FILL_RECT
- } else {
- usePath = !draw.fMatrix->preservesRightAngles();
- }
-#endif
- }
- // until we can both stroke and fill rectangles
- if (paint.getStyle() == SkPaint::kStrokeAndFill_Style) {
usePath = true;
}
« no previous file with comments | « src/gpu/GrDrawContext.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698