Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(64)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: media/ffmpeg/ffmpeg_common.cc

Issue 8052002: Fix support for yuv_422 pixel format. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Fixed lint errors. Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« media/ffmpeg/ffmpeg_common.h ('K') | « media/ffmpeg/ffmpeg_common.h ('k') | media/filters/ffmpeg_video_decoder.cc » ('j') | media/video/ffmpeg_video_decode_engine.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/ffmpeg/ffmpeg_common.cc
diff --git a/media/ffmpeg/ffmpeg_common.cc b/media/ffmpeg/ffmpeg_common.cc
index 72323c7e3e68dface57fae46de5806dae3974eb5..1a7da3eac4fdf4b4238b1c0681522e305b4b0ef8 100644
--- a/media/ffmpeg/ffmpeg_common.cc
+++ b/media/ffmpeg/ffmpeg_common.cc
@@ -215,6 +215,19 @@ ChannelLayout ChannelLayoutToChromeChannelLayout(int64_t layout,
}
}
+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: NOTREACHE
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);
« media/ffmpeg/ffmpeg_common.h ('K') | « media/ffmpeg/ffmpeg_common.h ('k') | media/filters/ffmpeg_video_decoder.cc » ('j') | media/video/ffmpeg_video_decode_engine.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698