Add usage metrics for display and image color profiles

third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp

 /*
  * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
  *
  *  This library is free software; you can redistribute it and/or
  *  modify it under the terms of the GNU Library General Public
  *  License as published by the Free Software Foundation; either
  *  version 2 of the License, or (at your option) any later version.
  *
  *  This library is distributed in the hope that it will be useful,
  *  but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 226 matching lines @@
         // this frame is a blank frame, so it can again be decoded alone.
         // Otherwise, the previous frame contributes to this frame.
         return (prevBuffer->originalFrameRect().contains(IntRect(IntPoint(), size()))
             || (prevBuffer->requiredPreviousFrameIndex() == kNotFound)) ? kNotFound : prevFrame;
     default:
         ASSERT_NOT_REACHED();
         return kNotFound;
     }
 }
 
+#if USE(QCMSLIB)
+ImageDecoder::OutputDeviceProfile::OutputDeviceProfile() : m_outputDeviceProfile(0)
+{
+    ColorProfileStatus status = ColorProfileSuccess;
+
+    ColorProfile profile = screenColorProfile();
+    if (!profile.isEmpty()) {
+        if (ImageDecoder::isICCv4(profile.data(), profile.size())) {
+            status = ColorProfileIsV4;

[Noel Gordon, 2015/12/23 15:28:48]

We don't seem to actually read / use the profile in this case :)  Any reason?

Perhaps we should just record the version of the display profile in another
stat, using qcms_profile_get_version(m_outputDeviceProfile) and record that if
needed?

[Justin Novosad, 2015/12/23 15:58:10]

The prupose is to track the prevalence of ICC v4 profiles in the wild in order
to justify implementing (or not implementing?) ICC v4 support.
Good idea. I will create a separate histogram just for profile versions

+        } else {
+            m_outputDeviceProfile = qcms_profile_from_memory(profile.data(), profile.size());
+            if (!m_outputDeviceProfile) {
+                status = ColorProfileFailedDecode;

[Noel Gordon, 2015/12/23 15:28:48]

ColorProfileFailedDecode -> ColorProfileInvalidContent

the profile has a size, but the content is invalid in some way, unsupported, bad
tags, bad PCS ...

+            } else {
+                if (qcms_profile_is_bogus(m_outputDeviceProfile)) {
+                    status = ColorProfileIsBogus;

[Noel Gordon, 2015/12/23 15:28:48]

ColorProfileIsBogus -> ColorProfileInvalidProfile

let's not use bogus :) ... the profile has a size and valid content, but we
can't use it as a display profile for reasons known to us (bogosity, bad
version, or other factors we might add later, etc).

+                    qcms_profile_release(m_outputDeviceProfile);
+                    m_outputDeviceProfile = 0;
+                }
+            }
+        }
+    } else {
+        status = ColorProfileNone;

[Noel Gordon, 2015/12/23 15:28:47]

ColorProfileNone -> ColorProfileEmpty

the profile has no size, or is empty.

+    }
+
+    Platform::current()->histogramEnumeration("ColorManagement.ColorProfileStatus.Display", status, ColorProfileStatus::ValueCount);
+
+    if (!m_outputDeviceProfile)
+        m_outputDeviceProfile = qcms_profile_sRGB();
+    if (m_outputDeviceProfile)
+        qcms_profile_precache_output_transform(m_outputDeviceProfile);
+}
+#endif
+
 ImagePlanes::ImagePlanes()
 {
     for (int i = 0; i < 3; ++i) {
         m_planes[i] = 0;
         m_rowBytes[i] = 0;
     }
 }
 
 ImagePlanes::ImagePlanes(void* planes[3], size_t rowBytes[3])
 {
     for (int i = 0; i < 3; ++i) {
         m_planes[i] = planes[i];
         m_rowBytes[i] = rowBytes[i];
     }
 }
 
 void* ImagePlanes::plane(int i)
 {
     ASSERT((i >= 0) && i < 3);
     return m_planes[i];
 }
 
 size_t ImagePlanes::rowBytes(int i) const
 {
     ASSERT((i >= 0) && i < 3);
     return m_rowBytes[i];
 }
 
 } // namespace blink