Change the cpp wrappers of audio input/video capture to use CompletionCallbackWithOutput.

ppapi/cpp/dev/audio_input_dev.h

Index: ppapi/cpp/dev/audio_input_dev.h
diff --git a/ppapi/cpp/dev/audio_input_dev.h b/ppapi/cpp/dev/audio_input_dev.h
index 2bcccc348c80f7c9e9a0251d103d267bc0a8e036..13af97d9061721ab6e8897e049a93c639859d7ae 100644
--- a/ppapi/cpp/dev/audio_input_dev.h
+++ b/ppapi/cpp/dev/audio_input_dev.h
@@ -9,12 +9,12 @@
 
 #include "ppapi/c/dev/ppb_audio_input_dev.h"
 #include "ppapi/cpp/audio_config.h"
+#include "ppapi/cpp/completion_callback.h"
+#include "ppapi/cpp/dev/device_ref_dev.h"
 #include "ppapi/cpp/resource.h"
 
 namespace pp {
 
-class CompletionCallback;
-class DeviceRef_Dev;
 class InstanceHandle;
 
 class AudioInput_Dev : public Resource {
@@ -38,12 +38,8 @@ class AudioInput_Dev : public Resource {
   /// resource. Please use the 5-parameter Open() if you used this constructor.
   explicit AudioInput_Dev(const InstanceHandle& instance);
 
-  AudioInput_Dev(const AudioInput_Dev& other);
-
   virtual ~AudioInput_Dev();
 
-  AudioInput_Dev& operator=(const AudioInput_Dev& other);
-
   /// Static function for determining whether the browser supports the required
   /// AudioInput interface.
   ///
@@ -63,10 +59,14 @@ class AudioInput_Dev : public Resource {
   /// struct.
   const AudioConfig& config() const { return config_; }
 
-  /// |devices| must stay alive until either this AudioInput_Dev object goes
-  /// away or |callback| is run, if this method returns PP_OK_COMPLETIONPENDING.
-  int32_t EnumerateDevices(std::vector<DeviceRef_Dev>* devices,
-                           const CompletionCallback& callback);
+  // TODO(yzshen, brettw): If we forward declare DeviceRef_Dev (as opposed to
+  // including its .h file), it still compiles. However, it is not recognized as
+  // a derived class of Resource and does the wrong thing!
+  // This is due to the limitation of IsBaseOf in ppapi/cpp/output_traits.h. We
+  // need to figure out a way to overcome this problem.
+  int32_t EnumerateDevices(
+      const CompletionCallbackWithOutput<std::vector<DeviceRef_Dev> >&
+          callback);
 
   /// If |device_ref| is null (i.e., is_null() returns true), the default device
   /// will be used.
@@ -91,21 +91,24 @@ class AudioInput_Dev : public Resource {
   void Close();
 
  private:
-  struct EnumerateDevicesState;
-
-  void AbortEnumerateDevices();
-
-  // |user_data| is an EnumerateDevicesState object. It is this method's
-  // responsibility to delete it.
-  static void OnEnumerateDevicesComplete(void* user_data, int32_t result);
+  // Heap-allocated data passed to the CallbackConverter for backward

[brettw, 2012/04/03 21:23:03]

I wonder if these can be defined in the ResourceArray_Dev class and exported so
you only need to write this once. I think the implementations in audio input and
the video devices are the same, right? I think the resource arrays were only
ever used for the devices, so putting it there makes sense.

[yzshen1, 2012/04/04 00:36:34]

Thanks for pointing it out! It looks much better as you suggested.
(I don't know why I didn't realize it earlier. Maybe I was in cruise-control
mode at that time. :))

On 2012/04/03 21:23:03, brettw wrote:

+  // compatibility.
+  struct EnumerateCallbackData0_2 {
+    PP_Resource devices;
+    PP_ArrayOutput output;
+    PP_CompletionCallback original_callback;
+  };
+
+  // This is an adapter to convert the v0.2 EnumerateDevices oputput, which is a

[brettw, 2012/04/03 21:23:03]

oputput

[yzshen1, 2012/04/04 00:36:34]

Done.

+  // PPB_ResourceArray_Dev resource, to PP_ArrayOutput that the caller is
+  // expecting.
+  //
+  // This takes a heap-allocated EnumerateCallbackData0_2 struct passed as the
+  // user data and deletes it when the call completes.
+  static void CallbackConverter(void* user_data, int32_t result);
 
   AudioConfig config_;
 
-  // It is set in EnumerateDevices(), and cleared in AbortEnumerateDevices() or
-  // OnEnumerateDevicesComplete(). The object will be deleted by
-  // OnEnumerateDevicesComplete().
-  EnumerateDevicesState* enum_state_;
-
   // Used to store the arguments of Open() for the v0.2 interface.
   PPB_AudioInput_Callback audio_input_callback_;
   void* user_data_;