[M108 Aura Migration][NaCl][PPFwk] Add error logs + SVACE/DLOG/Static analysis fix

[platform/framework/web/chromium-efl.git] / ppapi / proxy / audio_input_resource.cc

diff --git a/ppapi/proxy/audio_input_resource.cc b/ppapi/proxy/audio_input_resource.cc
index ef30ba7..2f38d0b 100644 (file)
--- a/ppapi/proxy/audio_input_resource.cc
+++ b/ppapi/proxy/audio_input_resource.cc
@@ -319,22 +319,33 @@ int32_t AudioInputResource::CommonOpen(
   if (device_ref != 0) {
     thunk::EnterResourceNoLock<thunk::PPB_DeviceRef_API> enter_device_ref(
         device_ref, true);
-    if (enter_device_ref.failed())
+    if (enter_device_ref.failed()) {
+      LOG(ERROR) << "Enter DeviceRef failed";
       return PP_ERROR_BADRESOURCE;
+    }
     device_id = enter_device_ref.object()->GetDeviceRefData().id;
   }
 
-  if (TrackedCallback::IsPending(open_callback_))
+  if (TrackedCallback::IsPending(open_callback_)) {
+    LOG(ERROR) << "Open is pending";
     return PP_ERROR_INPROGRESS;
-  if (open_state_ != BEFORE_OPEN)
+  }
+  if (open_state_ != BEFORE_OPEN) {
+    LOG(ERROR) << "Audio input already opened";
     return PP_ERROR_FAILED;
+  }
 
-  if (!audio_input_callback_0_3 && !audio_input_callback)
+  if (!audio_input_callback_0_3 && !audio_input_callback) {
+    LOG(ERROR) << "Cannot resolve audio_input_callback_0_3 or "
+                  "audio_input_callback";
     return PP_ERROR_BADARGUMENT;
+  }
   thunk::EnterResourceNoLock<thunk::PPB_AudioConfig_API> enter_config(config,
                                                                       true);
-  if (enter_config.failed())
+  if (enter_config.failed()) {
+    LOG(ERROR) << "Enter AudioConfig failed";
     return PP_ERROR_BADARGUMENT;
+  }
 
   config_ = config;
   audio_input_callback_0_3_ = audio_input_callback_0_3;