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

Unified Diff: src/core/SkCanvas.cpp

Issue 1330623003: more zero-length changes for svg compatibility (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: remove inline from hasOnlyMoveTo() Created 5 years, 3 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
Index: src/core/SkCanvas.cpp
diff --git a/src/core/SkCanvas.cpp b/src/core/SkCanvas.cpp
index 7043ec3bc346396b7be596b1d0a865a47dc43f0a..6f6bbf1e038db7376afcaea6d8db96c3d3617de0 100644
--- a/src/core/SkCanvas.cpp
+++ b/src/core/SkCanvas.cpp
@@ -2124,8 +2124,11 @@ void SkCanvas::onDrawPath(const SkPath& path, const SkPaint& paint) {
if (r.width() <= 0 && r.height() <= 0) {
if (path.isInverseFillType()) {
this->internalDrawPaint(paint);
+ return;
+ }
+ if (path.hasOnlyMoveTos()) {
reed1 2015/09/03 19:07:27 Is this test for correctness or for performance?
caryclark 2015/09/03 19:37:57 Correctness. Removing this test causes EmptyPath t
+ return;
}
- return;
}
LOOPER_BEGIN(paint, SkDrawFilter::kPath_Type, bounds)

Powered by Google App Engine
This is Rietveld 408576698