| 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 12ef076c6d58f378458d4bb87ccff204d3e3788f..05b09040017c750b3fd888f9e5a240c09b69180b 100644
|
| --- a/webrtc/video/video_capture_input_unittest.cc
|
| +++ b/webrtc/video/video_capture_input_unittest.cc
|
| @@ -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)) {}
|
|
|
| @@ -81,11 +81,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_;
|
| @@ -98,7 +98,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_;
|
| @@ -111,20 +111,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);
|
| @@ -134,7 +133,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) {
|
| @@ -171,12 +170,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);
|
|
|