diff options
author | herb <herb@google.com> | 2016-01-11 08:08:56 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-01-11 08:08:56 -0800 |
commit | 7cf12ddb8c1c2cf7a0eec63439148cc6b2bc104a (patch) | |
tree | 8dee486021054106d79f39b071431dfc3fac2773 /tests/DrawPathTest.cpp | |
parent | 8bae7c537ae77302a61a7fa5b990ca394710db32 (diff) |
Fix radii calculation code to handle large radii.
BUG=472147
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1569403002
Review URL: https://codereview.chromium.org/1569403002
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(); } |