diff options
author | caryclark <caryclark@google.com> | 2016-01-05 12:36:09 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-01-05 12:36:09 -0800 |
commit | ba25ddb7f2032f778a956f646e5aa993045c280e (patch) | |
tree | c7ff4011cc2002b21801aca0655f9afce21e83bc /tests | |
parent | 226aee25f426fa12ad16781257490cfd339fb95b (diff) |
fix nexus player
The path contains test has such large numbers that it is indeterminate
whether it will return true or false. Allow either result, ensuring
that the test itself doesn't crash.
TBR=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1565463002
Review URL: https://codereview.chromium.org/1565463002
Diffstat (limited to 'tests')
-rw-r--r-- | tests/PathTest.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/PathTest.cpp b/tests/PathTest.cpp index 7c5ca0e4ff..3c4638d741 100644 --- a/tests/PathTest.cpp +++ b/tests/PathTest.cpp @@ -3514,7 +3514,8 @@ static void test_contains(skiatest::Reporter* reporter) { p.lineTo(SkBits2Float(0x609b9872), SkBits2Float(0xdf730de8)); // 8.96947e+19f, -1.75139e+19f p.conicTo(SkBits2Float(0x6018b296), SkBits2Float(0xdeee870d), SkBits2Float(0xe008cd8e), SkBits2Float(0x5ed5b2db), SkBits2Float(0x3f3504f3)); // 4.40121e+19f, -8.59386e+18f, -3.94308e+19f, 7.69931e+18f, 0.707107f p.conicTo(SkBits2Float(0xe0d526d9), SkBits2Float(0x5fa67b31), SkBits2Float(0xe085e7b2), SkBits2Float(0x5f512c01), SkBits2Float(0x3f3504f3)); // -1.22874e+20f, 2.39925e+19f, -7.7191e+19f, 1.50724e+19f, 0.707107f - REPORTER_ASSERT(reporter, !p.contains(-77.2027664f, 15.3066053f)); + // this may return true or false, depending on the platform's numerics, but it should not crash + (void) p.contains(-77.2027664f, 15.3066053f); p.reset(); p.setFillType(SkPath::kInverseWinding_FillType); |