cc: Add conversion between gfx::ColorSpace and SkColorSpace

ui/gfx/color_space.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/gfx/color_space.h"
 
 #include <map>
 
 #include "base/lazy_instance.h"
 #include "base/synchronization/lock.h"
+#include "third_party/skia/include/core/SkColorSpace.h"
 #include "ui/gfx/icc_profile.h"
 
 namespace gfx {
 
 ColorSpace::ColorSpace() {
   memset(custom_primary_matrix_, 0, sizeof(custom_primary_matrix_));
 }
 
 ColorSpace::ColorSpace(PrimaryID primaries,
                        TransferID transfer,
@@ skipping 32 matching lines @@
 }
 
 // static
 ColorSpace ColorSpace::CreateREC709() {
   return ColorSpace(PrimaryID::BT709, TransferID::BT709, MatrixID::BT709,
                     RangeID::LIMITED);
 }
 
 bool ColorSpace::operator==(const ColorSpace& other) const {
   return primaries_ == other.primaries_ && transfer_ == other.transfer_ &&
-         matrix_ == other.matrix_ && range_ == other.range_;
+         matrix_ == other.matrix_ && range_ == other.range_ &&
+         !memcmp(custom_primary_matrix_, other.custom_primary_matrix_,

[hubbe, 2016/09/19 19:21:27]

We should only do this if primaries_ == CUSTOM I think.

[ccameron, 2016/09/19 22:23:15]

Done.

+                 sizeof(custom_primary_matrix_));
 }
 
 bool ColorSpace::operator!=(const ColorSpace& other) const {
   return !(*this == other);
 }
 
 bool ColorSpace::operator<(const ColorSpace& other) const {
   if (primaries_ < other.primaries_)
     return true;
   if (primaries_ > other.primaries_)
     return false;
   if (transfer_ < other.transfer_)
     return true;
   if (transfer_ > other.transfer_)
     return false;
   if (matrix_ < other.matrix_)
     return true;
   if (matrix_ > other.matrix_)
     return false;
   if (range_ < other.range_)
     return true;
   if (range_ > other.range_)
+    return false;
+  int primary_result =
+      memcmp(custom_primary_matrix_, other.custom_primary_matrix_,
+             sizeof(custom_primary_matrix_));
+  if (primary_result < 0)
     return true;
-
-  // TODO(hubbe): For "CUSTOM" primaries or tranfer functions, compare their
-  // coefficients here
+  if (primary_result > 0)
+    return false;
 
   return false;
 }
 
+sk_sp<SkColorSpace> ColorSpace::ToSkColorSpace() const {
+  // TODO(crbug.com/634102): Make this conversion more robust.
+  std::vector<char> icc_data =
+      gfx::ICCProfile::FromColorSpace(*this).GetData();

[hubbe, 2016/09/19 19:21:27]

1) Shouldn't we implement FromColorSpace() first?
2) This will probably never work properly for non-RGB color profiles (YUV in
particular)

[ccameron, 2016/09/19 22:23:15]

This is to un-block work on rasterization, and rasterization targets always come
from an ICC profile (from the monitor), so this covers the one case that is
important right now.
If we have to return nullptr for some spaces then that is reasonable (IIUC
software compositing of YUV also doesn't exist yet).

[hubbe, 2016/09/19 22:53:19]

Then maybe remove the DCHECK and tell callers to expect null values? I agree
that it will work for your use case, but there is no sensible way for a random
caller to know what to expect right now.

+  sk_sp<SkColorSpace> result =
+      SkColorSpace::NewICC(icc_data.data(), icc_data.size());
+  DCHECK(result);
+  return result;
+}
+
 }  // namespace gfx