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

media/base/video_color_space.h

Index: media/base/video_color_space.h
diff --git a/media/base/video_color_space.h b/media/base/video_color_space.h
index 59ec578e1d22715dda4f4c76d27d42679a9169dd..010707e7e0a76ad95785a96682bfc317198ebf3d 100644
--- a/media/base/video_color_space.h
+++ b/media/base/video_color_space.h
@@ -70,6 +70,21 @@ class MEDIA_EXPORT VideoColorSpace {
     INVALID = 255,
   };
 
+  union PrimaryIDUnion {
+    PrimaryID primaries;
+    uint8_t primary_id;
+  };
+
+  union TransferIDUnion {
+    TransferID transfer;
+    uint8_t transfer_id;
+  };
+
+  union MatrixIDUnion {
+    MatrixID matrix;
+    uint8_t matrix_id;
+  };
+
   VideoColorSpace();
   VideoColorSpace(int primaries,
                   int transfer,
@@ -99,6 +114,9 @@ class MEDIA_EXPORT VideoColorSpace {
   PrimaryID primaries = PrimaryID::INVALID;
   TransferID transfer = TransferID::INVALID;
   MatrixID matrix = MatrixID::INVALID;

[a.suchit2, 2017/04/25 14:05:08]

For the small usage of enum, we would not required union.
But if we are doing it then above 3 variables are not needed.
Please remove them.

[Uzair, 2017/04/25 14:12:34]

I am not sure about any other way without using union we can convert enum class
to int.
Please can you let me know which variables are not needed, union variables or
enum class
variables ?

[a.suchit2, 2017/04/25 14:45:51]

Enum variables : lines 114,115,116

[a.suchit2, 2017/04/25 14:45:51]

Enum variables : lines 114,115,116

+  PrimaryIDUnion primaries_union;
+  TransferIDUnion transfer_union;
+  MatrixIDUnion matrix_union;
   gfx::ColorSpace::RangeID range = gfx::ColorSpace::RangeID::INVALID;
 };