Commit ff3933f6 authored by Jasmine Chen's avatar Jasmine Chen Committed by Commit Bot

Fix sync_wait() usage in VCD request manager

This CL fixes sync_wait() usage in VCD request manager. sync_wait()
returns 0 when the wait is successful. !sync_wait() is true when
the wait operation is successful.

BUG=None
TEST=Recompiled and redeployed Chrome.

Change-Id: I6737b039e0c2c7fcfd5a685784a5bc026b3d18ba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1638209Reviewed-by: default avatarWei Lee <wtlee@chromium.org>
Reviewed-by: default avatarRicky Liang <jcliang@chromium.org>
Auto-Submit: Jasmine Chen <lnishan@google.com>
Commit-Queue: Jasmine Chen <lnishan@google.com>
Cr-Commit-Position: refs/heads/master@{#666578}
parent 72822beb
...@@ -5,6 +5,7 @@ ...@@ -5,6 +5,7 @@
#include "media/capture/video/chromeos/request_manager.h" #include "media/capture/video/chromeos/request_manager.h"
#include <sync/sync.h> #include <sync/sync.h>
#include <initializer_list> #include <initializer_list>
#include <map> #include <map>
#include <set> #include <set>
...@@ -647,7 +648,7 @@ void RequestManager::SubmitCaptureResult( ...@@ -647,7 +648,7 @@ void RequestManager::SubmitCaptureResult(
FROM_HERE, "Failed to unwrap release fence fd"); FROM_HERE, "Failed to unwrap release fence fd");
return; return;
} }
if (!sync_wait(fence.GetFD().get(), kSyncWaitTimeoutMs)) { if (sync_wait(fence.GetFD().get(), kSyncWaitTimeoutMs)) {
device_context_->SetErrorState( device_context_->SetErrorState(
media::VideoCaptureError:: media::VideoCaptureError::
kCrosHalV3BufferManagerSyncWaitOnReleaseFenceTimedOut, kCrosHalV3BufferManagerSyncWaitOnReleaseFenceTimedOut,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment