Pass forceClose "true" to SkPath::Iter constructor in
authorsenorblanco@chromium.org <senorblanco@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81>
Tue, 17 May 2011 18:29:26 +0000 (18:29 +0000)
committersenorblanco@chromium.org <senorblanco@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81>
Tue, 17 May 2011 18:29:26 +0000 (18:29 +0000)
GrPathUtils::worstCasePointCount().  worstCasePointCount() is sometimes
returning a lower value than the number of points subsequently generated by
the path renderers.  This is because it constructs the SkPath::Iter with
forceClose set to "false", while the path renderers use one with forceClose
set to "true".  They should both be the same, and since we're filling paths, I
think it should be set "true".

Review URL:  http://codereview.appspot.com/4552047/

git-svn-id: http://skia.googlecode.com/svn/trunk@1353 2bbb7eff-a529-9590-31e7-b0007b416f81

gpu/src/GrPathUtils.cpp

index 1fb043c..8a72ba8 100644 (file)
@@ -111,7 +111,7 @@ int GrPathUtils::worstCasePointCount(const GrPath& path, int* subpaths,
 
     bool first = true;
 
-    SkPath::Iter iter(path, false);
+    SkPath::Iter iter(path, true);
     GrPathCmd cmd;
 
     GrPoint pts[4];