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

Side by Side Diff: tests/SkColor4fTest.cpp

Issue 1622483002: Revert[2] of float color components (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: fix inversion of unpremul check Created 4 years, 11 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 | « src/effects/SkColorMatrixFilter.cpp ('k') | 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
(Empty)
1 /*
2 * Copyright 2016 Google Inc.
3 *
4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file.
6 */
7
8 #include "SkColor.h"
9 #include "SkShader.h"
10 #include "SkColorMatrixFilter.h"
11 #include "Test.h"
12 #include "SkRandom.h"
13
14 DEF_TEST(SkColor4f_FromColor, reporter) {
15 const struct {
16 SkColor fC;
17 SkColor4f fC4;
18 } recs[] = {
19 { SK_ColorBLACK, { 1, 0, 0, 0 } },
20 { SK_ColorWHITE, { 1, 1, 1, 1 } },
21 { SK_ColorRED, { 1, 1, 0, 0 } },
22 { SK_ColorGREEN, { 1, 0, 1, 0 } },
23 { SK_ColorBLUE, { 1, 0, 0, 1 } },
24 { 0, { 0, 0, 0, 0 } },
25 { 0x55AAFF00, { 1/3.0f, 2/3.0f, 1, 0 } },
26 };
27
28 for (const auto& r : recs) {
29 SkColor4f c4 = SkColor4f::FromColor(r.fC);
30 REPORTER_ASSERT(reporter, c4 == r.fC4);
31 }
32 }
33
34 static bool nearly_equal(float a, float b) {
35 const float kTolerance = 1.0f / (1 << 20);
36 return fabsf(a - b) < kTolerance;
37 }
38
39 DEF_TEST(SkColor4f_premul, reporter) {
40 SkRandom rand;
41
42 for (int i = 0; i < 1000000; ++i) {
43 // First just test opaque colors, so that the premul should be exact
44 SkColor4f c4 {
45 1, rand.nextUScalar1(), rand.nextUScalar1(), rand.nextUScalar1()
46 };
47 SkPM4f pm4 = c4.premul();
48 REPORTER_ASSERT(reporter, pm4.fVec[SK_A_INDEX] == c4.fA);
49 REPORTER_ASSERT(reporter, pm4.fVec[SK_R_INDEX] == c4.fA * c4.fR);
50 REPORTER_ASSERT(reporter, pm4.fVec[SK_G_INDEX] == c4.fA * c4.fG);
51 REPORTER_ASSERT(reporter, pm4.fVec[SK_B_INDEX] == c4.fA * c4.fB);
52
53 // We compare with a tolerance, in case our premul multiply is implement ed at slightly
54 // different precision than the test code.
55 c4.fA = rand.nextUScalar1();
56 pm4 = c4.premul();
57 REPORTER_ASSERT(reporter, pm4.fVec[SK_A_INDEX] == c4.fA);
58 REPORTER_ASSERT(reporter, nearly_equal(pm4.fVec[SK_R_INDEX], c4.fA * c4. fR));
59 REPORTER_ASSERT(reporter, nearly_equal(pm4.fVec[SK_G_INDEX], c4.fA * c4. fG));
60 REPORTER_ASSERT(reporter, nearly_equal(pm4.fVec[SK_B_INDEX], c4.fA * c4. fB));
61 }
62 }
OLDNEW
« no previous file with comments | « src/effects/SkColorMatrixFilter.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698