Add VAVDA, the VAAPI Video Decode Accelerator for Intel CPUs.

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index c14e518e8962b985c4e09a998b3f16e8d5d4a1d7..daf7223829a9b0bd728bc0c2cf60f0dc45585c71 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -2152,11 +2152,13 @@ WebMediaPlayer* RenderViewImpl::createMediaPlayer(
     collection->AddAudioRenderer(audio_renderer);
   }
 
-  // Currently only cros/arm has any HW video decode support in
+  // Currently only cros has any HW video decode support in
   // GpuVideoDecodeAccelerator so we don't even try to use it on other
   // platforms.  This is a startup-time optimization.  When new VDA
   // implementations are added, relax the #if above.

[Ami GONE FROM CHROMIUM, 2012/03/21 13:16:24]

s/above/below/

[Pawel Osciak, 2012/04/05 10:37:20]

Done.

-#if defined(OS_CHROMEOS) && defined(ARCH_CPU_ARMEL)
+#if defined(OS_CHROMEOS) \
+  && (defined(ARCH_CPU_ARMEL) || defined(ARCH_CPU_X86_FAMILY))

[Ami GONE FROM CHROMIUM, 2012/03/21 13:16:24]

&& belongs on previous line

[Ami GONE FROM CHROMIUM, 2012/03/21 13:16:24]

are there any cros platforms *other* than x86 & arm?

[Pawel Osciak, 2012/03/21 18:40:35]

Not that I am aware of. Wanted to be extra safe. Drop it?

[Ami GONE FROM CHROMIUM, 2012/03/22 17:01:36]

Yes please.

[Pawel Osciak, 2012/04/05 10:37:20]

Done.

+  // Currently only cros has any HW video decode support in

[Ami GONE FROM CHROMIUM, 2012/03/21 13:16:24]

delete?

[Pawel Osciak, 2012/03/21 18:40:35]

Your call, was here before :)

[Ami GONE FROM CHROMIUM, 2012/03/22 17:01:36]

You duplicated the pre-existing line.
(note this is a copy of l.2155; this copy should be deleted, the other one
should stay)

[Pawel Osciak, 2012/04/05 10:37:20]

Done.

   // Note we don't actually use the result of this blind down-cast unless it's
   // valid (not NULL and of the right type).
   WebGraphicsContext3DCommandBufferImpl* context3d =

[Ami GONE FROM CHROMIUM, 2012/03/21 13:16:24]

Why bother going all the way to the GPU process to find out VAAPI is disabled? 
I'd check for the enable flag here, instead.
(that would also let you not propagate the enable flag to the GPU process,
reverting gpu_process_host.cc)

[Pawel Osciak, 2012/04/05 10:37:20]

Do we want to have one common flag then for acceleration on ARM and x86, as well
as on cros and on Linux?