diff options
| author |  mtklein <mtklein@chromium.org> | 2015-09-01 06:29:45 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2015-09-01 06:29:45 -0700 |
| commit | a508f3c62d726867ad9f722623fbf0ab5d06e440 (patch) | |
| tree | f674583ce163e231c7dbc92c695ce29f661f2a80 /tests/SkNxTest.cpp | |
| parent | 580c40ae71a4914a9f4fa247accf86810e3463db (diff) | |
Require Sk4f::toBytes() clamps
BUG=skia:4117
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot;client.skia.android:Test-Android-GCC-Nexus9-CPU-Denver-Arm64-Release-Trybot
Review URL: https://codereview.chromium.org/1312053004
Diffstat (limited to 'tests/SkNxTest.cpp')
| -rw-r--r-- | tests/SkNxTest.cpp | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/tests/SkNxTest.cpp b/tests/SkNxTest.cpp index 4005d2518f..185940f195 100644 --- a/tests/SkNxTest.cpp +++ b/tests/SkNxTest.cpp @@ -208,3 +208,18 @@ DEF_TEST(Sk4px_widening, r) { wideLoHiAlt = wideLo + wideHi; REPORTER_ASSERT(r, 0 == memcmp(&wideLoHi, &wideLoHiAlt, sizeof(wideLoHi))); } + +DEF_TEST(Sk4f_toBytes, r) { + uint8_t bytes[4]; + + // toBytes truncates, not rounds. + Sk4f(0.7f).toBytes(bytes); + REPORTER_ASSERT(r, bytes[0] == 0); + + // Clamping edge cases. + Sk4f(-2.0f, -0.7f, 255.9f, 256.0f).toBytes(bytes); + REPORTER_ASSERT(r, bytes[0] == 0); + REPORTER_ASSERT(r, bytes[1] == 0); + REPORTER_ASSERT(r, bytes[2] == 255); + REPORTER_ASSERT(r, bytes[3] == 255); +} |