Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(35)

Unified Diff: webrtc/modules/utility/source/process_thread_impl_unittest.cc

Issue 1736663004: Revert of Remove ignored return code from modules. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.cc ('k') | webrtc/modules/video_capture/video_capture_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/utility/source/process_thread_impl_unittest.cc
diff --git a/webrtc/modules/utility/source/process_thread_impl_unittest.cc b/webrtc/modules/utility/source/process_thread_impl_unittest.cc
index 4f70e8acdec1e5d5db28c349cdcdc250808b79ee..0b35fad7d2b5f5e4e7c0119e4d3cb2414079ae27 100644
--- a/webrtc/modules/utility/source/process_thread_impl_unittest.cc
+++ b/webrtc/modules/utility/source/process_thread_impl_unittest.cc
@@ -28,7 +28,7 @@
class MockModule : public Module {
public:
MOCK_METHOD0(TimeUntilNextProcess, int64_t());
- MOCK_METHOD0(Process, void());
+ MOCK_METHOD0(Process, int32_t());
MOCK_METHOD1(ProcessThreadAttached, void(ProcessThread*));
};
@@ -77,8 +77,8 @@
MockModule module;
EXPECT_CALL(module, TimeUntilNextProcess()).WillRepeatedly(Return(0));
EXPECT_CALL(module, Process())
- .WillOnce(DoAll(SetEvent(event.get()), Return()))
- .WillRepeatedly(Return());
+ .WillOnce(DoAll(SetEvent(event.get()), Return(0)))
+ .WillRepeatedly(Return(0));
EXPECT_CALL(module, ProcessThreadAttached(&thread)).Times(1);
thread.RegisterModule(&module);
@@ -97,8 +97,8 @@
MockModule module;
EXPECT_CALL(module, TimeUntilNextProcess()).WillRepeatedly(Return(0));
EXPECT_CALL(module, Process())
- .WillOnce(DoAll(SetEvent(event.get()), Return()))
- .WillRepeatedly(Return());
+ .WillOnce(DoAll(SetEvent(event.get()), Return(0)))
+ .WillRepeatedly(Return(0));
thread.RegisterModule(&module);
@@ -122,8 +122,8 @@
EXPECT_CALL(module, Process())
.WillOnce(DoAll(SetEvent(event.get()),
Increment(&process_count),
- Return()))
- .WillRepeatedly(DoAll(Increment(&process_count), Return()));
+ Return(0)))
+ .WillRepeatedly(DoAll(Increment(&process_count), Return(0)));
thread.RegisterModule(&module);
@@ -163,8 +163,8 @@
EXPECT_CALL(module, Process())
.WillOnce(DoAll(SetTimestamp(&called_time),
SetEvent(event.get()),
- Return()))
- .WillRepeatedly(Return());
+ Return(0)))
+ .WillRepeatedly(Return(0));
EXPECT_CALL(module, ProcessThreadAttached(&thread)).Times(1);
thread.RegisterModule(&module);
@@ -225,7 +225,7 @@
.WillRepeatedly(Return(20));
EXPECT_CALL(module, Process())
.WillRepeatedly(DoAll(Increment(&callback_count),
- Return()));
+ Return(0)));
EXPECT_CALL(module, ProcessThreadAttached(&thread)).Times(1);
thread.RegisterModule(&module);
@@ -269,9 +269,10 @@
Return(1000)))
.WillOnce(Return(1000));
EXPECT_CALL(module, Process())
- .WillOnce(
- DoAll(SetTimestamp(&called_time), SetEvent(called.get()), Return()))
- .WillRepeatedly(Return());
+ .WillOnce(DoAll(SetTimestamp(&called_time),
+ SetEvent(called.get()),
+ Return(0)))
+ .WillRepeatedly(Return(0));
EXPECT_CALL(module, ProcessThreadAttached(&thread)).Times(1);
thread.RegisterModule(&module);
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.cc ('k') | webrtc/modules/video_capture/video_capture_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698