[media_stream_audio_processor] Avoids updating UMA stats before AEC has valid results

[media_stream_audio_processor] Avoids updating UMA stats before AEC has valid results

Stats are updated and logged continuously when echo cancellation is enabled. Being enabled is no guarantee for actually running since it, for example, is enabled by default when opening the microphone.
All these invalid data clutter the UMA histogram.

The improvement is a check for the first frame containing an echo state before start logging. This can only happen when the echo cancellation is actually running.

BUG=450193
Committed: https://crrev.com/40d81fd5fea87aa63e82f1875e5874315ab215cf
Cr-Commit-Position: refs/heads/master@{#326761}

[bjornv, 2015-04-22 06:44:56]

bjornv@chromium.org changed reviewers:
+ tommi@chromium.org

[tommi (sloooow) - chröme, 2015-04-23 14:35:35]

lgtm % nit

https://codereview.chromium.org/1081063003/diff/1/content/renderer/media/medi...
File content/renderer/media/media_stream_audio_processor_options.cc (right):

https://codereview.chromium.org/1081063003/diff/1/content/renderer/media/medi...
content/renderer/media/media_stream_audio_processor_options.cc:235: if
(!echo_frames_received_ & !echo_cancellation->stream_has_echo()) {
no {}

[bjornv, 2015-04-24 08:01:55]

Thanks for reviewing this. Now when perkj@ is on parental leave, you're the only
webrtc owner.

https://codereview.chromium.org/1081063003/diff/1/content/renderer/media/medi...
File content/renderer/media/media_stream_audio_processor_options.cc (right):

https://codereview.chromium.org/1081063003/diff/1/content/renderer/media/medi...
content/renderer/media/media_stream_audio_processor_options.cc:235: if
(!echo_frames_received_ & !echo_cancellation->stream_has_echo()) {
On 2015/04/23 14:35:35, tommi wrote:
> no {}

Done.

[bjornv, 2015-04-24 09:05:11]

The CQ bit was checked by bjornv@chromium.org

[bjornv, 2015-04-24 09:05:12]

The patchset sent to the CQ was uploaded after l-g-t-m from tommi@chromium.org
Link to the patchset: https://codereview.chromium.org/1081063003/#ps20001
(title: "Removed {}")

[commit-bot: I haz the power, 2015-04-24 09:05:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1081063003/20001

[commit-bot: I haz the power, 2015-04-24 09:09:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-24 09:10:30]

Patchset 2 (id:??) landed as
https://crrev.com/40d81fd5fea87aa63e82f1875e5874315ab215cf
Cr-Commit-Position: refs/heads/master@{#326761}