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

ppapi/shared_impl/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 "ppapi/shared_impl/video_decoder_impl.h"
 
 #include "base/logging.h"
 #include "gpu/command_buffer/client/gles2_implementation.h"
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/proxy/plugin_resource_tracker.h"
@@ skipping 71 matching lines @@
   PP_CompletionCallback cc = it->second;
   bitstream_buffer_callbacks_.erase(it);
   PP_RunCompletionCallback(&cc, PP_OK);
 }
 
 void VideoDecoderImpl::FlushCommandBuffer() {
   if (gles2_impl_)
     gles2_impl_->Flush();
 }
 
-bool VideoDecoderImpl::CopyConfigsToVector(
-    const PP_VideoConfigElement* configs_to_copy,
-    std::vector<PP_VideoConfigElement>* out_configs) {
-  // TODO(fischman/vrk): This is still broken. We need to get rid of the silly
-  // PP_VideoConfigElement vector in favor of a struct (see TODO in
-  // ppb_video_decoder_dev.h).
-  const PP_VideoConfigElement* current = configs_to_copy;
-  while (current && *current != PP_VIDEOATTR_DICTIONARY_TERMINATOR) {
-    out_configs->push_back(*current);
-    out_configs->push_back(*(current + 1));
-    current += 2;
-  }
-  return true;
-}
-
 }  // namespace ppapi