[Mac] Add a new WebSandboxSupport method to get the display's color space.

Source/platform/image-decoders/ImageDecoder.h

 /*
  * Copyright (C) 2006 Apple Computer, Inc.  All rights reserved.
  * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 24 matching lines @@
 #include "platform/image-decoders/ImageFrame.h"
 #include "public/platform/Platform.h"
 #include "wtf/Assertions.h"
 #include "wtf/RefPtr.h"
 #include "wtf/Vector.h"
 #include "wtf/text/WTFString.h"
 
 #if USE(QCMSLIB)
 #include "qcms.h"
 #if OS(MACOSX)
+#include "public/platform/mac/WebSandboxSupport.h"
 #include <ApplicationServices/ApplicationServices.h>
-#include "wtf/RetainPtr.h"
 #endif
 #endif
 
 namespace blink {
 
 // ImagePlanes can be used to decode color components into provided buffers instead of using an ImageFrame.
 class PLATFORM_EXPORT ImagePlanes {
 public:
     ImagePlanes();
     ImagePlanes(void* planes[3], size_t rowBytes[3]);
@@ skipping 136 matching lines @@
         return !memcmp(&profileData[12], "mntr", 4) || !memcmp(&profileData[12], "scnr", 4);
     }
 
     class OutputDeviceProfile {
     public:
         OutputDeviceProfile()
             : m_outputDeviceProfile(0)
         {
             // FIXME: Add optional ICCv4 support.
 #if OS(MACOSX)
-            RetainPtr<CGColorSpaceRef> monitorColorSpace(AdoptCF, CGDisplayCopyColorSpace(CGMainDisplayID()));
-            CFDataRef iccProfile(CGColorSpaceCopyICCProfile(monitorColorSpace.get()));
+            CGColorSpaceRef monitorColorSpace = Platform::current()->sandboxSupport()->displayColorSpace();

[Noel Gordon, 2014/09/10 08:46:27]

As asked on the other review, is this thread-safe?  This code could be called
from the blink main-thread, or on any impl-side-painting thread.  Maybe that's
why the unit tests are complaining ...

+            CFDataRef iccProfile(CGColorSpaceCopyICCProfile(monitorColorSpace));
             if (iccProfile) {
                 size_t length = CFDataGetLength(iccProfile);
                 const unsigned char* systemProfile = CFDataGetBytePtr(iccProfile);
                 m_outputDeviceProfile = qcms_profile_from_memory(systemProfile, length);
                 CFRelease(iccProfile);
             }
 #else
             // FIXME: add support for multiple monitors.
             ColorProfile profile;
             screenColorProfile(profile);

[Noel Gordon, 2014/09/10 08:46:27]

This finds its  way to your https://codereview.chromium.org/549213004
codereview, right? 

Seems we might be able to delete all mac-specific stuff from ImageDecoder.h with
some more work in content/renderer/renderer_webkitplatformsupport_impl.cc, no?

             if (!profile.isEmpty())
                 m_outputDeviceProfile = qcms_profile_from_memory(profile.data(), profile.size());
 #endif
             if (m_outputDeviceProfile && qcms_profile_is_bogus(m_outputDeviceProfile)) {
                 qcms_profile_release(m_outputDeviceProfile);
                 m_outputDeviceProfile = 0;
             }
             if (!m_outputDeviceProfile)
                 m_outputDeviceProfile = qcms_profile_sRGB();
             if (m_outputDeviceProfile)
@@ skipping 98 matching lines @@
 
     IntSize m_size;
     bool m_sizeAvailable;
     bool m_isAllDataReceived;
     bool m_failed;
 };
 
 } // namespace blink
 
 #endif