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

[platform/framework/web/chromium-efl.git] / content / renderer / pepper / pepper_video_capture_host.cc

diff --git a/content/renderer/pepper/pepper_video_capture_host.cc b/content/renderer/pepper/pepper_video_capture_host.cc
index b4c8c60..6243f0e 100644 (file)
--- a/content/renderer/pepper/pepper_video_capture_host.cc
+++ b/content/renderer/pepper/pepper_video_capture_host.cc
@@ -80,6 +80,7 @@ int32_t PepperVideoCaptureHost::OnResourceMessageReceived(
     PPAPI_DISPATCH_HOST_RESOURCE_CALL_0(PpapiHostMsg_VideoCapture_Close,
                                         OnClose)
   PPAPI_END_MESSAGE_MAP()
+  LOG(ERROR) << "Resource message unresolved";
   return PP_ERROR_FAILED;
 }
 
@@ -270,14 +271,18 @@ int32_t PepperVideoCaptureHost::OnOpen(
     const std::string& device_id,
     const PP_VideoCaptureDeviceInfo_Dev& requested_info,
     uint32_t buffer_count) {
-  if (platform_video_capture_.get())
+  if (platform_video_capture_.get()) {
+    LOG(ERROR) << "Platform video capture is not NULL";
     return PP_ERROR_FAILED;
+  }
 
   SetRequestedInfo(requested_info, buffer_count);
 
   GURL document_url = renderer_ppapi_host_->GetDocumentURL(pp_instance());
-  if (!document_url.is_valid())
+  if (!document_url.is_valid()) {
+    LOG(ERROR) << "Invalid document URL";
     return PP_ERROR_FAILED;
+  }
 
   platform_video_capture_ = std::make_unique<PepperPlatformVideoCapture>(
       renderer_ppapi_host_->GetRenderFrameForInstance(pp_instance())
@@ -292,8 +297,11 @@ int32_t PepperVideoCaptureHost::OnOpen(
 int32_t PepperVideoCaptureHost::OnStartCapture(
     ppapi::host::HostMessageContext* context) {
   if (!SetStatus(PP_VIDEO_CAPTURE_STATUS_STARTING, false) ||
-      !platform_video_capture_.get())
+      !platform_video_capture_.get()) {
+    LOG(ERROR) << "Cannot set PP_VIDEO_CAPTURE_STATUS_STARTING status or "
+               << "platform video capture is NULL";
     return PP_ERROR_FAILED;
+  }
 
   DCHECK(buffers_.empty());
 
@@ -306,8 +314,12 @@ int32_t PepperVideoCaptureHost::OnStartCapture(
 int32_t PepperVideoCaptureHost::OnReuseBuffer(
     ppapi::host::HostMessageContext* context,
     uint32_t buffer) {
-  if (buffer >= buffers_.size() || !buffers_[buffer].in_use)
+  if (buffer >= buffers_.size() || !buffers_[buffer].in_use) {
+    LOG(ERROR) << "Wrong buffer ID: " << buffer << "Should be less than "
+               << buffers_.size() << " and should be in use, buffer.in_use ="
+               << buffers_[buffer].in_use;
     return PP_ERROR_BADARGUMENT;
+  }
   buffers_[buffer].in_use = false;
   return PP_OK;
 }
@@ -323,8 +335,10 @@ int32_t PepperVideoCaptureHost::OnClose(
 }
 
 int32_t PepperVideoCaptureHost::StopCapture() {
-  if (!SetStatus(PP_VIDEO_CAPTURE_STATUS_STOPPING, false))
+  if (!SetStatus(PP_VIDEO_CAPTURE_STATUS_STOPPING, false)) {
+    LOG(ERROR) << "Cannot set PP_VIDEO_CAPTURE_STATUS_STOPPING status";
     return PP_ERROR_FAILED;
+  }
 
   DCHECK(platform_video_capture_.get());