diff options
author | Brian Osman <brianosman@google.com> | 2017-09-15 20:30:28 +0000 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-09-15 20:30:47 +0000 |
commit | f95352322496796ce4c99df9582dbc630fe8a327 (patch) | |
tree | 2f7525991adec1bab3944c30843f94fdc379fdd5 /tests/PathTest.cpp | |
parent | b6d2be1330f16fe05f1eed5abda927c0fbb50055 (diff) |
Revert "Don't ignore degenerates when deciding if a path is convex"
This reverts commit 53cd6c4331a2ef21a7c5eb6166c782cd33178a7b.
Reason for revert: Temporary to limit gold cross-talk with other CLs.
Original change's description:
> Don't ignore degenerates when deciding if a path is convex
>
> This ensures that a path with a second contour will always be marked
> concave. GrDefaultPathRenderer was incorrectly drawing paths of this type
> (thinking that it could fill them with simple triangulation).
>
> Bug: skia:7020 skia:1460
> Change-Id: I62bfd72e4c61da427687acf53c552357b57707aa
> Reviewed-on: https://skia-review.googlesource.com/47082
> Reviewed-by: Cary Clark <caryclark@google.com>
> Commit-Queue: Brian Osman <brianosman@google.com>
TBR=bsalomon@google.com,brianosman@google.com,caryclark@google.com,reed@google.com
Change-Id: Id7d121633faeb8a43dbd334409408ba51db43d68
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:7020 skia:1460
Reviewed-on: https://skia-review.googlesource.com/47343
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
Diffstat (limited to 'tests/PathTest.cpp')
-rw-r--r-- | tests/PathTest.cpp | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/tests/PathTest.cpp b/tests/PathTest.cpp index 8c629d9485..75e86e3f92 100644 --- a/tests/PathTest.cpp +++ b/tests/PathTest.cpp @@ -1880,10 +1880,15 @@ static void test_conservativelyContains(skiatest::Reporter* reporter) { // Test that multiple move commands do not cause asserts. + + // At the time of writing, this would not modify cached convexity. This caused an assert while + // checking conservative containment again. https://bug.skia.org/1460 path.moveTo(SkIntToScalar(100), SkIntToScalar(100)); +#if 0 REPORTER_ASSERT(reporter, path.conservativelyContainsRect(SkRect::MakeXYWH(SkIntToScalar(50), 0, SkIntToScalar(10), SkIntToScalar(10)))); +#endif // Same as above path and first test but with an extra moveTo. path.reset(); @@ -1891,12 +1896,10 @@ static void test_conservativelyContains(skiatest::Reporter* reporter) { path.moveTo(0, 0); path.lineTo(SkIntToScalar(100), 0); path.lineTo(0, SkIntToScalar(100)); - // Convexity logic is now more conservative, so that multiple (non-trailing) moveTos make a - // path non-convex. - REPORTER_ASSERT(reporter, !path.conservativelyContainsRect( - SkRect::MakeXYWH(SkIntToScalar(50), 0, - SkIntToScalar(10), - SkIntToScalar(10)))); + + REPORTER_ASSERT(reporter, path.conservativelyContainsRect(SkRect::MakeXYWH(SkIntToScalar(50), 0, + SkIntToScalar(10), + SkIntToScalar(10)))); // Same as above path and first test but with the extra moveTo making a degenerate sub-path // following the non-empty sub-path. Verifies that this does not trigger assertions. |