<video> decode in hardware!

<video> decode in hardware! 

This uses the GpuVideoDecodeAccelerator machinery (already written to enable 
ppapi to take advantage of OpenMAX HW where available) to decode <video> data. 
This increases idle CPU from 20% to 45% on one particularly large (internal) 
test video (red0.mp4), on an ARM crosbook. 

HW decode is done on a best-effort basis; if the GPU code doesn't know how to 
deal with a codec/profile we still fall back to ffmpeg for decode. Because the 
vast majority of chrome installs will be on HW with no video decode support 
(yet) we only attempt HW video decode on platforms we know have a shot at it. 

This is a copy of http://codereview.chromium.org/8686010/ PS#13 to work around
rietveld losing the "status" for some files in the patch.

BUG=104579
TEST=manual testing w/ video test matrix, trybots.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114183

[Ami GONE FROM CHROMIUM, 2011-12-13 00:45:09]

piman/scherkus: please migrate your LGTMs from
http://codereview.chromium.org/8686010/ to here (in hopes of using the CQ).
PS#1 here is identical to PS#13 there.

[scherkus (not reviewing), 2011-12-13 00:46:37]

rubber stamp lgtm

[piman, 2011-12-13 02:17:24]

lgtm

http://codereview.chromium.org/8922010/diff/1/content/renderer/renderer_gpu_v...
File content/renderer/renderer_gpu_video_decoder_factories.cc (right):

http://codereview.chromium.org/8922010/diff/1/content/renderer/renderer_gpu_v...
content/renderer/renderer_gpu_video_decoder_factories.cc:16: :
gpu_channel_host_(gpu_channel_host), context_(context) {
nit: one initializer per line.

[Ami GONE FROM CHROMIUM, 2011-12-13 02:21:01]

Thanks for the review guys.

http://codereview.chromium.org/8922010/diff/1/content/renderer/renderer_gpu_v...
File content/renderer/renderer_gpu_video_decoder_factories.cc (right):

http://codereview.chromium.org/8922010/diff/1/content/renderer/renderer_gpu_v...
content/renderer/renderer_gpu_video_decoder_factories.cc:16: :
gpu_channel_host_(gpu_channel_host), context_(context) {
On 2011/12/13 02:17:24, piman wrote:
> nit: one initializer per line.

Done.

[commit-bot: I haz the power, 2011-12-13 02:21:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/8922010/11001

[commit-bot: I haz the power, 2011-12-13 02:21:41]

Can't apply patch for file content/renderer/media/audio_renderer_impl.cc.
While running patch -p1 --forward --force;
patching file content/renderer/media/audio_renderer_impl.cc
Hunk #1 FAILED at 202.
1 out of 1 hunk FAILED -- saving rejects to file
content/renderer/media/audio_renderer_impl.cc.rej

[commit-bot: I haz the power, 2011-12-13 02:25:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/8922010/12002

[commit-bot: I haz the power, 2011-12-13 03:21:04]

Try job failure for 8922010-12002 (retry) on linux_rel for step "ui_tests"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2011-12-13 03:46:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/8922010/12002

[commit-bot: I haz the power, 2011-12-13 05:12:24]

Change committed as 114183