Fix support for yuv_422 pixel format.

media/ffmpeg/ffmpeg_common.cc

 // Copyright (c) 2011 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/ffmpeg/ffmpeg_common.h"
 
 #include "base/logging.h"
 
 namespace media {
 
@@ skipping 197 matching lines @@
       // from the number of channels, otherwise report errors.
       if (channels == 1)
         return CHANNEL_LAYOUT_MONO;
       if (channels == 2)
         return CHANNEL_LAYOUT_STEREO;
       DLOG(WARNING) << "Unsupported/unencountered channel layout values";
       return CHANNEL_LAYOUT_UNSUPPORTED;
   }
 }
 
+VideoFrame::Format PixFmtToVideoFormat(PixelFormat pix_fmt) {
+  switch (pix_fmt) {
+    case PIX_FMT_YUV422P:
+      return VideoFrame::YV16;
+    case PIX_FMT_YUV420P:
+      return VideoFrame::YV12;
+    default:
+      // TODO(shadi): Should YV12 be the default?

[scherkus (not reviewing), 2011/09/28 17:32:17]

nope I'd rather return INVALID

[shadi1, 2011/09/29 18:31:03]

Done.

+      NOTREACHED();

[scherkus (not reviewing), 2011/09/28 17:32:17]

could you add an error message to this:

NOTREACHED() << "Unsupported PixelFormat: " << pix_fmt;

[shadi1, 2011/09/29 18:31:03]

Done.

+  }
+  return VideoFrame::INVALID;
+}
+
 base::TimeDelta GetFrameDuration(AVStream* stream) {
   AVRational time_base = { stream->r_frame_rate.den, stream->r_frame_rate.num };
   return ConvertFromTimeBase(time_base, 1);
 }
 
 bool GetSeekTimeAfter(AVStream* stream, const base::TimeDelta& timestamp,
                       base::TimeDelta* seek_time) {
   DCHECK(stream);
   DCHECK(timestamp >= base::TimeDelta::FromSeconds(0));
   DCHECK(seek_time);
@@ skipping 107 matching lines @@
         avcodec_close(stream->codec);
       }
     }
   }
 
   // Then finally cleanup the format context.
   av_close_input_file(format_context);
 }
 
 }  // namespace media