Require Sk4f::toBytes() clamps

tests/SkNxTest.cpp

Index: tests/SkNxTest.cpp
diff --git a/tests/SkNxTest.cpp b/tests/SkNxTest.cpp
index 4005d2518f5684a4b782a097cd01579afb58751b..0e76014d09c37137283d9f089694253b1e412809 100644
--- a/tests/SkNxTest.cpp
+++ b/tests/SkNxTest.cpp
@@ -208,3 +208,12 @@ DEF_TEST(Sk4px_widening, r) {
          wideLoHiAlt = wideLo + wideHi;
     REPORTER_ASSERT(r, 0 == memcmp(&wideLoHi, &wideLoHiAlt, sizeof(wideLoHi)));
 }
+
+DEF_TEST(Sk4f_clamps, r) {
+    uint8_t bytes[4];
+    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);

[Noel Gordon, 2015/09/01 12:58:31]

Thanks for the clamp test.  Perhaps add a +0.7f test-case (so I can tell if it
rounds or truncates)?

[mtklein_C, 2015/09/01 13:01:15]

Done.

+}