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

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

Issue 1899123002: Removed the issue with the leading semicolon in the audio processing module experiment description… (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Changes in response to reviewer comments 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 397 matching lines...) Expand 10 before | Expand all | Expand 10 after
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() { 416 std::string EchoCancellationImpl::GetExperimentsDescription() {
417 rtc::CritScope cs(crit_capture_); 417 rtc::CritScope cs(crit_capture_);
418 std::string description = (aec3_enabled_ ? "AEC3" : ""); 418 std::string description = (aec3_enabled_ ? "AEC3;" : "");
419 if (refined_adaptive_filter_enabled_) { 419 if (refined_adaptive_filter_enabled_) {
420 description += ";RefinedAdaptiveFilter"; 420 description += "RefinedAdaptiveFilter;";
421 } 421 }
422 return description; 422 return description;
423 } 423 }
424 424
425 bool EchoCancellationImpl::is_refined_adaptive_filter_enabled() const { 425 bool EchoCancellationImpl::is_refined_adaptive_filter_enabled() const {
426 rtc::CritScope cs(crit_capture_); 426 rtc::CritScope cs(crit_capture_);
427 return refined_adaptive_filter_enabled_; 427 return refined_adaptive_filter_enabled_;
428 } 428 }
429 429
430 bool EchoCancellationImpl::is_extended_filter_enabled() const { 430 bool EchoCancellationImpl::is_extended_filter_enabled() const {
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
578 return error; 578 return error;
579 } 579 }
580 580
581 size_t EchoCancellationImpl::NumCancellersRequired() const { 581 size_t EchoCancellationImpl::NumCancellersRequired() const {
582 RTC_DCHECK(stream_properties_); 582 RTC_DCHECK(stream_properties_);
583 return stream_properties_->num_output_channels * 583 return stream_properties_->num_output_channels *
584 stream_properties_->num_reverse_channels; 584 stream_properties_->num_reverse_channels;
585 } 585 }
586 586
587 } // namespace webrtc 587 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698