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

media/base/video_color_space.cc

Index: media/base/video_color_space.cc
diff --git a/media/base/video_color_space.cc b/media/base/video_color_space.cc
index 036cd57f92050a4d55eaccd4fdc0fbc3c7bf7c88..e44d242428a2c53603f7305b8ec108851bddea8f 100644
--- a/media/base/video_color_space.cc
+++ b/media/base/video_color_space.cc
@@ -54,10 +54,136 @@ bool VideoColorSpace::operator!=(const VideoColorSpace& other) const {
 }
 
 gfx::ColorSpace VideoColorSpace::ToGfxColorSpace() const {
-  // TODO(hubbe): Make this type-safe.
-  return gfx::ColorSpace::CreateVideo(static_cast<int>(primaries),

[hubbe, 2017/05/02 17:15:44]

We should remove the CreateVideo() function from gfx::ColorSpace() (or at least
add a TODO there for removing it.)

[Uzair, 2017/05/03 08:35:39]

Removed CreateVideo() function from gfx::ColorSpace().

-                                      static_cast<int>(transfer),
-                                      static_cast<int>(matrix), range);
+  gfx::ColorSpace::PrimaryID primary_id;
+  gfx::ColorSpace::TransferID transfer_id;
+  gfx::ColorSpace::MatrixID matrix_id;
+
+  switch (primaries) {
+    default:

[hubbe, 2017/05/02 17:15:44]

remove default in all of these switch statements.

Without the default, the compiler will complain if someone adds additional
values without updating these switch statements, which is what we want.

[Uzair, 2017/05/03 08:35:39]

Done.

+    case PrimaryID::INVALID:
+    case PrimaryID::BT709:
+    case PrimaryID::UNSPECIFIED:
+      primary_id = gfx::ColorSpace::PrimaryID::BT709;
+      break;
+    case PrimaryID::BT470M:
+      primary_id = gfx::ColorSpace::PrimaryID::BT470M;
+      break;
+    case PrimaryID::BT470BG:
+      primary_id = gfx::ColorSpace::PrimaryID::BT470BG;
+      break;
+    case PrimaryID::SMPTE170M:
+      primary_id = gfx::ColorSpace::PrimaryID::SMPTE170M;
+      break;
+    case PrimaryID::SMPTE240M:
+      primary_id = gfx::ColorSpace::PrimaryID::SMPTE240M;
+      break;
+    case PrimaryID::FILM:
+      primary_id = gfx::ColorSpace::PrimaryID::FILM;
+      break;
+    case PrimaryID::BT2020:
+      primary_id = gfx::ColorSpace::PrimaryID::BT2020;
+      break;
+    case PrimaryID::SMPTEST428_1:
+      primary_id = gfx::ColorSpace::PrimaryID::SMPTEST428_1;
+      break;
+    case PrimaryID::SMPTEST431_2:
+      primary_id = gfx::ColorSpace::PrimaryID::SMPTEST431_2;
+      break;
+    case PrimaryID::SMPTEST432_1:
+      primary_id = gfx::ColorSpace::PrimaryID::SMPTEST432_1;
+      break;
+  }
+
+  switch (transfer) {
+    default:
+    case TransferID::INVALID:
+    case TransferID::BT709:
+    case TransferID::UNSPECIFIED:
+      transfer_id = gfx::ColorSpace::TransferID::BT709;
+      break;
+    case TransferID::GAMMA22:
+      transfer_id = gfx::ColorSpace::TransferID::GAMMA22;
+      break;
+    case TransferID::GAMMA28:
+      transfer_id = gfx::ColorSpace::TransferID::GAMMA28;
+      break;
+    case TransferID::SMPTE170M:
+      transfer_id = gfx::ColorSpace::TransferID::SMPTE170M;
+      break;
+    case TransferID::SMPTE240M:
+      transfer_id = gfx::ColorSpace::TransferID::SMPTE240M;
+      break;
+    case TransferID::LINEAR:
+      transfer_id = gfx::ColorSpace::TransferID::LINEAR;
+      break;
+    case TransferID::LOG:
+      transfer_id = gfx::ColorSpace::TransferID::LOG;
+      break;
+    case TransferID::LOG_SQRT:
+      transfer_id = gfx::ColorSpace::TransferID::LOG_SQRT;
+      break;
+    case TransferID::IEC61966_2_4:
+      transfer_id = gfx::ColorSpace::TransferID::IEC61966_2_4;
+      break;
+    case TransferID::BT1361_ECG:
+      transfer_id = gfx::ColorSpace::TransferID::BT1361_ECG;
+      break;
+    case TransferID::IEC61966_2_1:
+      transfer_id = gfx::ColorSpace::TransferID::IEC61966_2_1;
+      break;
+    case TransferID::BT2020_10:
+      transfer_id = gfx::ColorSpace::TransferID::BT2020_10;
+      break;
+    case TransferID::BT2020_12:
+      transfer_id = gfx::ColorSpace::TransferID::BT2020_12;
+      break;
+    case TransferID::SMPTEST2084:
+      transfer_id = gfx::ColorSpace::TransferID::SMPTEST2084;
+      break;
+    case TransferID::SMPTEST428_1:
+      transfer_id = gfx::ColorSpace::TransferID::SMPTEST428_1;
+      break;
+    case TransferID::ARIB_STD_B67:
+      transfer_id = gfx::ColorSpace::TransferID::ARIB_STD_B67;
+      break;
+  }
+
+  switch (matrix) {
+    case MatrixID::RGB:
+      matrix_id = gfx::ColorSpace::MatrixID::RGB;
+      break;
+    default:
+    case MatrixID::BT709:
+    case MatrixID::UNSPECIFIED:
+      matrix_id = gfx::ColorSpace::MatrixID::BT709;
+      break;
+    case MatrixID::FCC:
+      matrix_id = gfx::ColorSpace::MatrixID::FCC;
+      break;
+    case MatrixID::BT470BG:
+      matrix_id = gfx::ColorSpace::MatrixID::BT470BG;
+      break;
+    case MatrixID::SMPTE170M:
+      matrix_id = gfx::ColorSpace::MatrixID::SMPTE170M;
+      break;
+    case MatrixID::SMPTE240M:
+      matrix_id = gfx::ColorSpace::MatrixID::SMPTE240M;
+      break;
+    case MatrixID::YCOCG:
+      matrix_id = gfx::ColorSpace::MatrixID::YCOCG;
+      break;
+    case MatrixID::BT2020_NCL:
+      matrix_id = gfx::ColorSpace::MatrixID::BT2020_NCL;
+      break;
+    case MatrixID::BT2020_CL:
+      matrix_id = gfx::ColorSpace::MatrixID::BT2020_CL;
+      break;
+    case MatrixID::YDZDX:
+      matrix_id = gfx::ColorSpace::MatrixID::YDZDX;
+      break;
+  }
+
+  return gfx::ColorSpace(primary_id, transfer_id, matrix_id, range);
 }
 
 VideoColorSpace VideoColorSpace::REC709() {