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

Side by Side Diff: webrtc/modules/audio_processing/echo_cancellation_impl.cc

Issue 1886233003: Added a protobuf field for the audio processing module to store the status of experiments (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Added unittest for reading of the added protobuf field Created 4 years, 8 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 (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 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 395 matching lines...) Expand 10 before | Expand all | Expand 10 after
406 bool EchoCancellationImpl::is_delay_agnostic_enabled() const { 406 bool EchoCancellationImpl::is_delay_agnostic_enabled() const {
407 rtc::CritScope cs(crit_capture_); 407 rtc::CritScope cs(crit_capture_);
408 return delay_agnostic_enabled_; 408 return delay_agnostic_enabled_;
409 } 409 }
410 410
411 bool EchoCancellationImpl::is_aec3_enabled() const { 411 bool EchoCancellationImpl::is_aec3_enabled() const {
412 rtc::CritScope cs(crit_capture_); 412 rtc::CritScope cs(crit_capture_);
413 return aec3_enabled_; 413 return aec3_enabled_;
414 } 414 }
415 415
416 std::string EchoCancellationImpl::GetExperimentsDescription() {
417 rtc::CritScope cs(crit_capture_);
418 return aec3_enabled_ ? "AEC3" : "";
419 }
420
416 bool EchoCancellationImpl::is_extended_filter_enabled() const { 421 bool EchoCancellationImpl::is_extended_filter_enabled() const {
417 rtc::CritScope cs(crit_capture_); 422 rtc::CritScope cs(crit_capture_);
418 return extended_filter_enabled_; 423 return extended_filter_enabled_;
419 } 424 }
420 425
421 // TODO(bjornv): How should we handle the multi-channel case? 426 // TODO(bjornv): How should we handle the multi-channel case?
422 int EchoCancellationImpl::GetDelayMetrics(int* median, int* std) { 427 int EchoCancellationImpl::GetDelayMetrics(int* median, int* std) {
423 rtc::CritScope cs(crit_capture_); 428 rtc::CritScope cs(crit_capture_);
424 float fraction_poor_delays = 0; 429 float fraction_poor_delays = 0;
425 return GetDelayMetrics(median, std, &fraction_poor_delays); 430 return GetDelayMetrics(median, std, &fraction_poor_delays);
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
559 return error; 564 return error;
560 } 565 }
561 566
562 size_t EchoCancellationImpl::NumCancellersRequired() const { 567 size_t EchoCancellationImpl::NumCancellersRequired() const {
563 RTC_DCHECK(stream_properties_); 568 RTC_DCHECK(stream_properties_);
564 return stream_properties_->num_output_channels * 569 return stream_properties_->num_output_channels *
565 stream_properties_->num_reverse_channels; 570 stream_properties_->num_reverse_channels;
566 } 571 }
567 572
568 } // namespace webrtc 573 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/audio_processing/echo_cancellation_impl.h ('k') | webrtc/modules/audio_processing/test/debug_dump_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698