Clean up video frame sizes, types in Video Decode API

media/video/video_decode_accelerator.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_VIDEO_VIDEO_DECODE_ACCELERATOR_H_
 #define MEDIA_VIDEO_VIDEO_DECODE_ACCELERATOR_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 164 matching lines @@
     VIDEODECODERERROR_UNINITIALIZED,
     VIDEODECODERERROR_UNSUPPORTED,
     VIDEODECODERERROR_INVALIDINPUT,
     VIDEODECODERERROR_MEMFAILURE,
     VIDEODECODERERROR_INSUFFICIENT_BUFFERS,
     VIDEODECODERERROR_INSUFFICIENT_RESOURCES,
     VIDEODECODERERROR_HARDWARE,
     VIDEODECODERERROR_UNEXPECTED_FLUSH,
   };
 
-  // Represents the type of data buffer to be used by the decoder.

[Ami GONE FROM CHROMIUM, 2011/07/18 18:33:00]

Do you want to DO the TODO from picture.h now, too?

[vrk (LEFT CHROMIUM), 2011/07/19 01:26:08]

Done.

-  enum MemoryType {
-    PICTUREBUFFER_MEMORYTYPE_NONE = 0,
-    PICTUREBUFFER_MEMORYTYPE_SYSTEM,
-    PICTUREBUFFER_MEMORYTYPE_GL_TEXTURE,
-  };
-
   // 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
   // to the plugin, through the PPP_VideoDecode_Dev interface the plugin
   // implements.
   class Client {
    public:
     virtual ~Client() {}
 
     // Callback to notify client that decoder has been initialized.
     virtual void NotifyInitializeDone() = 0;
 
-    // Callback to tell the information needed by the client to provide decoding
-    // buffer to the decoder.
+    // 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,
-        MemoryType type) = 0;
+        uint32 requested_num_of_buffers, const gfx::Size& dimensions) = 0;
 
     // Callback to dismiss picture buffer that was assigned earlier.
     virtual void DismissPictureBuffer(int32 picture_buffer_id) = 0;
 
     // Callback to deliver decoded pictures ready to be displayed.
     virtual void PictureReady(const Picture& picture) = 0;
 
     // Callback to notify that decoder has decoded end of stream marker and has
     // outputted all displayable pictures.
     virtual void NotifyEndOfStream() = 0;
@@ skipping 64 matching lines @@
   virtual void Destroy() = 0;
 
  protected:
   friend class base::RefCountedThreadSafe<VideoDecodeAccelerator>;
   virtual ~VideoDecodeAccelerator();
 };
 
 }  // namespace media
 
 #endif  // MEDIA_VIDEO_VIDEO_DECODE_ACCELERATOR_H_