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

Unified Diff: ui/gfx/icc_profile_unittest.cc

Issue 2876463003: color: Add unit test to ensure sRGB SkColorSpaces match (Closed)
Patch Set: Created 3 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/gfx/icc_profile_unittest.cc
diff --git a/ui/gfx/icc_profile_unittest.cc b/ui/gfx/icc_profile_unittest.cc
index ecb17139b9f10d2a3692a897bcc3a876307270fa..57a52c0c5b83a0254dfebb022d0431b3b45bd52b 100644
--- a/ui/gfx/icc_profile_unittest.cc
+++ b/ui/gfx/icc_profile_unittest.cc
@@ -22,10 +22,17 @@ TEST(ICCProfile, Conversions) {
}
TEST(ICCProfile, SRGB) {
+ ICCProfile icc_profile = ICCProfileForTestingSRGB();
ColorSpace color_space = ColorSpace::CreateSRGB();
sk_sp<SkColorSpace> sk_color_space = SkColorSpace::MakeSRGB();
- // These should be the same pointer, not just equal.
+ // The ICC profile parser should note that this is SRGB.
+ EXPECT_EQ(icc_profile.GetColorSpace().ToSkColorSpace().get(),
+ sk_color_space.get());
+ // The parametric generating code should recognize that this is SRGB.
+ EXPECT_EQ(icc_profile.GetParametricColorSpace().ToSkColorSpace().get(),
+ sk_color_space.get());
+ // The generated color space should recognize that this is SRGB.
EXPECT_EQ(color_space.ToSkColorSpace().get(), sk_color_space.get());
}
« 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