Issue 11644078: Add wrapper class to media for support of VP9 video, and add a command line flag to enable the supp…

Issue 11644078: Add wrapper class to media for support of VP9 video, and add a command line flag to enable the supp… (Closed)

Created:
8 years ago by Tom Finegan

Modified:
8 years ago

Reviewers:
fgalligan1, brettw, xhwang, scherkus (not reviewing)

CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, feature-media-reviews_chromium.org

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

Add wrapper class to media for support of VP9 video, and add a command line flag to enable the support. This initial version of the wrapper provides support for decoding VP9 video in WebM container files, and is disabled by default. New flag added: --enable-vp9-playback TBR=brettw,scherkus,xhwang BUG=166094 TEST=VP9 video in WebM containers plays back in <video> elements when --enable-vp9-playback is specified on the command line. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=174488

Patch Set 1 #

Patch Set 2 : Replace the DEPS roll of libvpx: It's required. #

Patch Set 3 : Roll libvpx_revision to pick up a bug fix. #

Created: 8 years ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+245 lines, -291 lines)			Patch
M	DEPS	View	1 2	1 chunk	+1 line, -1 line	0 comments	Download
M	chrome/app/generated_resources.grd	View		1 chunk	+6 lines, -0 lines	0 comments	Download
M	chrome/browser/about_flags.cc	View		1 chunk	+7 lines, -0 lines	0 comments	Download
M	content/browser/renderer_host/render_process_host_impl.cc	View		1 chunk	+1 line, -0 lines	0 comments	Download
M	media/DEPS	View		1 chunk	+1 line, -0 lines	0 comments	Download
M	media/base/media_switches.h	View		1 chunk	+2 lines, -0 lines	0 comments	Download
M	media/base/media_switches.cc	View		1 chunk	+3 lines, -0 lines	0 comments	Download
M	media/base/video_decoder_config.h	View		2 chunks	+6 lines, -2 lines	0 comments	Download
M	media/ffmpeg/ffmpeg_common.cc	View		4 chunks	+31 lines, -3 lines	0 comments	Download
A +	media/filters/vpx_video_decoder.h	View		4 chunks	+24 lines, -33 lines	0 comments	Download
A +	media/filters/vpx_video_decoder.cc	View		9 chunks	+140 lines, -252 lines	0 comments	Download
M	media/media.gyp	View		3 chunks	+16 lines, -0 lines	0 comments	Download
M	webkit/media/filter_helpers.cc	View		2 chunks	+7 lines, -0 lines	0 comments	Download

Messages

Total messages: 5 (0 generated)

Expand Messages | Collapse Messages

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tomfinegan@chromium.org/11644078/1

8 years ago (2012-12-21 19:54:38 UTC) #3

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tomfinegan@chromium.org/11644078/15001

8 years ago (2012-12-21 22:01:13 UTC) #4

commit-bot: I haz the power

8 years ago (2012-12-22 01:49:08 UTC) #5

Retried try job too often on win_rel for step(s) browser_tests

Expand Messages | Collapse Messages