From bf47f340ee2e11d297f6dc14f5d76abbe7c9a1e2 Mon Sep 17 00:00:00 2001 From: Fredrik Hernqvist Date: Thu, 9 May 2019 10:50:31 +0200 Subject: [PATCH] Add comments to clarify argument meanings in APM impl test Bug: webrtc:10608 Change-Id: Iac1111b739458a1b0ce1cac5e59de06905c085d2 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/135574 Commit-Queue: Fredrik Hernqvist Reviewed-by: Alessio Bazzica Cr-Commit-Position: refs/heads/master@{#27891} --- .../audio_processing/audio_processing_impl_unittest.cc | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/modules/audio_processing/audio_processing_impl_unittest.cc b/modules/audio_processing/audio_processing_impl_unittest.cc index 69fc7793b4..7359f6ba6b 100644 --- a/modules/audio_processing/audio_processing_impl_unittest.cc +++ b/modules/audio_processing/audio_processing_impl_unittest.cc @@ -240,11 +240,15 @@ TEST(AudioProcessingImplTest, MockEchoControl* echo_control_mock = echo_control_factory_ptr->GetNext(); EXPECT_CALL(*echo_control_mock, AnalyzeCapture(NotNull())).Times(1); - EXPECT_CALL(*echo_control_mock, ProcessCapture(NotNull(), false)).Times(1); + EXPECT_CALL(*echo_control_mock, + ProcessCapture(NotNull(), /*echo_path_change=*/false)) + .Times(1); apm->ProcessStream(&frame); EXPECT_CALL(*echo_control_mock, AnalyzeCapture(NotNull())).Times(1); - EXPECT_CALL(*echo_control_mock, ProcessCapture(NotNull(), true)).Times(1); + EXPECT_CALL(*echo_control_mock, + ProcessCapture(NotNull(), /*echo_path_change=*/true)) + .Times(1); apm->SetRuntimeSetting( AudioProcessing::RuntimeSetting::CreateCapturePreGain(2.f)); apm->ProcessStream(&frame);