Require Sk4f::toBytes() clamps

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

Committed: https://skia.googlesource.com/skia/+/a508f3c62d726867ad9f722623fbf0ab5d06e440

[mtklein_C, 2015-09-01 12:24:54]

The CQ bit was checked by mtklein@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-09-01 12:25:00]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1312053004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1312053004/20001

[mtklein_C, 2015-09-01 12:30:41]

mtklein@chromium.org changed reviewers:
+ noel@chromium.org

[commit-bot: I haz the power, 2015-09-01 12:38:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-01 12:38:14]

Dry run: This issue passed the CQ dry run.

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

LGTM.

https://codereview.chromium.org/1312053004/diff/20001/tests/SkNxTest.cpp
File tests/SkNxTest.cpp (right):

https://codereview.chromium.org/1312053004/diff/20001/tests/SkNxTest.cpp#newc...
tests/SkNxTest.cpp:218: REPORTER_ASSERT(r, bytes[3] == 255);
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]

https://codereview.chromium.org/1312053004/diff/20001/tests/SkNxTest.cpp
File tests/SkNxTest.cpp (right):

https://codereview.chromium.org/1312053004/diff/20001/tests/SkNxTest.cpp#newc...
tests/SkNxTest.cpp:218: REPORTER_ASSERT(r, bytes[3] == 255);
On 2015/09/01 12:58:31, noel gordon wrote:
> Thanks for the clamp test.  Perhaps add a +0.7f test-case (so I can tell if it
> rounds or truncates)?

Done.

[mtklein, 2015-09-01 13:01:50]

mtklein@google.com changed reviewers:
+ mtklein@google.com

[mtklein, 2015-09-01 13:01:50]

The CQ bit was checked by mtklein@google.com

[mtklein, 2015-09-01 13:01:51]

lgtm

[mtklein, 2015-09-01 13:01:51]

The patchset sent to the CQ was uploaded after l-g-t-m from noel@chromium.org
Link to the patchset: https://codereview.chromium.org/1312053004/#ps60001
(title: "tweak")

[commit-bot: I haz the power, 2015-09-01 13:02:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1312053004/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1312053004/60001

[commit-bot: I haz the power, 2015-09-01 13:29:48]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/a508f3c62d726867ad9f722623fbf0ab5d06e440