content/renderer/media/: Change when to populate WebRTC.AecDelayBasedQuality

content/renderer/media/media_stream_audio_processor_options.cc

Index: content/renderer/media/media_stream_audio_processor_options.cc
diff --git a/content/renderer/media/media_stream_audio_processor_options.cc b/content/renderer/media/media_stream_audio_processor_options.cc
index 2a8eb5f35a8e8c89ce8a1ad7a6a521811c0db049..5c1d45c4196f0e6c62fc5d01b4f4931f237e0912 100644
--- a/content/renderer/media/media_stream_audio_processor_options.cc
+++ b/content/renderer/media/media_stream_audio_processor_options.cc
@@ -249,12 +249,13 @@ void EchoInformation::UpdateAecDelayStats(
           &dummy_median, &dummy_std, &fraction_poor_delays) ==
       webrtc::AudioProcessing::kNoError) {
     num_chunks_ = 0;
+    // Map |fraction_poor_delays| to an Echo Cancellation quality and log in UMA
+    // histogram. See DelayBasedEchoQuality for information on histogram
+    // buckets.
+    UMA_HISTOGRAM_ENUMERATION("WebRTC.AecDelayBasedQuality",

[perkj_chrome, 2015/03/27 09:34:09]

So you no longer want to log when GetDelayMetrics fails? 

[bjornv, 2015/03/27 10:01:53]

That's a good point. I'll remember that one if we're changing the behavior in
the future.
Anyhow, right now a failure is logged as "Great AEC quality", which actually is
unknown.

+                              EchoDelayFrequencyToQuality(fraction_poor_delays),
+                              DELAY_BASED_ECHO_QUALITY_MAX);
   }
-  // Map |fraction_poor_delays| to an Echo Cancellation quality and log in UMA
-  // histogram. See DelayBasedEchoQuality for information on histogram buckets.
-  UMA_HISTOGRAM_ENUMERATION("WebRTC.AecDelayBasedQuality",
-                            EchoDelayFrequencyToQuality(fraction_poor_delays),
-                            DELAY_BASED_ECHO_QUALITY_MAX);
 }
 
 void EnableEchoCancellation(AudioProcessing* audio_processing) {