diff options
author | herb <herb@google.com> | 2016-01-07 11:53:16 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-01-07 11:53:16 -0800 |
commit | f96bf1a2d091a917bb93f9e9a3a8d53bb39d068e (patch) | |
tree | 9df64f85e061e43ae1b7069b6bed33d8ff5ab7b4 /tests/DrawPathTest.cpp | |
parent | 6544817b9848b7b6ffb28c761a1011e0df42f0e9 (diff) |
Fix handling of radii scaling to force the result to always be less
than a given side.
BUG=472147
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1567723004
Review URL: https://codereview.chromium.org/1567723004
Diffstat (limited to 'tests/DrawPathTest.cpp')
-rw-r--r-- | tests/DrawPathTest.cpp | 35 |
1 files changed, 35 insertions, 0 deletions
diff --git a/tests/DrawPathTest.cpp b/tests/DrawPathTest.cpp index 364a297123..e9aa4499d9 100644 --- a/tests/DrawPathTest.cpp +++ b/tests/DrawPathTest.cpp @@ -313,6 +313,39 @@ static void test_crbug_165432(skiatest::Reporter* reporter) { REPORTER_ASSERT(reporter, filteredPath.isEmpty()); } +// http://crbug.com/472147 +// This is a simplified version from the bug. RRect radii not properly scaled. +static void test_crbug_472147_simple(skiatest::Reporter* reporter) { + SkAutoTUnref<SkSurface> surface(SkSurface::NewRasterN32Premul(1000, 1000)); + SkCanvas* canvas = surface->getCanvas(); + SkPaint p; + SkRect r = SkRect::MakeLTRB(-246.0f, 33.0f, 848.0f, 33554464.0f); + SkVector radii[4] = { + { 13.0f, 8.0f }, { 170.0f, 2.0 }, { 256.0f, 33554430.0f }, { 120.0f, 5.0f } + }; + SkRRect rr; + rr.setRectRadii(r, radii); + canvas->drawRRect(rr, p); +} + +// http://crbug.com/472147 +// RRect radii not properly scaled. +static void test_crbug_472147_actual(skiatest::Reporter* reporter) { + SkAutoTUnref<SkSurface> surface(SkSurface::NewRasterN32Premul(1000, 1000)); + SkCanvas* canvas = surface->getCanvas(); + SkPaint p; + SkRect r = SkRect::MakeLTRB(-246.0f, 33.0f, 848.0f, 33554464.0f); + SkVector radii[4] = { + { 13.0f, 8.0f }, { 170.0f, 2.0 }, { 256.0f, 33554430.0f }, { 120.0f, 5.0f } + }; + SkRRect rr; + rr.setRectRadii(r, radii); + canvas->clipRRect(rr, SkRegion::kIntersect_Op, false); + + SkRect r2 = SkRect::MakeLTRB(0, 33, 1102, 33554464); + canvas->drawRect(r2, p); +} + DEF_TEST(DrawPath, reporter) { test_giantaa(); test_bug533(); @@ -325,6 +358,8 @@ DEF_TEST(DrawPath, reporter) { if (false) test_crbug131181(); test_infinite_dash(reporter); test_crbug_165432(reporter); + test_crbug_472147_simple(reporter); + test_crbug_472147_actual(reporter); test_big_aa_rect(reporter); test_halfway(); } |