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

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

Issue 2458993002: Reland of New statistics interface for APM (Closed)
Patch Set: Added implementation of the new non-pure interface function. Created 4 years, 2 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.cc
diff --git a/webrtc/modules/audio_processing/audio_processing_impl.cc b/webrtc/modules/audio_processing/audio_processing_impl.cc
index 12b4c949046f08ddcc3f974d26024bea7e516b54..d535e62d4e0d158064fd97b1f20a4fd517aaabb5 100644
--- a/webrtc/modules/audio_processing/audio_processing_impl.cc
+++ b/webrtc/modules/audio_processing/audio_processing_impl.cc
@@ -443,6 +443,10 @@ int AudioProcessingImpl::InitializeLocked() {
num_proc_channels());
AllocateRenderQueue();
+ int success = public_submodules_->echo_cancellation->enable_metrics(true);
+ RTC_DCHECK_EQ(0, success);
+ success = public_submodules_->echo_cancellation->enable_delay_logging(true);
+ RTC_DCHECK_EQ(0, success);
public_submodules_->echo_control_mobile->Initialize(
proc_split_sample_rate_hz(), num_reverse_channels(),
num_output_channels());
@@ -1417,6 +1421,28 @@ int AudioProcessingImpl::StopDebugRecording() {
#endif // WEBRTC_AUDIOPROC_DEBUG_DUMP
}
+// TODO(ivoc): Remove this when GetStatistics() becomes pure virtual.
+AudioProcessing::AudioProcessingStatistics AudioProcessing::GetStatistics()
+ const {
+ return AudioProcessingStatistics();
+}
+
+AudioProcessing::AudioProcessingStatistics AudioProcessingImpl::GetStatistics()
+ const {
+ AudioProcessingStatistics stats;
+ EchoCancellation::Metrics metrics;
+ public_submodules_->echo_cancellation->GetMetrics(&metrics);
+ stats.a_nlp.Set(metrics.a_nlp);
+ stats.divergent_filter_fraction = metrics.divergent_filter_fraction;
+ stats.echo_return_loss.Set(metrics.echo_return_loss);
+ stats.echo_return_loss_enhancement.Set(metrics.echo_return_loss_enhancement);
+ stats.residual_echo_return_loss.Set(metrics.residual_echo_return_loss);
+ public_submodules_->echo_cancellation->GetDelayMetrics(
+ &stats.delay_median, &stats.delay_standard_deviation,
+ &stats.fraction_poor_delays);
+ return stats;
+}
+
EchoCancellation* AudioProcessingImpl::echo_cancellation() const {
return public_submodules_->echo_cancellation.get();
}
« no previous file with comments | « webrtc/modules/audio_processing/audio_processing_impl.h ('k') | webrtc/modules/audio_processing/audio_processing_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698