From 160e4dcc4ec8c17578b3800c8683981834b903cb Mon Sep 17 00:00:00 2001 From: "skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com" Date: Mon, 30 Jul 2018 13:38:30 +0000 Subject: Roll skia/third_party/skcms 51fba282d9a0..6b7ef53d3298 (1 commits) https://skia.googlesource.com/skcms.git/+log/51fba282d9a0..6b7ef53d3298 2018-07-30 mtklein@google.com add unit test for clamp-before-premul The AutoRoll server is located here: https://skcms-skia-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel TBR=scroggo@google.com Change-Id: I072c40574deb81b6e1bde928e232eecd7e2b4925 Reviewed-on: https://skia-review.googlesource.com/144320 Commit-Queue: skcms-skia-autoroll Reviewed-by: skcms-skia-autoroll --- third_party/skcms/skcms.cc | 1 - third_party/skcms/version.sha1 | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/third_party/skcms/skcms.cc b/third_party/skcms/skcms.cc index 3595088633..6fb4af96f4 100644 --- a/third_party/skcms/skcms.cc +++ b/third_party/skcms/skcms.cc @@ -2230,7 +2230,6 @@ bool skcms_Transform(const void* src, // // E.g. r = 1.1, a = 0.5 would fit fine in fixed point after premul (ra=0.55,a=0.5), // but would be carrying r > 1, which is really unexpected for downstream consumers. - // TODO(mtklein): add a unit test if (dstFmt < skcms_PixelFormat_RGB_hhh) { *ops++ = Op_clamp; } diff --git a/third_party/skcms/version.sha1 b/third_party/skcms/version.sha1 index 5554bc6e81..432b33bc37 100755 --- a/third_party/skcms/version.sha1 +++ b/third_party/skcms/version.sha1 @@ -1 +1 @@ -51fba282d9a06baa9f8ed2e3d679c5f439420322 \ No newline at end of file +6b7ef53d3298a7ab7e3410ba3ffe5623539f025e \ No newline at end of file -- cgit v1.2.3