Add PIXEL_FORMAT_I422.

media/remoting/rpc/proto_enum_utils.cc

 // Copyright 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 "media/remoting/rpc/proto_enum_utils.h"
 
 namespace media {
 namespace remoting {
 
 #define CASE_RETURN_OTHER(x) \
@@ skipping 292 matching lines @@
   return base::nullopt;  // Not a 'default' to ensure compile-time checks.
 }
 
 base::Optional<::media::VideoPixelFormat> ToMediaVideoPixelFormat(
     pb::VideoDecoderConfig::Format value) {
   using OriginType = pb::VideoDecoderConfig;
   using OtherType = ::media::VideoPixelFormat;
   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);
     CASE_RETURN_OTHER(PIXEL_FORMAT_YV24);
     CASE_RETURN_OTHER(PIXEL_FORMAT_NV12);
     CASE_RETURN_OTHER(PIXEL_FORMAT_NV21);
     CASE_RETURN_OTHER(PIXEL_FORMAT_UYVY);
     CASE_RETURN_OTHER(PIXEL_FORMAT_YUY2);
     CASE_RETURN_OTHER(PIXEL_FORMAT_ARGB);
     CASE_RETURN_OTHER(PIXEL_FORMAT_XRGB);
@@ skipping 16 matching lines @@
   return base::nullopt;  // Not a 'default' to ensure compile-time checks.
 }
 
 base::Optional<pb::VideoDecoderConfig::Format> ToProtoVideoDecoderConfigFormat(
     ::media::VideoPixelFormat value) {
   using OriginType = ::media::VideoPixelFormat;
   using OtherType = pb::VideoDecoderConfig;
   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);
     CASE_RETURN_OTHER(PIXEL_FORMAT_YV24);
     CASE_RETURN_OTHER(PIXEL_FORMAT_NV12);
     CASE_RETURN_OTHER(PIXEL_FORMAT_NV21);
     CASE_RETURN_OTHER(PIXEL_FORMAT_UYVY);
     CASE_RETURN_OTHER(PIXEL_FORMAT_YUY2);
     CASE_RETURN_OTHER(PIXEL_FORMAT_ARGB);
     CASE_RETURN_OTHER(PIXEL_FORMAT_XRGB);
@@ skipping 221 matching lines @@
   switch (value) {
     CASE_RETURN_OTHER(kOk);
     CASE_RETURN_OTHER(kAborted);
     CASE_RETURN_OTHER(kConfigChanged);
   }
   return base::nullopt;  // Not a 'default' to ensure compile-time checks.
 }
 
 }  // namespace remoting
 }  // namespace media