Removed non-Identity matrix in non-PCSXYZ Input check for ICC profiles

The presence seems to be disallowed in the specs (10.8/10.9), but two
test images on the ICC website have them, and fail if we abort. Replaced
the failure with a print warning.

BUG=skia:

Change-Id: I463f1a8861eaa0eb62081c3213017dba9012c890
Reviewed-on: https://skia-review.googlesource.com/5658
Reviewed-by: Matt Sarett <msarett@google.com>
Commit-Queue: Robert Aftias <raftias@google.com>

1 files changed, 5 insertions, 2 deletions

diff --git a/src/core/SkColorSpace_ICC.cpp b/src/core/SkColorSpace_ICC.cpp
index 04e240a6de..2ae1090483 100644
--- a/src/core/SkColorSpace_ICC.cpp
+++ b/src/core/SkColorSpace_ICC.cpp
@@ -1113,8 +1113,11 @@ bool load_a2b0_lutn_type(std::vector<SkColorSpace_A2B::Element>* elements, const
         // ICC specs (10.8/10.9) say lut8/16Type profiles must have identity matrices
         // if the input color space is not PCSXYZ, and we do not support PCSXYZ input color spaces
         // so we should never encounter a non-identity matrix here.
-        SkColorSpacePrintf("non-Identity matrix found in non-XYZ input color space lut profile");
-        return false;
+        // However, 2 test images from the ICC website have RGB input spaces and non-identity
+        // matrices so we're not going to fail here, despite being against the spec.
+        SkColorSpacePrintf("Warning: non-Identity matrix found in non-XYZ input color space"
+                           "lut profile");
+        elements->push_back(SkColorSpace_A2B::Element(matrix));
     }
 
     size_t dataOffset      = 48;