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

webkit/plugins/ppapi/ppb_video_decoder_impl.cc

 // 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.
 
 #include "webkit/plugins/ppapi/ppb_video_decoder_impl.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "base/message_loop.h"
@@ skipping 33 matching lines @@
   }
 }
 
 PPB_VideoDecoder_Impl::~PPB_VideoDecoder_Impl() {
 }
 
 PPB_VideoDecoder_API* PPB_VideoDecoder_Impl::AsPPB_VideoDecoder_API() {
   return this;
 }
 
+// Convert PP_VideoDecoder_Profile to media::VideoDecodeAccelerator::Profile.
+static media::VideoDecodeAccelerator::Profile PPToMediaProfile(
+    const PP_VideoDecoder_Profile pp_profile) {
+  // TODO(fischman,vrk): this assumes the enum values in the two Profile types
+  // match up exactly.  Add a COMPILE_ASSERT for this somewhere.
+  return static_cast<media::VideoDecodeAccelerator::Profile>(pp_profile);
+}
+
 // static
-PP_Resource PPB_VideoDecoder_Impl::Create(PP_Instance instance,
-                                          PP_Resource graphics_context,
-                                          const PP_VideoConfigElement* config) {
+PP_Resource PPB_VideoDecoder_Impl::Create(
+    PP_Instance instance,
+    PP_Resource graphics_context,
+    PP_VideoDecoder_Profile profile) {
   PluginDelegate::PlatformContext3D* platform_context = NULL;
   gpu::gles2::GLES2Implementation* gles2_impl = NULL;
   EnterResourceNoLock<PPB_Context3D_API> enter_context(graphics_context, false);
   if (enter_context.succeeded()) {
     PPB_Context3D_Impl* context3d_impl =
         static_cast<PPB_Context3D_Impl*>(enter_context.object());
     platform_context = context3d_impl->platform_context();
     gles2_impl = context3d_impl->gles2_impl();
   } else {
     EnterResourceNoLock<PPB_Graphics3D_API> enter_context(graphics_context,
                                                           true);
     if (enter_context.failed())
       return 0;
     PPB_Graphics3D_Impl* graphics3d_impl =
         static_cast<PPB_Graphics3D_Impl*>(enter_context.object());
     platform_context = graphics3d_impl->platform_context();
     gles2_impl = graphics3d_impl->gles2_impl();
   }
 
   scoped_refptr<PPB_VideoDecoder_Impl> decoder(
       new PPB_VideoDecoder_Impl(instance));
-  if (decoder->Init(graphics_context, platform_context, gles2_impl, config))
+  if (decoder->Init(graphics_context, platform_context, gles2_impl, profile))
     return decoder->GetReference();
   return 0;
 }
 
 bool PPB_VideoDecoder_Impl::Init(
     PP_Resource graphics_context,
     PluginDelegate::PlatformContext3D* context,
     gpu::gles2::GLES2Implementation* gles2_impl,
-    const PP_VideoConfigElement* config) {
+    PP_VideoDecoder_Profile profile) {
   InitCommon(graphics_context, gles2_impl);
 
   int command_buffer_route_id = context->GetCommandBufferRouteId();
   if (command_buffer_route_id == 0)
     return false;
 
-  std::vector<int32> copied;
-  if (!CopyConfigsToVector(config, &copied))
-    return false;
-
   PluginDelegate* plugin_delegate = ResourceHelper::GetPluginDelegate(this);
   if (!plugin_delegate)
     return false;
 
   platform_video_decoder_ = plugin_delegate->CreateVideoDecoder(
       this, command_buffer_route_id);
   if (!platform_video_decoder_)
     return false;
 
   FlushCommandBuffer();
-  return platform_video_decoder_->Initialize(copied);
+  return platform_video_decoder_->Initialize(PPToMediaProfile(profile));
 }
 
 int32_t PPB_VideoDecoder_Impl::Decode(
     const PP_VideoBitstreamBuffer_Dev* bitstream_buffer,
     PP_CompletionCallback callback) {
   if (!platform_video_decoder_)
     return PP_ERROR_BADRESOURCE;
 
   EnterResourceNoLock<PPB_Buffer_API> enter(bitstream_buffer->data, true);
   if (enter.failed())
@@ skipping 133 matching lines @@
 void PPB_VideoDecoder_Impl::NotifyFlushDone() {
   RunFlushCallback(PP_OK);
 }
 
 void PPB_VideoDecoder_Impl::NotifyInitializeDone() {
   NOTREACHED() << "PlatformVideoDecoder::Initialize() is synchronous!";
 }
 
 }  // namespace ppapi
 }  // namespace webkit