Refactor video capture to new design

ppapi/thunk/ppb_video_capture_api.h

Index: ppapi/thunk/ppb_video_capture_api.h
diff --git a/ppapi/thunk/ppb_video_capture_api.h b/ppapi/thunk/ppb_video_capture_api.h
index 117b49566a8a77198b88625c13ef8ae8a5feff47..d3c80871c6696aad160b0a030e4468b6d9e526dc 100644
--- a/ppapi/thunk/ppb_video_capture_api.h
+++ b/ppapi/thunk/ppb_video_capture_api.h
@@ -9,6 +9,8 @@
 #include <vector>
 
 #include "base/memory/ref_counted.h"
+#include "ppapi/c/pp_array_output.h"
+#include "ppapi/c/pp_resource.h"
 #include "ppapi/c/dev/ppb_video_capture_dev.h"
 
 namespace ppapi {
@@ -33,14 +35,9 @@ class PPB_VideoCapture_API {
   virtual int32_t StopCapture() = 0;
   virtual void Close() = 0;
 
-  // For backward compatibility.
-  virtual int32_t StartCapture0_1(
-      const PP_VideoCaptureDeviceInfo_Dev& requested_info,
-      uint32_t buffer_count) = 0;
-
-  // This function is not exposed through the C API, but returns the internal
-  // data for easy proxying.
-  virtual const std::vector<DeviceRefData>& GetDeviceRefData() const = 0;
+  // This function is not exposed through the C API.  It is only used by flash
+  // to make synchronous device enumeration.
+  virtual int32_t EnumerateDevicesSync(PP_ArrayOutput* devices) = 0;

[yzshen1, 2012/11/14 06:54:16]

Please use const PP_ArrayOutput&. PP_ArrayOutput is not some output buffer
itself. So I don't think the 'output parameter should be a pointer' style guide
should be followed here.

[victorhsieh, 2012/11/14 08:22:27]

Make sense.  Done.

[yzshen1, 2012/11/14 17:55:36]

I mean const &, please
On 2012/11/14 08:22:27, Victor Hsieh wrote:

[victorhsieh, 2012/11/14 23:23:08]

Done.

 };
 
 }  // namespace thunk