Clean up video frame sizes, types in Video Decode API

ppapi/c/dev/pp_video_dev.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 PPAPI_C_DEV_PP_VIDEO_DEV_H_
 #define PPAPI_C_DEV_PP_VIDEO_DEV_H_
 
 #include "ppapi/c/dev/ppb_opengles_dev.h"
 #include "ppapi/c/pp_bool.h"
 #include "ppapi/c/pp_instance.h"
@@ skipping 147 matching lines @@
 
 // Enumeration for various color formats.
 enum PP_ColorFormat_Dev {
   // Value represents 32-bit RGBA format where each component is 8-bit in order
   // R-G-B-A. Regardless of endianness of the architecture color components are
   // stored in this order in the memory.
   PP_VIDEOATTR_COLORFORMAT_RGBA = 0
 };
 PP_COMPILE_ASSERT_ENUM_SIZE_IN_BYTES(PP_ColorFormat_Dev, 4);
 
-// Enumeration to determine which type of memory for buffer is used.
-enum PP_PictureBufferType_Dev {

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

Technically requires bumping the PPB_ version number, too?

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

Makes sense; done.

-  PP_PICTUREBUFFERTYPE_NONE = 0,
-  // System memory a.k.a. RAM.
-  PP_PICTUREBUFFERTYPE_SYSTEM = 1,
-  // GLES texture allocated using OpenGL ES APIs.
-  PP_PICTUREBUFFERTYPE_GLESTEXTURE = 1 << 1
-};
-PP_COMPILE_ASSERT_ENUM_SIZE_IN_BYTES(PP_PictureBufferType_Dev, 4);
-
 // The data structure for video bitstream buffer.
 struct PP_VideoBitstreamBuffer_Dev {
   // Client-specified identifier for the bitstream buffer.
   int32_t id;
 
   // Buffer to hold the bitstream data. Should be allocated using the PPB_Buffer
   // interface for consistent interprocess behaviour.
   PP_Resource data;
 
   // Size of the bitstream contained in buffer (in bytes).
@@ skipping 24 matching lines @@
 struct PP_SysmemBuffer_Dev {
   // Resource representing system memory from shared memory address space.
   // Use PPB_Buffer_Dev interface to handle this resource.
   PP_Resource data;
 
   // Information about the buffer.
   struct PP_BufferInfo_Dev info;
 };
 
 // Structure to describe a decoded output frame.
-// The decoded pixels will always begin flush with the upper left-hand corner
-// of the buffer (0, 0).
 struct PP_Picture_Dev {
   // ID of the picture buffer where the picture is stored.
   int32_t picture_buffer_id;
 
   // ID of the bitstream from which this data was decoded.
   int32_t bitstream_buffer_id;
-
-  // Visible size of the picture.
-  // This describes the dimensions of the picture that is intended to be
-  // displayed from the decoded output.
-  struct PP_Size visible_size;
-
-  // Decoded size of the picture.
-  // This describes the dimensions of the decoded output. This may be slightly
-  // larger than the visible size because the stride is sometimes larger than
-  // the width of the output. The plugin should handle rendering the frame
-  // appropriately with respect to the sizes.
-  struct PP_Size decoded_size;
 };
 
 // Enumeration for error events that may be reported through
 // PP_VideoDecodeErrorHandler_Func_Dev callback function to the plugin. Default
 // error handling functionality expected from the plugin is to Flush and Destroy
 // the decoder.
 enum PP_VideoDecodeError_Dev {
   PP_VIDEODECODEERROR_NONE = 0,
   // Decoder has not been initialized and configured properly.
   PP_VIDEODECODEERROR_UNINITIALIZED,
   // Decoder does not support feature of configuration or bitstream.
   PP_VIDEODECODEERROR_UNSUPPORTED,
   // Decoder did not get valid input.
   PP_VIDEODECODERERROR_INVALIDINPUT,
   // Failure in memory allocation or mapping.
   PP_VIDEODECODERERROR_MEMFAILURE,
   // Decoder was given bitstream that would result in output pictures but it
   // has not been provided buffers to do all this.
   PP_VIDEODECODEERROR_INSUFFICIENT_BUFFERS,
   // Decoder cannot continue operation due to insufficient resources for the
   // current configuration.
   PP_VIDEODECODEERROR_INSUFFICIENTRESOURCES,
   // Decoder hardware has reported hardware error.
   PP_VIDEODECODEERROR_HARDWARE
 };
 PP_COMPILE_ASSERT_ENUM_SIZE_IN_BYTES(PP_VideoDecodeError_Dev, 4);
 
 #endif  /* PPAPI_C_DEV_PP_VIDEO_DEV_H_ */