commit 0e6274f54084d816abd70d445dded8657eb01abd
author msarett <msarett@google.com> Mon Mar 21 08:04:40 2016 -0700
committer Commit bot <commit-bot@chromium.org> Mon Mar 21 08:04:40 2016 -0700
tree c80aa88eeaa6ef846df754e1c1ac44d8d8ea8f83
parent 85c9447d4da7b9a12b0fe2894267bd0dc7e6d0f8

Parse icc profiles and exif orientation from jpeg markers

New resources should be fine to add since they are already
checked into chromium.

BUG=skia:3834
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1813273002

Review URL: https://codereview.chromium.org/1813273002

tests/ColorSpaceTest.cpp

diff --git a/tests/ColorSpaceTest.cpp b/tests/ColorSpaceTest.cpp
index 3361aa3..37785e2 100644
--- a/tests/ColorSpaceTest.cpp
+++ b/tests/ColorSpaceTest.cpp
@@ -17,13 +17,11 @@
     return SkStream::NewFromFile(fullPath.c_str());
 }
 
-#if (PNG_LIBPNG_VER_MAJOR > 1) || (PNG_LIBPNG_VER_MAJOR == 1 && PNG_LIBPNG_VER_MINOR >= 6)
 static bool almost_equal(float a, float b) {
-    return SkTAbs(a - b) < 0.0001f;
+    return SkTAbs(a - b) < 0.001f;
 }
-#endif
 
-DEF_TEST(ColorSpaceParseICCProfile, r) {
+DEF_TEST(ColorSpaceParsePngICCProfile, r) {
     SkAutoTDelete<SkStream> stream(resource("color_wheel_with_profile.png"));
     REPORTER_ASSERT(r, nullptr != stream);
 
@@ -55,3 +53,35 @@
     REPORTER_ASSERT(r, almost_equal(0.714096f, xyz.fMat[8]));
 #endif
 }
+
+DEF_TEST(ColorSpaceParseJpegICCProfile, r) {
+    SkAutoTDelete<SkStream> stream(resource("icc-v2-gbr.jpg"));
+    REPORTER_ASSERT(r, nullptr != stream);
+
+    SkAutoTDelete<SkCodec> codec(SkCodec::NewFromStream(stream.release()));
+    REPORTER_ASSERT(r, nullptr != codec);
+
+    SkColorSpace* colorSpace = codec->getColorSpace();
+    REPORTER_ASSERT(r, nullptr != colorSpace);
+
+    // It's important to use almost equal here.  This profile sets gamma as
+    // 563 / 256, which actually comes out to about 2.19922.
+    SkFloat3 gammas = colorSpace->gamma();
+    REPORTER_ASSERT(r, almost_equal(2.2f, gammas.fVec[0]));
+    REPORTER_ASSERT(r, almost_equal(2.2f, gammas.fVec[1]));
+    REPORTER_ASSERT(r, almost_equal(2.2f, gammas.fVec[2]));
+
+    // These nine values were extracted from the color profile.  Until we know any
+    // better, we'll assume these are the right values and test that we continue
+    // to extract them properly.
+    SkFloat3x3 xyz = colorSpace->xyz();
+    REPORTER_ASSERT(r, almost_equal(0.385117f, xyz.fMat[0]));
+    REPORTER_ASSERT(r, almost_equal(0.716904f, xyz.fMat[1]));
+    REPORTER_ASSERT(r, almost_equal(0.0970612f, xyz.fMat[2]));
+    REPORTER_ASSERT(r, almost_equal(0.143051f, xyz.fMat[3]));
+    REPORTER_ASSERT(r, almost_equal(0.0606079f, xyz.fMat[4]));
+    REPORTER_ASSERT(r, almost_equal(0.713913f, xyz.fMat[5]));
+    REPORTER_ASSERT(r, almost_equal(0.436035f, xyz.fMat[6]));
+    REPORTER_ASSERT(r, almost_equal(0.222488f, xyz.fMat[7]));
+    REPORTER_ASSERT(r, almost_equal(0.013916f, xyz.fMat[8]));
+}