Use GPU blacklist to control accelerated video decode.

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
===================================================================
--- content/renderer/render_view_impl.cc	(revision 151736)
+++ content/renderer/render_view_impl.cc	(working copy)
@@ -2396,12 +2396,7 @@
     collection->AddAudioRenderer(audio_renderer);
   }
 
-  // Accelerated video decode is not enabled by default on Linux.
-  // crbug.com/137247
-  bool use_accelerated_video_decode = false;
-#if defined(OS_CHROMEOS)
-  use_accelerated_video_decode = true;
-#endif
+  bool use_accelerated_video_decode = true;
   use_accelerated_video_decode &= !CommandLine::ForCurrentProcess()->HasSwitch(

[Ami GONE FROM CHROMIUM, 2012/08/17 00:19:17]

I think this change is wrong:
- The &= is unnecessary given the default-true assignment above.
- This turns on by default the feature everywhere, which is not correct.

[Zhenyao Mo, 2012/08/17 00:25:09]

My change says accelerated_video_decode is on unless the switch to disable it is
passed - either by directly passing the switch at command-line, or inserted to
renderer process by blacklist.

       switches::kDisableAcceleratedVideoDecode);
   WebGraphicsContext3DCommandBufferImpl* context3d =