Implement plugin side of sync EnumerateVideoCaptureDevices

ppapi/thunk/ppb_flash_thunk.cc

 // Copyright (c) 2012 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/c/pp_errors.h"
 #include "ppapi/c/private/ppb_flash.h"
 #include "ppapi/shared_impl/ppapi_globals.h"
 #include "ppapi/shared_impl/proxy_lock.h"
 #include "ppapi/shared_impl/var.h"
 #include "ppapi/thunk/enter.h"
 #include "ppapi/thunk/ppb_flash_api.h"
+#include "ppapi/thunk/ppb_flash_functions_api.h"
 #include "ppapi/thunk/ppb_instance_api.h"
 #include "ppapi/thunk/thunk.h"
 
+struct PP_ArrayOutput;

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

If the includes above doesn't have a definition of PP_ArrayOutput. I don't think
this forward declaration would work. Right?

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

Done.

+
 namespace ppapi {
 namespace thunk {
 
 namespace {
 
 void SetInstanceAlwaysOnTop(PP_Instance instance, PP_Bool on_top) {
   EnterInstance enter(instance);
   if (enter.failed())
     return;
   enter.functions()->GetFlashAPI()->SetInstanceAlwaysOnTop(instance, on_top);
@@ skipping 121 matching lines @@
 
 PP_Bool SetCrashData(PP_Instance instance,
                      PP_FlashCrashKey key,
                      PP_Var value) {
   EnterInstance enter(instance);
   if (enter.failed())
     return PP_FALSE;
   return enter.functions()->GetFlashAPI()->SetCrashData(instance, key, value);
 }
 
+int32_t EnumerateVideoDevicesSync(PP_Instance instance,
+                                  PP_Resource video_capture,
+                                  PP_ArrayOutput devices) {
+  EnterInstance enter(instance);
+  if (enter.succeeded()) {
+    PPB_Flash_Functions_API* api =
+        enter.functions()->GetFlashFunctionsAPI(instance);
+    if (api) {
+      return api->EnumerateVideoDevicesSync(instance, video_capture,
+                                            devices);
+    } else {
+      return PP_ERROR_NOINTERFACE;
+    }
+  }
+  return PP_ERROR_BADRESOURCE;

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

The comment of this new method doesn't mentioned any error values other than
PP_ERROR_FAILED/PP_OK.

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

Done.

+}
+
 const PPB_Flash_12_0 g_ppb_flash_12_0_thunk = {
   &SetInstanceAlwaysOnTop,
   &DrawGlyphs,
   &GetProxyForURL,
   &Navigate,
   &RunMessageLoop,
   &QuitMessageLoop,
   &GetLocalTimeZoneOffset,
   &GetCommandLineArgs,
   &PreLoadFontWin
@@ skipping 77 matching lines @@
   &PreLoadFontWin,
   &IsRectTopmost,
   &InvokePrinting,
   &UpdateActivity,
   &GetDeviceID,
   &GetSettingInt,
   &GetSetting,
   &SetCrashData
 };
 
+const PPB_Flash_12_6 g_ppb_flash_12_6_thunk = {
+  &SetInstanceAlwaysOnTop,
+  &DrawGlyphs,
+  &GetProxyForURL,
+  &Navigate,
+  &RunMessageLoop,
+  &QuitMessageLoop,
+  &GetLocalTimeZoneOffset,
+  &GetCommandLineArgs,
+  &PreLoadFontWin,
+  &IsRectTopmost,
+  &InvokePrinting,
+  &UpdateActivity,
+  &GetDeviceID,
+  &GetSettingInt,
+  &GetSetting,
+  &SetCrashData,
+  &EnumerateVideoDevicesSync
+};
+
 }  // namespace
 
 const PPB_Flash_12_0* GetPPB_Flash_12_0_Thunk() {
   return &g_ppb_flash_12_0_thunk;
 }
 
 const PPB_Flash_12_1* GetPPB_Flash_12_1_Thunk() {
   return &g_ppb_flash_12_1_thunk;
 }
 
 const PPB_Flash_12_2* GetPPB_Flash_12_2_Thunk() {
   return &g_ppb_flash_12_2_thunk;
 }
 
 const PPB_Flash_12_3* GetPPB_Flash_12_3_Thunk() {
   return &g_ppb_flash_12_3_thunk;
 }
 
 const PPB_Flash_12_4* GetPPB_Flash_12_4_Thunk() {
   return &g_ppb_flash_12_4_thunk;
 }
 
 const PPB_Flash_12_5* GetPPB_Flash_12_5_Thunk() {
   return &g_ppb_flash_12_5_thunk;
 }
 
+const PPB_Flash_12_6* GetPPB_Flash_12_6_Thunk() {
+  return &g_ppb_flash_12_6_thunk;
+}
+
 }  // namespace thunk
 }  // namespace ppapi