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

Side by Side Diff: webrtc/modules/audio_processing/test/aec_dump_based_simulator.cc

Issue 2415403002: Introduced the new parameter setting scheme for activating the high-pass filter in APM (Closed)
Patch Set: Changes in response to reviewer comments Created 4 years 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 (c) 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2016 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 385 matching lines...) Expand 10 before | Expand all | Expand 10 after
396 : msg.intelligibility_enhancer_enabled(); 396 : msg.intelligibility_enhancer_enabled();
397 config.Set<Intelligibility>(new Intelligibility(enable)); 397 config.Set<Intelligibility>(new Intelligibility(enable));
398 if (settings_.use_verbose_logging) { 398 if (settings_.use_verbose_logging) {
399 std::cout << " intelligibility_enhancer_enabled: " 399 std::cout << " intelligibility_enhancer_enabled: "
400 << (enable ? "true" : "false") << std::endl; 400 << (enable ? "true" : "false") << std::endl;
401 } 401 }
402 } 402 }
403 403
404 if (msg.has_hpf_enabled() || settings_.use_hpf) { 404 if (msg.has_hpf_enabled() || settings_.use_hpf) {
405 bool enable = settings_.use_hpf ? *settings_.use_hpf : msg.hpf_enabled(); 405 bool enable = settings_.use_hpf ? *settings_.use_hpf : msg.hpf_enabled();
406 RTC_CHECK_EQ(AudioProcessing::kNoError, 406 apm_config.high_pass_filter.enabled = enable;
407 ap_->high_pass_filter()->Enable(enable));
408 if (settings_.use_verbose_logging) { 407 if (settings_.use_verbose_logging) {
409 std::cout << " hpf_enabled: " << (enable ? "true" : "false") 408 std::cout << " hpf_enabled: " << (enable ? "true" : "false")
410 << std::endl; 409 << std::endl;
411 } 410 }
412 } 411 }
413 412
414 if (msg.has_ns_enabled() || settings_.use_ns) { 413 if (msg.has_ns_enabled() || settings_.use_ns) {
415 bool enable = settings_.use_ns ? *settings_.use_ns : msg.ns_enabled(); 414 bool enable = settings_.use_ns ? *settings_.use_ns : msg.ns_enabled();
416 RTC_CHECK_EQ(AudioProcessing::kNoError, 415 RTC_CHECK_EQ(AudioProcessing::kNoError,
417 ap_->noise_suppression()->Enable(enable)); 416 ap_->noise_suppression()->Enable(enable));
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
527 } 526 }
528 527
529 void AecDumpBasedSimulator::HandleMessage( 528 void AecDumpBasedSimulator::HandleMessage(
530 const webrtc::audioproc::ReverseStream& msg) { 529 const webrtc::audioproc::ReverseStream& msg) {
531 PrepareReverseProcessStreamCall(msg); 530 PrepareReverseProcessStreamCall(msg);
532 ProcessReverseStream(interface_used_ == InterfaceType::kFixedInterface); 531 ProcessReverseStream(interface_used_ == InterfaceType::kFixedInterface);
533 } 532 }
534 533
535 } // namespace test 534 } // namespace test
536 } // namespace webrtc 535 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698