Add PIXEL_FORMAT_I422.

media/remoting/rpc/proto_enum_utils.cc

Index: media/remoting/rpc/proto_enum_utils.cc
diff --git a/media/remoting/rpc/proto_enum_utils.cc b/media/remoting/rpc/proto_enum_utils.cc
index cea08c4fd9fd4985bafd785f81afc70a838c2b05..cefa2b5695bd94edba4c41d03a5604a055690e71 100644
--- a/media/remoting/rpc/proto_enum_utils.cc
+++ b/media/remoting/rpc/proto_enum_utils.cc
@@ -310,6 +310,7 @@ base::Optional<::media::VideoPixelFormat> ToMediaVideoPixelFormat(
   switch (value) {
     CASE_RETURN_OTHER(PIXEL_FORMAT_UNKNOWN);
     CASE_RETURN_OTHER(PIXEL_FORMAT_I420);
+    CASE_RETURN_OTHER(PIXEL_FORMAT_I422);

[erickung1, 2016/12/14 19:42:26]

any reason to add it here rather than add it at the end of switch case (line
#339) in order to make it consistent with enum value order ? since there is no
grouping logic in the switch case

same question in line #350

[jcliang, 2016/12/15 04:28:43]

Nope. I put it here only because the name was similar to I420. I'll move it to
the end to align with the enum value order.

     CASE_RETURN_OTHER(PIXEL_FORMAT_YV12);
     CASE_RETURN_OTHER(PIXEL_FORMAT_YV16);
     CASE_RETURN_OTHER(PIXEL_FORMAT_YV12A);
@@ -346,6 +347,7 @@ base::Optional<pb::VideoDecoderConfig::Format> ToProtoVideoDecoderConfigFormat(
   switch (value) {
     CASE_RETURN_OTHER(PIXEL_FORMAT_UNKNOWN);
     CASE_RETURN_OTHER(PIXEL_FORMAT_I420);
+    CASE_RETURN_OTHER(PIXEL_FORMAT_I422);
     CASE_RETURN_OTHER(PIXEL_FORMAT_YV12);
     CASE_RETURN_OTHER(PIXEL_FORMAT_YV16);
     CASE_RETURN_OTHER(PIXEL_FORMAT_YV12A);