aboutsummaryrefslogtreecommitdiffhomepage
path: root/tests/PathOpsAngleIdeas.cpp
diff options
context:
space:
mode:
authorGravatar caryclark <caryclark@google.com>2016-06-08 04:28:19 -0700
committerGravatar Commit bot <commit-bot@chromium.org>2016-06-08 04:28:19 -0700
commitdae6b97705fde08958b1a36fa6ce685d28fc692c (patch)
tree39b7c096c331a0a29a43ddc7455705f821f5f640 /tests/PathOpsAngleIdeas.cpp
parent2af4599b5c514933bf997d4837ddaaf24fc61cd7 (diff)
fix pathops fuzz bugs
Fail out in a couple of new places when the input data is very large and exceeds the limits of the pathops machinery. Most of the change here plumbs in a way to exclude an assert in one of these exceptional cases. The current SkAddIntersection implementation and the inner functions it calls has no way to report an error to the root caller for an early exit, so rather than add that in, exclude the assert when the test that would trigger it runs (allowing the test to otherwise ensure that it properly fails). TBR=reed@google.com BUG=617586,617635 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2046713003 Review-Url: https://codereview.chromium.org/2046713003
Diffstat (limited to 'tests/PathOpsAngleIdeas.cpp')
-rwxr-xr-xtests/PathOpsAngleIdeas.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/PathOpsAngleIdeas.cpp b/tests/PathOpsAngleIdeas.cpp
index 8fc100ee02..2e3c7b9066 100755
--- a/tests/PathOpsAngleIdeas.cpp
+++ b/tests/PathOpsAngleIdeas.cpp
@@ -419,7 +419,7 @@ static void testQuadAngles(skiatest::Reporter* reporter, const SkDQuad& quad1, c
SkPoint shortQuads[2][3];
SkOpContourHead contour;
- SkOpGlobalState state(nullptr, &contour SkDEBUGPARAMS(nullptr));
+ SkOpGlobalState state(nullptr, &contour SkDEBUGPARAMS(false) SkDEBUGPARAMS(nullptr));
contour.init(&state, false, false);
makeSegment(&contour, quad1, shortQuads[0], allocator);
makeSegment(&contour, quad1, shortQuads[1], allocator);