commit 62b3004439cab5e246c66803c5841c35b2df9c90
author Cary Clark <caryclark@skia.org> Tue Oct 23 10:05:06 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Tue Oct 23 14:33:15 2018 +0000
tree 62f7144ac7f6124934bcd9680470a1dee9532c10
parent 1f0170ce7a908a1112b60d5f2bff2132624000e1

fix Simplify pathop fuzz

Simplify was out of sync with Op, and needed
to return failure when contour could not be
followed.

Also fix timeout where coincident detection
got stuck.

R=kjlubick@google.com

Bug:898114
Change-Id: Ia9076b9fd48e093bcc40ef21809c8eabe4cb9367
Reviewed-on: https://skia-review.googlesource.com/c/164601
Commit-Queue: Cary Clark <caryclark@google.com>
Commit-Queue: Kevin Lubick <kjlubick@google.com>
Auto-Submit: Cary Clark <caryclark@skia.org>
Reviewed-by: Kevin Lubick <kjlubick@google.com>

src/pathops/SkPathOpsSimplify.cpp

diff --git a/src/pathops/SkPathOpsSimplify.cpp b/src/pathops/SkPathOpsSimplify.cpp
index 86b2c06..b6d7b40 100644
--- a/src/pathops/SkPathOpsSimplify.cpp
+++ b/src/pathops/SkPathOpsSimplify.cpp
@@ -57,7 +57,9 @@
                 writer->finishContour();
             } else {
                 SkOpSpanBase* last;
-                SkAssertResult(current->markAndChaseDone(start, end, &last));
+                 if (!current->markAndChaseDone(start, end, &last)) {
+                    return false;
+                }
                 if (last && !last->chased()) {
                     last->setChased(true);
                     SkASSERT(!SkPathOpsDebug::ChaseContains(chase, last));