commit d0a151c6983d0dcb6011d6ca50943f5236dcc084
author ivoc <ivoc@webrtc.org> Wed Nov 02 09:14:37 2016 -0700
committer Commit bot <commit-bot@chromium.org> Wed Nov 02 16:14:42 2016 +0000
tree 1baa9fb9d9cebf1d7f7028f95099b7b3f69e1a15
parent 827cab3fc2f43153e840078003ad1eb01ab6f61c

Update default values for APM stats to match old behavior.

In the new APM statistics interface, the default values did not match those previously used in AudioSendStream::Stats.

BUG=webrtc:6525

Review-Url: https://codereview.webrtc.org/2469783002
Cr-Commit-Position: refs/heads/master@{#14896}

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 fe4dcee..f63cb20 100644
--- a/webrtc/modules/audio_processing/audio_processing_impl.cc
+++ b/webrtc/modules/audio_processing/audio_processing_impl.cc
@@ -1487,12 +1487,15 @@
     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);
+  int success = public_submodules_->echo_cancellation->GetMetrics(&metrics);
+  if (success == Error::kNoError) {
+    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);