Add NetEq::FilteredCurrentDelayMs() and use it in VoiceEngine

Add NetEq::FilteredCurrentDelayMs() and use it in VoiceEngine

The new method returns the current total delay (packet buffer and sync
buffer) in ms, with smoothing applied to even out short-time
fluctuations due to jitter. The packet buffer part of the delay is not
updated during DTX/CNG periods.

This CL also pipes the new metric through ACM and uses it in
VoiceEngine. It replaces the previous method of estimating the buffer
delay (where an inserted packet's RTP timestamp was compared with the
last played timestamp from NetEq). The new method works better under
periods of DTX/CNG.

Committed: https://crrev.com/b3f1c5d2fe00ac2201e533345f55eee220d66876
Cr-Commit-Position: refs/heads/master@{#13855}

[hlundin-webrtc, 2016-08-22 15:27:18]

henrik.lundin@webrtc.org changed reviewers:
+ tina.legrand@webrtc.org

[hlundin-webrtc, 2016-08-22 15:27:18]

Tina, PTAL.

[tlegrand-webrtc, 2016-08-22 18:57:17]

On 2016/08/22 15:27:18, hlundin-webrtc wrote:
> Tina, PTAL.

Nice! LGTM

[hlundin-webrtc, 2016-08-22 21:18:07]

The CQ bit was checked by henrik.lundin@webrtc.org

[commit-bot: I haz the power, 2016-08-22 21:18:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-08-22 22:39:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-22 22:40:05]

Description was changed from

==========
Add NetEq::FilteredCurrentDelayMs() and use it in VoiceEngine

The new method returns the current total delay (packet buffer and sync
buffer) in ms, with smoothing applied to even out short-time
fluctuations due to jitter. The packet buffer part of the delay is not
updated during DTX/CNG periods.

This CL also pipes the new metric through ACM and uses it in
VoiceEngine. It replaces the previous method of estimating the buffer
delay (where an inserted packet's RTP timestamp was compared with the
last played timestamp from NetEq). The new method works better under
periods of DTX/CNG.
==========

to

==========
Add NetEq::FilteredCurrentDelayMs() and use it in VoiceEngine

The new method returns the current total delay (packet buffer and sync
buffer) in ms, with smoothing applied to even out short-time
fluctuations due to jitter. The packet buffer part of the delay is not
updated during DTX/CNG periods.

This CL also pipes the new metric through ACM and uses it in
VoiceEngine. It replaces the previous method of estimating the buffer
delay (where an inserted packet's RTP timestamp was compared with the
last played timestamp from NetEq). The new method works better under
periods of DTX/CNG.

Committed: https://crrev.com/b3f1c5d2fe00ac2201e533345f55eee220d66876
Cr-Commit-Position: refs/heads/master@{#13855}
==========

[commit-bot: I haz the power, 2016-08-22 22:40:06]

Patchset 1 (id:??) landed as
https://crrev.com/b3f1c5d2fe00ac2201e533345f55eee220d66876
Cr-Commit-Position: refs/heads/master@{#13855}

[the sun, 2016-08-23 06:45:06]

solenberg@webrtc.org changed reviewers:
+ solenberg@webrtc.org

[the sun, 2016-08-23 06:45:07]

https://codereview.webrtc.org/2262203002/diff/1/webrtc/voice_engine/channel.cc
File webrtc/voice_engine/channel.cc (right):

https://codereview.webrtc.org/2262203002/diff/1/webrtc/voice_engine/channel.c...
webrtc/voice_engine/channel.cc:3391: // be deleted, along with a few member
variables. (WebRTC issue 6237.)
Oh, that would be so nice!