Converting video color space enums to their corresponding gfx:color space eqvivalent.

ui/gfx/color_transform_unittest.cc

 // Copyright (c) 2016 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 "base/logging.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gfx/color_space.h"
 #include "ui/gfx/color_transform.h"
 #include "ui/gfx/icc_profile.h"
 #include "ui/gfx/test/icc_profiles.h"
@@ skipping 295 matching lines @@
 
   tmp = ColorTransform::TriStim(0.0f, 0.0f, 1.0f);
   t->Transform(&tmp, 1);
   EXPECT_NEAR(tmp.x(), 0.0f, kEpsilon);
   EXPECT_NEAR(tmp.y(), 0.0f, kEpsilon);
   EXPECT_NEAR(tmp.z(), 1.0f, kEpsilon);
 }
 
 TEST(SimpleColorSpace, UnknownVideoToSRGB) {
   // Invalid video spaces should be BT709.
-  ColorSpace unknown = gfx::ColorSpace::CreateVideo(
-      -1, -1, -1, gfx::ColorSpace::RangeID::LIMITED);
+  ColorSpace unknown = gfx::ColorSpace(
+      gfx::ColorSpace::PrimaryID::INVALID, gfx::ColorSpace::TransferID::INVALID,
+      gfx::ColorSpace::MatrixID::INVALID, gfx::ColorSpace::RangeID::LIMITED);

[ccameron, 2017/05/03 14:56:24]

I think that this test will fail because INVALID will not be treated as BT709.

You can just delete this test, and add a test in media/ that unspecified values
are interpreted correctly.

   ColorSpace sRGB = ColorSpace::CreateSRGB();
   std::unique_ptr<ColorTransform> t(ColorTransform::NewColorTransform(
       unknown, sRGB, ColorTransform::Intent::INTENT_PERCEPTUAL));
 
   ColorTransform::TriStim tmp(16.0f / 255.0f, 0.5f, 0.5f);
   t->Transform(&tmp, 1);
   EXPECT_NEAR(tmp.x(), 0.0f, 0.001f);
   EXPECT_NEAR(tmp.y(), 0.0f, 0.001f);
   EXPECT_NEAR(tmp.z(), 0.0f, 0.001f);
 
@@ skipping 299 matching lines @@
 
 INSTANTIATE_TEST_CASE_P(
     C,
     ColorSpaceTest,
     testing::Combine(testing::ValuesIn(all_primaries),
                      testing::Values(ColorSpace::TransferID::BT709),
                      testing::ValuesIn(all_matrices),
                      testing::ValuesIn(all_ranges),
                      testing::ValuesIn(intents)));
 }  // namespace