commit 3bf7509c3f68aefbe5f8b2ea5888319a71a91d2d
author msarett <msarett@google.com> Wed Aug 24 06:22:41 2016 -0700
committer Commit bot <commit-bot@chromium.org> Wed Aug 24 06:22:41 2016 -0700
tree 009826a9c9d6ed6724a82adc6827b0b245ec8f03
parent 8aeec39f1fedfdd3508365ff0548cc3f876f6a00

Warn on bad rendering intents (instead of valid ones)

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2274843002

Review-Url: https://codereview.chromium.org/2274843002

src/core/SkColorSpace_ICC.cpp

diff --git a/src/core/SkColorSpace_ICC.cpp b/src/core/SkColorSpace_ICC.cpp
index 92dcc0f..88fd1b3 100755
--- a/src/core/SkColorSpace_ICC.cpp
+++ b/src/core/SkColorSpace_ICC.cpp
@@ -138,7 +138,7 @@
         // Should we treat different rendering intents differently?
         // Valid rendering intents include kPerceptual (0), kRelative (1),
         // kSaturation (2), and kAbsolute (3).
-        if (fRenderingIntent <= 3) {
+        if (fRenderingIntent > 3) {
             // Warn rather than fail here.  Occasionally, we see perfectly
             // normal profiles with wacky rendering intents.
             SkColorSpacePrintf("Warning, bad rendering intent.\n");