Implement plugin side of sync EnumerateVideoCaptureDevices

ppapi/api/private/ppb_flash.idl

Index: ppapi/api/private/ppb_flash.idl
diff --git a/ppapi/api/private/ppb_flash.idl b/ppapi/api/private/ppb_flash.idl
index c5423d3cbe6763628ac92d2aee81c74860bf79cb..6b63e56f774821260899dc32e065ce387e1c361b 100644
--- a/ppapi/api/private/ppb_flash.idl
+++ b/ppapi/api/private/ppb_flash.idl
@@ -13,7 +13,8 @@ label Chrome {
   M20_0 = 12.2,
   M20_1 = 12.3,
   M21 = 12.4,
-  M22 = 12.5
+  M22 = 12.5,
+  M24 = 12.6
 };
 
 [assert_size(4)]
@@ -22,12 +23,12 @@ enum PP_FlashLSORestrictions {
    * No restrictions on Flash LSOs.
    */
   PP_FLASHLSORESTRICTIONS_NONE = 1,
-  
+
   /**
    * Don't allow access to Flash LSOs.
    */
   PP_FLASHLSORESTRICTIONS_BLOCK = 2,
-  
+
   /**
    * Store Flash LSOs in memory only.
    */
@@ -81,7 +82,7 @@ enum PP_FlashSetting {
    * Specifies the number of CPU cores that are present on the system.
    */
   PP_FLASHSETTING_NUMCORES = 5,
-  
+
   /**
    * Specifies restrictions on how flash should handle LSOs. The result is an
    * int from <code>PP_FlashLSORestrictions</code>.
@@ -247,4 +248,23 @@ interface PPB_Flash {
   PP_Bool SetCrashData([in] PP_Instance instance,
                        [in] PP_FlashCrashKey key,
                        [in] PP_Var value);
+
+  /**
+   * Enumerates video capture devices. |video_capture| is a valid
+   * PPB_VideoCapture_Dev resource. Once the operation has completed
+   * successfully, |devices| will be set up with an array of
+   * |PPB_DeviceRef_Dev| resources.
+   *
+   * |PP_OK| is returned on success and |PP_ERROR| on failure. This method
+   * ignores. The ref count of the returned |devices| has already been

[yzshen1, 2012/10/05 18:16:46]

"This method ignores"?

[raymes, 2012/10/08 17:08:50]

Done.

+   * increased by 1 for the caller.
+   *
+   * NOTE: This method is a synchronous version of |EnumerateDevices| in
+   * |PP_VideoCapture_Dev|.

[yzshen1, 2012/10/05 18:16:46]

PP*B*

[raymes, 2012/10/08 17:08:50]

Done.

+   */
+  [version=12.6]
+  int32_t EnumerateVideoDevicesSync(

[yzshen1, 2012/10/05 18:16:46]

Please change the name to EnumerateVideo*Capture*Devices. I personally don't
think 'sync' is necessary, since it is quite obvious.

[raymes, 2012/10/08 17:08:50]

Done.

+      [in] PP_Instance instance,

[yzshen1, 2012/10/05 18:16:46]

As we discussed, I am not sure |instance| is useful here.

[raymes, 2012/10/08 17:08:50]

I'll check with Brett

+      [in] PP_Resource video_capture,
+      [in] PP_ArrayOutput devices);
 };