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

Unified Diff: src/pathops/SkPathOpsSimplify.cpp

Issue 2321773002: pathops tiger checkpoint (Closed)
Patch Set: checkpoint (all not-under-development tests work) Created 4 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
« no previous file with comments | « src/pathops/SkPathOpsDebug.cpp ('k') | tests/PathOpsSimplifyTest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/pathops/SkPathOpsSimplify.cpp
diff --git a/src/pathops/SkPathOpsSimplify.cpp b/src/pathops/SkPathOpsSimplify.cpp
index dcd75f166641b90f6c3b2f2fce9088d685cf9d57..2376e1d25967e2bd7b064bdc2b5fbd958fa7a0c4 100644
--- a/src/pathops/SkPathOpsSimplify.cpp
+++ b/src/pathops/SkPathOpsSimplify.cpp
@@ -36,7 +36,10 @@ static bool bridgeWinding(SkOpContourHead* contourList, SkPathWriter* simple, bo
if (!unsortable && simple->hasMove()
&& current->verb() != SkPath::kLine_Verb
&& !simple->isClosed()) {
- if (!current->addCurveTo(start, end, simple)) {
+ // FIXME: put in the next two lines to avoid handling already added
+ if (start->starter(end)->checkAlreadyAdded()) {
+ simple->close();
+ } else if (!current->addCurveTo(start, end, simple)) {
return false;
}
if (!simple->isClosed()) {
« no previous file with comments | « src/pathops/SkPathOpsDebug.cpp ('k') | tests/PathOpsSimplifyTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698