Plumb texture target to VideoDecodeAccelerator::Client

media/video/video_decode_accelerator.h

Index: media/video/video_decode_accelerator.h
diff --git a/media/video/video_decode_accelerator.h b/media/video/video_decode_accelerator.h
index 8128094f9cb9defecda86a6818415910d9117bab..ceb3617a4d35114356ed7a00be87eed0bb8bca2d 100644
--- a/media/video/video_decode_accelerator.h
+++ b/media/video/video_decode_accelerator.h
@@ -39,6 +39,13 @@ class MEDIA_EXPORT VideoDecodeAccelerator
     PLATFORM_FAILURE,
   };
 
+  // Enumeration of texture targets that the decoder might request.
+  // Note: Keep these in sync with PP_VideoDecoder_TextureTarget_Dev.
+  enum TextureTarget {

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

I don't think this (and its PP_VD_TT_D equivalent) are worth the extra
text/casting vs. just using a uint32 (like texture id).

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

Done.

+    TEXTURE_TARGET_2D = 0,
+    TEXTURE_TARGET_ARB
+  };
+
   // Interface for collaborating with picture interface to provide memory for
   // output picture and blitting them.
   // This interface is extended by the various layers that relay messages back
@@ -52,8 +59,9 @@ class MEDIA_EXPORT VideoDecodeAccelerator
     virtual void NotifyInitializeDone() = 0;
 
     // Callback to tell client how many and what size of buffers to provide.
-    virtual void ProvidePictureBuffers(
-        uint32 requested_num_of_buffers, const gfx::Size& dimensions) = 0;
+    virtual void ProvidePictureBuffers(uint32 requested_num_of_buffers,
+                                       const gfx::Size& dimensions,
+                                       TextureTarget texture_target) = 0;
 
     // Callback to dismiss picture buffer that was assigned earlier.
     virtual void DismissPictureBuffer(int32 picture_buffer_id) = 0;