Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Side by Side Diff: tests/SRGBTest.cpp

Issue 2164843002: We have some work to do. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2016 Google Inc. 2 * Copyright 2016 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "SkSRGB.h" 8 #include "SkSRGB.h"
9 #include "SkTypes.h" 9 #include "SkTypes.h"
10 #include "Test.h" 10 #include "Test.h"
11 #include <math.h> 11 #include <math.h>
12 12
13 static uint8_t linear_to_srgb(float l) { 13 static uint8_t linear_to_srgb(float l) {
14 // Round float to int, truncate that to uint8_t. 14 // Round float to int, truncate that to uint8_t.
15 return (uint8_t)Sk4f_round( sk_linear_to_srgb(Sk4f{l}) )[0]; 15 return (uint8_t)Sk4f_round( sk_linear_to_srgb(Sk4f{l}) )[0];
16 } 16 }
17 17
18 DEF_TEST(sk_linear_to_srgb, r) { 18 DEF_TEST(sk_linear_to_srgb, r) {
19 // Should map 0 -> 0 and 1 -> 1. 19 // Should map 0 -> 0 and 1 -> 1.
20 REPORTER_ASSERT(r, 0 == linear_to_srgb(0.0f)); 20 REPORTER_ASSERT(r, 0 == linear_to_srgb(0.0f));
21 REPORTER_ASSERT(r, 255 == linear_to_srgb(1.0f)); 21 REPORTER_ASSERT(r, 255 == linear_to_srgb(1.0f));
22 22
23 int n = 0;
24 for (int i = 0; i < 256; i++) {
25 int actual = linear_to_srgb(sk_linear_from_srgb[i]);
26 if (actual != i) {
27 printf("%d -> %d\n", i, actual);
28 n++;
29 }
30 }
31 printf("n = %d\n", n);
32 return;
33
23 // Should be monotonic between 0 and 1. 34 // Should be monotonic between 0 and 1.
24 // We don't bother checking denorm values. 35 // We don't bother checking denorm values.
25 int tolerated_regressions = 0; 36 int tolerated_regressions = 0;
26 #if defined(SK_ARM_HAS_NEON) 37 #if defined(SK_ARM_HAS_NEON)
27 // Values around 0.166016 are usually 72 but drop briefly (41 floats) down t o 71. 38 // Values around 0.166016 are usually 72 but drop briefly (41 floats) down t o 71.
28 tolerated_regressions = 1; 39 tolerated_regressions = 1;
29 #endif 40 #endif
30 uint8_t prev = 0; 41 uint8_t prev = 0;
31 for (float f = FLT_MIN; f <= 1.0f; ) { 42 for (float f = FLT_MIN; f <= 1.0f; ) {
32 uint8_t srgb = linear_to_srgb(f); 43 uint8_t srgb = linear_to_srgb(f);
33 44
34 REPORTER_ASSERT(r, srgb >= prev || tolerated_regressions > 0); 45 REPORTER_ASSERT(r, srgb >= prev || tolerated_regressions > 0);
35 if (srgb < prev) { tolerated_regressions--; } 46 if (srgb < prev) { tolerated_regressions--; }
36 prev = srgb; 47 prev = srgb;
37 48
38 union { float flt; uint32_t bits; } pun = { f }; 49 union { float flt; uint32_t bits; } pun = { f };
39 pun.bits++; 50 pun.bits++;
40 f = pun.flt; 51 f = pun.flt;
41 } 52 }
42 } 53 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698