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

Side by Side Diff: tests/PMFloatTest.cpp

Issue 974643002: Test and fix SkPMFloat rounding. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: f Created 5 years, 9 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/opts/SkPMFloat_none.h ('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
1 #include "SkPMFloat.h" 1 #include "SkPMFloat.h"
2 #include "Test.h" 2 #include "Test.h"
3 3
4 DEF_TEST(SkPMFloat, r) { 4 DEF_TEST(SkPMFloat, r) {
5 // Test SkPMColor <-> SkPMFloat 5 // Test SkPMColor <-> SkPMFloat
6 SkPMColor c = SkPreMultiplyColor(0xFFCC9933); 6 SkPMColor c = SkPreMultiplyColor(0xFFCC9933);
7 7
8 SkPMFloat pmf; 8 SkPMFloat pmf;
9 pmf.set(c); 9 pmf.set(c);
10 REPORTER_ASSERT(r, SkScalarNearlyEqual(255.0f, pmf.a())); 10 REPORTER_ASSERT(r, SkScalarNearlyEqual(255.0f, pmf.a()));
11 REPORTER_ASSERT(r, SkScalarNearlyEqual(204.0f, pmf.r())); 11 REPORTER_ASSERT(r, SkScalarNearlyEqual(204.0f, pmf.r()));
12 REPORTER_ASSERT(r, SkScalarNearlyEqual(153.0f, pmf.g())); 12 REPORTER_ASSERT(r, SkScalarNearlyEqual(153.0f, pmf.g()));
13 REPORTER_ASSERT(r, SkScalarNearlyEqual( 51.0f, pmf.b())); 13 REPORTER_ASSERT(r, SkScalarNearlyEqual( 51.0f, pmf.b()));
14 REPORTER_ASSERT(r, c == pmf.get());
14 15
16 // Test rounding. (Don't bother testing .5... we don't care which way it go es.)
17 pmf.setA(254.6f);
18 pmf.setR(204.3f);
19 pmf.setG(153.1f);
20 pmf.setB( 50.8f);
15 REPORTER_ASSERT(r, c == pmf.get()); 21 REPORTER_ASSERT(r, c == pmf.get());
16 22
17 // Test clamping. 23 // Test clamping.
18 SkPMFloat unclamped; 24 SkPMFloat unclamped;
19 unclamped.setA(+510.0f); 25 unclamped.setA(+510.0f);
20 unclamped.setR(+153.0f); 26 unclamped.setR(+153.0f);
21 unclamped.setG( +1.0f); 27 unclamped.setG( +1.0f);
22 unclamped.setB( -0.2f); 28 unclamped.setB( -0.2f);
23 29
24 SkPMFloat clamped; 30 SkPMFloat clamped;
25 clamped.set(unclamped.clamped()); 31 clamped.set(unclamped.clamped());
26 32
27 REPORTER_ASSERT(r, SkScalarNearlyEqual(255.0f, clamped.a())); 33 REPORTER_ASSERT(r, SkScalarNearlyEqual(255.0f, clamped.a()));
28 REPORTER_ASSERT(r, SkScalarNearlyEqual(153.0f, clamped.r())); 34 REPORTER_ASSERT(r, SkScalarNearlyEqual(153.0f, clamped.r()));
29 REPORTER_ASSERT(r, SkScalarNearlyEqual( 1.0f, clamped.g())); 35 REPORTER_ASSERT(r, SkScalarNearlyEqual( 1.0f, clamped.g()));
30 REPORTER_ASSERT(r, SkScalarNearlyEqual( 0.0f, clamped.b())); 36 REPORTER_ASSERT(r, SkScalarNearlyEqual( 0.0f, clamped.b()));
31 37
32 // Test SkPMFloat <-> Sk4f conversion. 38 // Test SkPMFloat <-> Sk4f conversion.
33 Sk4f fs = clamped; 39 Sk4f fs = clamped;
34 SkPMFloat scaled = fs.multiply(0.25f); 40 SkPMFloat scaled = fs.multiply(0.25f);
35 41
36 REPORTER_ASSERT(r, SkScalarNearlyEqual(63.75f, scaled.a())); 42 REPORTER_ASSERT(r, SkScalarNearlyEqual(63.75f, scaled.a()));
37 REPORTER_ASSERT(r, SkScalarNearlyEqual(38.25f, scaled.r())); 43 REPORTER_ASSERT(r, SkScalarNearlyEqual(38.25f, scaled.r()));
38 REPORTER_ASSERT(r, SkScalarNearlyEqual( 0.25f, scaled.g())); 44 REPORTER_ASSERT(r, SkScalarNearlyEqual( 0.25f, scaled.g()));
39 REPORTER_ASSERT(r, SkScalarNearlyEqual( 0.00f, scaled.b())); 45 REPORTER_ASSERT(r, SkScalarNearlyEqual( 0.00f, scaled.b()));
40 } 46 }
OLDNEW
« no previous file with comments | « src/opts/SkPMFloat_none.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698