[qcms] Allow negative XYZ for display profiles on the APPLE port

third_party/qcms/src/iccread.c

 /* vim: set ts=8 sw=8 noexpandtab: */
 //  qcms
 //  Copyright (C) 2009 Mozilla Foundation
 //  Copyright (C) 1998-2007 Marti Maria
 //
 // Permission is hereby granted, free of charge, to any person obtaining 
 // a copy of this software and associated documentation files (the "Software"), 
 // to deal in the Software without restriction, including without limitation 
 // the rights to use, copy, modify, merge, publish, distribute, sublicense, 
 // and/or sell copies of the Software, and to permit persons to whom the Software 
@@ skipping 235 matching lines @@
                 for (i = 0; i < index.count; i++) {
                         index.tags[i].signature = read_u32(mem, 128 + 4 + 4*i*3);
                         index.tags[i].offset    = read_u32(mem, 128 + 4 + 4*i*3 + 4);
                         index.tags[i].size      = read_u32(mem, 128 + 4 + 4*i*3 + 8);
                 }
         }
 
         return index;
 }
 
-// Checks a profile for obvious inconsistencies and returns
-// true if the profile looks bogus and should probably be
-// ignored.
+/* Checks a profile for obvious inconsistencies and return true if the
+ * profile looks bogus and should probably be ignored.
+ */
 qcms_bool qcms_profile_is_bogus(qcms_profile *profile)
 {
-       float sum[3], target[3], tolerance[3];
-       float rX, rY, rZ, gX, gY, gZ, bX, bY, bZ;
-       bool negative;
-       unsigned i;
+        float rX, rY, rZ, gX, gY, gZ, bX, bY, bZ;
+        float target[3], tolerance[3], sum[3];
+        unsigned i;
 
-       // We currently only check the bogosity of RGB profiles
-       if (profile->color_space != RGB_SIGNATURE)
-               return false;
-
-       if (qcms_supports_iccv4 && (profile->A2B0 || profile->B2A0))
+        // We currently only check the bogosity of RGB profiles.
+        if (profile->color_space != RGB_SIGNATURE)
                return false;
 
-       rX = s15Fixed16Number_to_float(profile->redColorant.X);
-       rY = s15Fixed16Number_to_float(profile->redColorant.Y);
-       rZ = s15Fixed16Number_to_float(profile->redColorant.Z);
+        if (qcms_supports_iccv4 && (profile->A2B0 || profile->B2A0))
+               return false;
 
-       gX = s15Fixed16Number_to_float(profile->greenColorant.X);
-       gY = s15Fixed16Number_to_float(profile->greenColorant.Y);
-       gZ = s15Fixed16Number_to_float(profile->greenColorant.Z);
+        rX = s15Fixed16Number_to_float(profile->redColorant.X);
+        rY = s15Fixed16Number_to_float(profile->redColorant.Y);
+        rZ = s15Fixed16Number_to_float(profile->redColorant.Z);
 
-       bX = s15Fixed16Number_to_float(profile->blueColorant.X);
-       bY = s15Fixed16Number_to_float(profile->blueColorant.Y);
-       bZ = s15Fixed16Number_to_float(profile->blueColorant.Z);
+        gX = s15Fixed16Number_to_float(profile->greenColorant.X);
+        gY = s15Fixed16Number_to_float(profile->greenColorant.Y);
+        gZ = s15Fixed16Number_to_float(profile->greenColorant.Z);
 
-       // Check if any of the XYZ values are negative (see mozilla bug 498245)
-       // CIEXYZ tristimulus values cannot be negative according to the spec.
-       negative =
-               (rX < 0) || (rY < 0) || (rZ < 0) ||
-               (gX < 0) || (gY < 0) || (gZ < 0) ||
-               (bX < 0) || (bY < 0) || (bZ < 0);
+        bX = s15Fixed16Number_to_float(profile->blueColorant.X);
+        bY = s15Fixed16Number_to_float(profile->blueColorant.Y);
+        bZ = s15Fixed16Number_to_float(profile->blueColorant.Z);
 
-       if (negative)
-               return true;
+        // Build our target vector: CIE D50 white. See also mozilla bug 460629,
+        // and http://www.color.org/whyd50.xalter "Why is the media white point
+        // of a display profile always D50?"
 
+        target[0] = (float) 0.96420;
+        target[1] = (float) 1.00000;
+        target[2] = (float) 0.82491;
 
-       // Sum the values; they should add up to something close to white
-       sum[0] = rX + gX + bX;
-       sum[1] = rY + gY + bY;
-       sum[2] = rZ + gZ + bZ;
+        // Our tolerance vector - Recommended by Chris Murphy [1] based on
+        // conversion from the L*a*b space criterion of no more than 3 in any
+        // one channel. This is similar to, but slightly more tolerant than
+        // Adobe's criterion. [1] https://bugzil.la/460629#c10
 
-#if defined (_MSC_VER)
-#pragma warning(push)
-/* Disable double to float truncation warning 4305 */
-#pragma warning(disable:4305)
+        tolerance[0] = (float) 0.02;
+        tolerance[1] = (float) 0.02;
+        tolerance[2] = (float) 0.04;
+
+        // Sum the XYZ values: they should add to D50 white, within tolerance.
+
+        // FIXME: this test assumes the TRC RGB curves equal 1.0 for the white
+        // input (255,255,255) RGB test color. For user display profiles, that
+        // is the normal case. Profiles with abnormal TRC exist. A better test
+        // would transform 255,255,255 white through the profile to either XYZ
+        // or L*a*b color and compare the result to D50 in XYZ or L*a*b color.
+
+        sum[0] = rX + gX + bX;
+        sum[1] = rY + gY + bY;
+        sum[2] = rZ + gZ + bZ;
+
+        for (i = 0; i < 3; ++i) {
+            if (!(((sum[i] - tolerance[i]) <= target[i]) &&
+                  ((sum[i] + tolerance[i]) >= target[i]))) {
+                return true; // out of tolerance: bogus
+            }
+        }
+
+#ifndef __APPLE__
+
+        // Check if any of the XYZ values are negative (see mozilla bug 498245)
+        // CIEXYZ tristimulus values cannot be negative according to the spec.
+
+        bool negative =
+            (rX < 0) || (rY < 0) || (rZ < 0) ||
+            (gX < 0) || (gY < 0) || (gZ < 0) ||
+            (bX < 0) || (bY < 0) || (bZ < 0);
+
+        if (negative)
+            return true; // bogus
+#else
+        // Chromatic adaption to D50 can result in negative XYZ, but the white
+        // point D50 tolerance test has passed. Accept negative values herein.
+        // See https://bugzilla.mozilla.org/show_bug.cgi?id=498245#c18 onwards
+        // for discussion about whether profile XYZ can or cannot be negative,
+        // per the spec. Also the https://bugzil.la/450923 user report.
+
+        // FIXME: allow this relaxation on all ports?
 #endif
-       // Build our target vector (see mozilla bug 460629)
-       target[0] = 0.96420;
-       target[1] = 1.00000;
-       target[2] = 0.82491;
-
-       // Our tolerance vector - Recommended by Chris Murphy based on
-       // conversion from the LAB space criterion of no more than 3 in any one
-       // channel. This is similar to, but slightly more tolerant than Adobe's
-       // criterion.
-       tolerance[0] = 0.02;
-       tolerance[1] = 0.02;
-       tolerance[2] = 0.04;
-
-#if defined (_MSC_VER)
-/* Restore warnings */
-#pragma warning(pop)
-#endif
-       // Compare with our tolerance
-       for (i = 0; i < 3; ++i) {
-           if (!(((sum[i] - tolerance[i]) <= target[i]) &&
-                 ((sum[i] + tolerance[i]) >= target[i])))
-               return true;
-       }
-
-       // All Good
-       return false;
+        // All good.
+        return false;
 }
 
 #define TAG_bXYZ 0x6258595a
 #define TAG_gXYZ 0x6758595a
 #define TAG_rXYZ 0x7258595a
 #define TAG_rTRC 0x72545243
 #define TAG_bTRC 0x62545243
 #define TAG_gTRC 0x67545243
 #define TAG_kTRC 0x6b545243
 #define TAG_A2B0 0x41324230
@@ skipping 1204 matching lines @@
 {
         qcms_profile *profile = NULL;
         FILE *file = _wfopen(path, L"rb");
         if (file) {
                 profile = qcms_profile_from_file(file);
                 fclose(file);
         }
         return profile;
 }
 #endif