media/filters/ffmpeg_video_decoder.cc - Issue 88403004: Add plumbing for video pixel formats with JPEG color range.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: media/filters/ffmpeg_video_decoder.cc

Issue 88403004: Add plumbing for video pixel formats with JPEG color range. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Add YV12J to a DCHECK Created 7 years 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

Index: media/filters/ffmpeg_video_decoder.cc

diff --git a/media/filters/ffmpeg_video_decoder.cc b/media/filters/ffmpeg_video_decoder.cc

index eccfee281bfae314dd4cad9585b74012868f042c..75289cef27bf26bb88d102931ca155e9ca2d6144 100644

--- a/media/filters/ffmpeg_video_decoder.cc

+++ b/media/filters/ffmpeg_video_decoder.cc

@@ -71,7 +71,8 @@ int FFmpegVideoDecoder::GetVideoBuffer(AVCodecContext* codec_context,

VideoFrame::Format format = PixelFormatToVideoFormat(codec_context->pix_fmt);

if (format == VideoFrame::UNKNOWN)

return AVERROR(EINVAL);

- DCHECK(format == VideoFrame::YV12 || format == VideoFrame::YV16);

+ DCHECK(format == VideoFrame::YV12 || format == VideoFrame::YV16 ||

+ format == VideoFrame::YV12J);

gfx::Size size(codec_context->width, codec_context->height);

int ret;

« no previous file with comments | « media/ffmpeg/ffmpeg_common.cc ('k') | media/filters/skcanvas_video_renderer.cc » ('j') | no next file with comments »