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 152 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.
-  enum MemoryType {
-    PICTUREBUFFER_MEMORYTYPE_NONE = 0,
-    PICTUREBUFFER_MEMORYTYPE_SYSTEM,
-    PICTUREBUFFER_MEMORYTYPE_GL_TEXTURE,
-  };

[Ville-Mikko Rautio, 2011/05/17 08:58:49]

See the comment in pp_video_dev.h

-
   // 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 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 75 matching lines @@
   // been finished. After abort all buffers can be considered dismissed, even
   // when there has not been callbacks to dismiss them.
   //
   // Returns true when command successfully accepted. Otherwise false.
   virtual bool Abort() = 0;
 };
 
 }  // namespace media
 
 #endif  // MEDIA_VIDEO_VIDEO_DECODE_ACCELERATOR_H_