Support multiple video decoders and encoders

content/common/gpu/media/gpu_video_encode_accelerator.h

Index: content/common/gpu/media/gpu_video_encode_accelerator.h
diff --git a/content/common/gpu/media/gpu_video_encode_accelerator.h b/content/common/gpu/media/gpu_video_encode_accelerator.h
index ca60f6f1c8dcac639a5424a3bc66867e3acf4666..ed7e686cdd00ec20e2bb7304636a27e26f10e09d 100644
--- a/content/common/gpu/media/gpu_video_encode_accelerator.h
+++ b/content/common/gpu/media/gpu_video_encode_accelerator.h
@@ -60,14 +60,14 @@ class GpuVideoEncodeAccelerator
   // Static query for supported profiles.  This query calls the appropriate
   // platform-specific version.
   static std::vector<gpu::VideoEncodeAcceleratorSupportedProfile>
-  GetSupportedProfiles();
+      GetSupportedProfiles();
   static std::vector<gpu::VideoEncodeAcceleratorSupportedProfile>
-  ConvertMediaToGpuProfiles(const std::vector<
-      media::VideoEncodeAccelerator::SupportedProfile>& media_profiles);
+      ConvertMediaToGpuProfiles(const std::vector<
+          media::VideoEncodeAccelerator::SupportedProfile>& media_profiles);
 
  private:
-  // Create the appropriate platform-specific VEA.
-  static scoped_ptr<media::VideoEncodeAccelerator> CreateEncoder();
+  // Create and return the pointers of the appropriate platform-specific VEAs.

[Pawel Osciak, 2014/12/28 23:28:03]

s/of/to/

[henryhsu, 2014/12/29 09:43:27]

Done.

+  static std::vector<media::VideoEncodeAccelerator*> CreateEncoders();

[Pawel Osciak, 2014/12/28 23:28:03]

This makes me worried. Who owns the pointers? How do we ensure we free them
properly in all cases?

Why not a ScopedVector?

[henryhsu, 2014/12/29 09:43:26]

ScopedVector doesn't support DefaultDeleter. As discussed, I'll keep to use
scoped_ptr and duplicate CreateEncoder in profile and initialize function.

 
   // IPC handlers, proxying media::VideoEncodeAccelerator for the renderer
   // process.