Plumb texture target to VideoDecodeAccelerator::Client

media/filters/gpu_video_decoder.cc

Index: media/filters/gpu_video_decoder.cc
diff --git a/media/filters/gpu_video_decoder.cc b/media/filters/gpu_video_decoder.cc
index 44005f3fdb07ba26b12f7bfe00f58de0e8d05877..1db78c36f4217869933e064216cd33cf2d277cc5 100644
--- a/media/filters/gpu_video_decoder.cc
+++ b/media/filters/gpu_video_decoder.cc
@@ -316,14 +316,20 @@ void GpuVideoDecoder::NotifyInitializeDone() {
   NOTREACHED() << "GpuVideoDecodeAcceleratorHost::Initialize is synchronous!";
 }
 
-void GpuVideoDecoder::ProvidePictureBuffers(uint32 count,
-                                            const gfx::Size& size) {
+void GpuVideoDecoder::ProvidePictureBuffers(
+    uint32 count,
+    const gfx::Size& size,
+    media::VideoDecodeAccelerator::TextureTarget texture_target) {
   if (!gvd_loop_proxy_->BelongsToCurrentThread()) {
     gvd_loop_proxy_->PostTask(FROM_HERE, base::Bind(
-        &GpuVideoDecoder::ProvidePictureBuffers, this, count, size));
+        &GpuVideoDecoder::ProvidePictureBuffers, this, count, size,
+        texture_target));
     return;
   }
 
+  // TODO(sail) : Add support for ARB texture targets.

[Ami GONE FROM CHROMIUM, 2012/05/23 23:42:33]

I think it's important to do these TODOs before the CL lands as a sanity check. 
For example, see 5 lines down.

[sail, 2012/05/29 18:58:09]

Done.

+  CHECK_EQ(media::VideoDecodeAccelerator::TEXTURE_TARGET_2D, texture_target);
+
   std::vector<uint32> texture_ids;
   if (!factories_->CreateTextures(
       count, size, &texture_ids, &decoder_texture_target_)) {

[Ami GONE FROM CHROMIUM, 2012/05/23 23:42:33]

Here CreateTextures() expects to be able to proscribe which target the texture
was created against.
In fact, it doesn't need freedom, but the resulting textures need to be tagged
w/ their targets.

So I think you should replace the TODO at l330 above with passing the
texture_target arg to PPB on to Factories::CreateTextures() and use the
passed-in texture value there (and change CreateTextures() to accept the target
as an input instead of output param).

[sail, 2012/05/29 18:58:09]

Done.