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

Side by Side Diff: webrtc/api/test/fakeaudiocapturemodule_unittest.cc

Issue 1737013002: Reland of move ignored return code from modules. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 9 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 116
117 char rec_buffer_[FakeAudioCaptureModule::kNumberSamples * 117 char rec_buffer_[FakeAudioCaptureModule::kNumberSamples *
118 FakeAudioCaptureModule::kNumberBytesPerSample]; 118 FakeAudioCaptureModule::kNumberBytesPerSample];
119 size_t rec_buffer_bytes_; 119 size_t rec_buffer_bytes_;
120 }; 120 };
121 121
122 TEST_F(FakeAdmTest, TestProccess) { 122 TEST_F(FakeAdmTest, TestProccess) {
123 // Next process call must be some time in the future (or now). 123 // Next process call must be some time in the future (or now).
124 EXPECT_LE(0, fake_audio_capture_module_->TimeUntilNextProcess()); 124 EXPECT_LE(0, fake_audio_capture_module_->TimeUntilNextProcess());
125 // Process call updates TimeUntilNextProcess() but there are no guarantees on 125 // Process call updates TimeUntilNextProcess() but there are no guarantees on
126 // timing so just check that Process can ba called successfully. 126 // timing so just check that Process can be called successfully.
127 EXPECT_LE(0, fake_audio_capture_module_->Process()); 127 fake_audio_capture_module_->Process();
128 } 128 }
129 129
130 TEST_F(FakeAdmTest, PlayoutTest) { 130 TEST_F(FakeAdmTest, PlayoutTest) {
131 EXPECT_EQ(0, fake_audio_capture_module_->RegisterAudioCallback(this)); 131 EXPECT_EQ(0, fake_audio_capture_module_->RegisterAudioCallback(this));
132 132
133 bool stereo_available = false; 133 bool stereo_available = false;
134 EXPECT_EQ(0, 134 EXPECT_EQ(0,
135 fake_audio_capture_module_->StereoPlayoutIsAvailable( 135 fake_audio_capture_module_->StereoPlayoutIsAvailable(
136 &stereo_available)); 136 &stereo_available));
137 EXPECT_TRUE(stereo_available); 137 EXPECT_TRUE(stereo_available);
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
190 190
191 EXPECT_EQ(0, fake_audio_capture_module_->InitRecording()); 191 EXPECT_EQ(0, fake_audio_capture_module_->InitRecording());
192 EXPECT_EQ(0, fake_audio_capture_module_->StartRecording()); 192 EXPECT_EQ(0, fake_audio_capture_module_->StartRecording());
193 193
194 EXPECT_TRUE_WAIT(push_iterations() > 0, kMsInSecond); 194 EXPECT_TRUE_WAIT(push_iterations() > 0, kMsInSecond);
195 EXPECT_TRUE_WAIT(pull_iterations() > 0, kMsInSecond); 195 EXPECT_TRUE_WAIT(pull_iterations() > 0, kMsInSecond);
196 196
197 EXPECT_EQ(0, fake_audio_capture_module_->StopPlayout()); 197 EXPECT_EQ(0, fake_audio_capture_module_->StopPlayout());
198 EXPECT_EQ(0, fake_audio_capture_module_->StopRecording()); 198 EXPECT_EQ(0, fake_audio_capture_module_->StopRecording());
199 } 199 }
OLDNEW
« no previous file with comments | « webrtc/api/test/fakeaudiocapturemodule.cc ('k') | webrtc/media/engine/fakewebrtcvideocapturemodule.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698