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

Unified Diff: webrtc/modules/audio_processing/audio_processing_impl_locking_unittest.cc

Issue 1678813002: Moved the GainControlForNewAGC class to a separate file. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Removed threadchecker and introduced lock 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
Index: webrtc/modules/audio_processing/audio_processing_impl_locking_unittest.cc
diff --git a/webrtc/modules/audio_processing/audio_processing_impl_locking_unittest.cc b/webrtc/modules/audio_processing/audio_processing_impl_locking_unittest.cc
index 2e22b2c51ebfc773c30b0272e20f92edc517f86f..7ce6a569d16566507c0867a2004dbb69236452d8 100644
--- a/webrtc/modules/audio_processing/audio_processing_impl_locking_unittest.cc
+++ b/webrtc/modules/audio_processing/audio_processing_impl_locking_unittest.cc
@@ -600,7 +600,6 @@ bool StatsProcessor::Process() {
(test_config_->aec_type ==
AecType::BasicWebRtcAecSettingsWithAecMobile));
EXPECT_TRUE(apm_->gain_control()->is_enabled());
- apm_->gain_control()->stream_analog_level();
EXPECT_TRUE(apm_->noise_suppression()->is_enabled());
// The below return values are not testable.
@@ -713,9 +712,12 @@ void CaptureProcessor::CallApmCaptureSide() {
// Prepare a proper capture side processing API call input.
PrepareFrame();
- // Set the stream delay
+ // Set the stream delay.
apm_->set_stream_delay_ms(30);
+ // Set the analog level.
+ apm_->gain_control()->set_stream_analog_level(80);
+
// Call the specified capture side API processing method.
int result = AudioProcessing::kNoError;
switch (test_config_->capture_api_function) {
@@ -738,6 +740,9 @@ void CaptureProcessor::CallApmCaptureSide() {
FAIL();
}
+ // Retrieve the new analog level.
+ apm_->gain_control()->stream_analog_level();
+
// Check the return code for error.
ASSERT_EQ(AudioProcessing::kNoError, result);
}

Powered by Google App Engine
This is Rietveld 408576698