From 3de40f8bee6db21e61ccabcb3f7130796c017a06 Mon Sep 17 00:00:00 2001 From: Stephen White Date: Thu, 28 Jun 2018 09:36:49 -0400 Subject: GrTessellator: handle collinear final vertex. If the last vertex in a contour is collinear with its neighbours, remove it (this edge case was missing). Otherwise, the simplify step may try to split it indefinitely. Bug: 851409 Change-Id: I7efa4e616cdc1508a73c7a9f3de9d3f571569af8 Reviewed-on: https://skia-review.googlesource.com/138106 Reviewed-by: Robert Phillips Commit-Queue: Stephen White --- tests/TessellatingPathRendererTests.cpp | 12 ++++++++++++ 1 file changed, 12 insertions(+) (limited to 'tests/TessellatingPathRendererTests.cpp') diff --git a/tests/TessellatingPathRendererTests.cpp b/tests/TessellatingPathRendererTests.cpp index 0b30feecc0..a9913769df 100644 --- a/tests/TessellatingPathRendererTests.cpp +++ b/tests/TessellatingPathRendererTests.cpp @@ -560,6 +560,17 @@ static SkPath create_path_38() { return path; } +// Reduction from crbug.com/851409. +static SkPath create_path_39() { + SkPath path; + path.moveTo(2072553216, 0); + path.lineTo(2072553216, 1); + path.lineTo(2072553472, -13.5); + path.lineTo(2072553216, 0); + path.lineTo(2072553472, -6.5); + return path; +} + static std::unique_ptr create_linear_gradient_processor(GrContext* ctx) { SkPoint pts[2] = { {0, 0}, {1, 1} }; @@ -658,4 +669,5 @@ DEF_GPUTEST_FOR_ALL_CONTEXTS(TessellatingPathRendererTests, reporter, ctxInfo) { test_path(ctx, rtc.get(), create_path_36()); test_path(ctx, rtc.get(), create_path_37()); test_path(ctx, rtc.get(), create_path_38(), SkMatrix(), GrAAType::kCoverage); + test_path(ctx, rtc.get(), create_path_39()); } -- cgit v1.2.3