Replace the use of an int32* with a proper struct for decoder configuration.

ppapi/cpp/dev/video_decoder_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_CPP_DEV_VIDEO_DECODER_DEV_H_
 #define PPAPI_CPP_DEV_VIDEO_DECODER_DEV_H_
 
 #include <vector>
 
 #include "ppapi/c/dev/pp_video_dev.h"
 #include "ppapi/cpp/completion_callback.h"
 #include "ppapi/cpp/dev/buffer_dev.h"
 #include "ppapi/cpp/resource.h"
 
 namespace pp {
 
 class Context3D_Dev;
 class Graphics3D_Dev;
 class Instance;
 
 // C++ wrapper for the Pepper Video Decoder interface. For more detailed
 // documentation refer to the C interfaces.
 //
 // C++ version of the PPB_VideoDecoder_Dev interface.
 class VideoDecoder_Dev : public Resource {
  public:
   // See PPB_VideoDecoder_Dev::Create.
   VideoDecoder_Dev(const Instance& instance,

[brettw, 2011/08/31 15:45:38]

While you're here, can you convert the Instance& to Instance* in these two
constructors? We've been using pointers for instance parameters. Since calling
code is usually in the instance object, it's nicer to pass "this" instead of
"*this".

[Ami GONE FROM CHROMIUM, 2011/08/31 18:28:34]

Done.

                    const Context3D_Dev& context,
-                   const PP_VideoConfigElement* config);
+                   const PP_VideoDecoderConfig_Dev& config);
   VideoDecoder_Dev(const Instance& instance,
                    const Graphics3D_Dev& context,
-                   const PP_VideoConfigElement* config);
+                   const PP_VideoDecoderConfig_Dev& config);
 
   explicit VideoDecoder_Dev(PP_Resource resource);
   virtual ~VideoDecoder_Dev();
 
   // PPB_VideoDecoder_Dev implementation.
   void AssignPictureBuffers(const std::vector<PP_PictureBuffer_Dev>& buffers);
   int32_t Decode(const PP_VideoBitstreamBuffer_Dev& bitstream_buffer,
                  CompletionCallback callback);
   void ReusePictureBuffer(int32_t picture_buffer_id);
   int32_t Flush(CompletionCallback callback);
   int32_t Reset(CompletionCallback callback);
 
  private:
   // Disallow copy-construction to ensure Destroy() is called exactly once.
   VideoDecoder_Dev(const VideoDecoder_Dev&);
 };
 
 }  // namespace pp
 
 #endif  // PPAPI_CPP_DEV_VIDEO_DECODER_DEV_H_