Index: webrtc/video/video_capture_input_unittest.cc |
diff --git a/webrtc/video/video_capture_input_unittest.cc b/webrtc/video/video_capture_input_unittest.cc |
index 34bb7d609fc2dbec0603758a3db265d42d738117..ed9c73a28816df7db65bbf3f205c8ef3e33c03a5 100644 |
--- a/webrtc/video/video_capture_input_unittest.cc |
+++ b/webrtc/video/video_capture_input_unittest.cc |
@@ -13,11 +13,11 @@ |
#include "testing/gmock/include/gmock/gmock.h" |
#include "testing/gtest/include/gtest/gtest.h" |
+#include "webrtc/base/event.h" |
#include "webrtc/base/scoped_ptr.h" |
#include "webrtc/common.h" |
#include "webrtc/modules/utility/include/mock/mock_process_thread.h" |
#include "webrtc/system_wrappers/include/critical_section_wrapper.h" |
-#include "webrtc/system_wrappers/include/event_wrapper.h" |
#include "webrtc/system_wrappers/include/ref_count.h" |
#include "webrtc/system_wrappers/include/scoped_vector.h" |
#include "webrtc/test/fake_texture_frame.h" |
@@ -51,7 +51,7 @@ class VideoCaptureInputTest : public ::testing::Test { |
VideoCaptureInputTest() |
: mock_process_thread_(new NiceMock<MockProcessThread>), |
mock_frame_callback_(new NiceMock<MockVideoCaptureCallback>), |
- output_frame_event_(EventWrapper::Create()), |
+ output_frame_event_(false, false), |
stats_proxy_(Clock::GetRealTimeClock(), |
webrtc::VideoSendStream::Config(nullptr), |
webrtc::VideoEncoderConfig::ContentType::kRealtimeVideo) {} |
@@ -82,11 +82,11 @@ class VideoCaptureInputTest : public ::testing::Test { |
if (frame.native_handle() == NULL) |
output_frame_ybuffers_.push_back(frame.buffer(kYPlane)); |
output_frames_.push_back(new VideoFrame(frame)); |
- output_frame_event_->Set(); |
+ output_frame_event_.Set(); |
} |
void WaitOutputFrame() { |
- EXPECT_EQ(kEventSignaled, output_frame_event_->Wait(FRAME_TIMEOUT_MS)); |
+ EXPECT_TRUE(output_frame_event_.Wait(FRAME_TIMEOUT_MS)); |
} |
rtc::scoped_ptr<MockProcessThread> mock_process_thread_; |
@@ -99,7 +99,7 @@ class VideoCaptureInputTest : public ::testing::Test { |
ScopedVector<VideoFrame> input_frames_; |
// Indicate an output frame has arrived. |
- rtc::scoped_ptr<EventWrapper> output_frame_event_; |
+ rtc::Event output_frame_event_; |
// Output delivered frames of VideoCaptureInput. |
ScopedVector<VideoFrame> output_frames_; |
@@ -112,20 +112,19 @@ class VideoCaptureInputTest : public ::testing::Test { |
TEST_F(VideoCaptureInputTest, DoesNotRetainHandleNorCopyBuffer) { |
// Indicate an output frame has arrived. |
- rtc::scoped_ptr<EventWrapper> frame_destroyed_event(EventWrapper::Create()); |
+ rtc::Event frame_destroyed_event(false, false); |
class TestBuffer : public webrtc::I420Buffer { |
public: |
- explicit TestBuffer(EventWrapper* event) |
- : I420Buffer(5, 5), event_(event) {} |
+ explicit TestBuffer(rtc::Event* event) : I420Buffer(5, 5), event_(event) {} |
private: |
friend class rtc::RefCountedObject<TestBuffer>; |
~TestBuffer() override { event_->Set(); } |
- EventWrapper* event_; |
+ rtc::Event* const event_; |
}; |
VideoFrame frame( |
- new rtc::RefCountedObject<TestBuffer>(frame_destroyed_event.get()), 1, 1, |
+ new rtc::RefCountedObject<TestBuffer>(&frame_destroyed_event), 1, 1, |
kVideoRotation_0); |
AddInputFrame(&frame); |
@@ -135,7 +134,7 @@ TEST_F(VideoCaptureInputTest, DoesNotRetainHandleNorCopyBuffer) { |
frame.video_frame_buffer().get()); |
output_frames_.clear(); |
frame.Reset(); |
- EXPECT_EQ(kEventSignaled, frame_destroyed_event->Wait(FRAME_TIMEOUT_MS)); |
+ EXPECT_TRUE(frame_destroyed_event.Wait(FRAME_TIMEOUT_MS)); |
} |
TEST_F(VideoCaptureInputTest, TestNtpTimeStampSetIfRenderTimeSet) { |
@@ -172,12 +171,12 @@ TEST_F(VideoCaptureInputTest, DropsFramesWithSameOrOldNtpTimestamp) { |
// Repeat frame with the same NTP timestamp should drop. |
AddInputFrame(input_frames_[0]); |
- EXPECT_EQ(kEventTimeout, output_frame_event_->Wait(FRAME_TIMEOUT_MS)); |
+ EXPECT_FALSE(output_frame_event_.Wait(FRAME_TIMEOUT_MS)); |
// As should frames with a decreased NTP timestamp. |
input_frames_[0]->set_ntp_time_ms(input_frames_[0]->ntp_time_ms() - 1); |
AddInputFrame(input_frames_[0]); |
- EXPECT_EQ(kEventTimeout, output_frame_event_->Wait(FRAME_TIMEOUT_MS)); |
+ EXPECT_FALSE(output_frame_event_.Wait(FRAME_TIMEOUT_MS)); |
// But delivering with an increased NTP timestamp should succeed. |
input_frames_[0]->set_ntp_time_ms(4711); |