Add histogram stats for jitter buffer delay and target delay for received video streams:

Add histogram stats for jitter buffer delay and current/target delay for received video streams:
- "WebRTC.Video.JitterBufferDelayInMs"
- "WebRTC.Video.TargetDelayInMs" (jitter delay + decode time + render delay)
- "WebRTC.Video.CurrentDelayInMs"

BUG=

Committed: https://crrev.com/8688a4e2b53865cfd647a263f8685c3b6de8dacc
Cr-Commit-Position: refs/heads/master@{#12539}

[åsapersson, 2016-04-14 12:23:28]

Description was changed from

==========
Add histogram stats for jitter buffer delay and target delay for received video
streams:
- "WebRTC.Video.JitterBufferDelayInMs"
- "WebRTC.Video.TargetDelayInMs" (jitter delay + decode time + render delay)

BUG=
==========

to

==========
Add histogram stats for jitter buffer delay and target delay for received video
streams:
- "WebRTC.Video.JitterBufferDelayInMs"
- "WebRTC.Video.TargetDelayInMs" (jitter delay + decode time + render delay)

BUG=
==========

[åsapersson, 2016-04-14 12:23:28]

asapersson@webrtc.org changed reviewers:
+ mflodman@webrtc.org

[åsapersson, 2016-04-20 10:46:23]

asapersson@webrtc.org changed reviewers:
+ stefan@webrtc.org

[åsapersson, 2016-04-20 10:48:47]

https://codereview.webrtc.org/1885393002/diff/40001/webrtc/video/receive_stat...
File webrtc/video/receive_statistics_proxy.cc (right):

https://codereview.webrtc.org/1885393002/diff/40001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy.cc:185:
target_delay_counter_.Add(target_delay_ms);
Let target delay stats replace metric below and then reland
https://codereview.webrtc.org/1905563002/ ?

[stefan-webrtc, 2016-04-26 08:19:26]

LG

https://codereview.webrtc.org/1885393002/diff/40001/webrtc/video/receive_stat...
File webrtc/video/receive_statistics_proxy.cc (right):

https://codereview.webrtc.org/1885393002/diff/40001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy.cc:85: int target_delay_ms =
target_delay_counter_.Avg(kMinRequiredDecodeSamples);
We also have "current delay ms" which refers to the actual delay that we impose
on frames, with the goal of reaching the target delay. It can be different
because we're only allowed to change the current delay with 10% per second.
Maybe we should track that too, or instead?

[åsapersson, 2016-04-26 10:19:13]

Description was changed from

==========
Add histogram stats for jitter buffer delay and target delay for received video
streams:
- "WebRTC.Video.JitterBufferDelayInMs"
- "WebRTC.Video.TargetDelayInMs" (jitter delay + decode time + render delay)

BUG=
==========

to

==========
Add histogram stats for jitter buffer delay and current/target delay for
received video streams:
- "WebRTC.Video.JitterBufferDelayInMs"
- "WebRTC.Video.TargetDelayInMs" (jitter delay + decode time + render delay)
- "WebRTC.Video.CurrentDelayInMs"

BUG=
==========

[åsapersson, 2016-04-26 10:23:58]

https://codereview.webrtc.org/1885393002/diff/40001/webrtc/video/receive_stat...
File webrtc/video/receive_statistics_proxy.cc (right):

https://codereview.webrtc.org/1885393002/diff/40001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy.cc:85: int target_delay_ms =
target_delay_counter_.Avg(kMinRequiredDecodeSamples);
On 2016/04/26 08:19:26, stefan-webrtc (holmer) wrote:
> We also have "current delay ms" which refers to the actual delay that we
impose
> on frames, with the goal of reaching the target delay. It can be different
> because we're only allowed to change the current delay with 10% per second.
> Maybe we should track that too, or instead?

Ok, added current delay.

https://codereview.webrtc.org/1885393002/diff/40001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy.cc:185:
target_delay_counter_.Add(target_delay_ms);
On 2016/04/20 10:48:47, åsapersson wrote:
> Let target delay stats replace metric below and then reland
> https://codereview.webrtc.org/1905563002/ ?

Reland CL above?

[stefan-webrtc, 2016-04-26 10:39:42]

lgtm

https://codereview.webrtc.org/1885393002/diff/40001/webrtc/video/receive_stat...
File webrtc/video/receive_statistics_proxy.cc (right):

https://codereview.webrtc.org/1885393002/diff/40001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy.cc:185:
target_delay_counter_.Add(target_delay_ms);
On 2016/04/26 10:23:58, åsapersson wrote:
> On 2016/04/20 10:48:47, åsapersson wrote:
> > Let target delay stats replace metric below and then reland
> > https://codereview.webrtc.org/1905563002/ ?
> 
> Reland CL above?

Sgtm if Magnus agrees.

[åsapersson, 2016-04-27 12:10:05]

The CQ bit was checked by asapersson@webrtc.org

[commit-bot: I haz the power, 2016-04-27 12:10:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1885393002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1885393002/60001

[commit-bot: I haz the power, 2016-04-27 14:10:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-27 14:10:45]

Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[åsapersson, 2016-04-28 06:40:35]

The CQ bit was checked by asapersson@webrtc.org

[commit-bot: I haz the power, 2016-04-28 06:40:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1885393002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1885393002/60001

[commit-bot: I haz the power, 2016-04-28 06:42:38]

Description was changed from

==========
Add histogram stats for jitter buffer delay and current/target delay for
received video streams:
- "WebRTC.Video.JitterBufferDelayInMs"
- "WebRTC.Video.TargetDelayInMs" (jitter delay + decode time + render delay)
- "WebRTC.Video.CurrentDelayInMs"

BUG=
==========

to

==========
Add histogram stats for jitter buffer delay and current/target delay for
received video streams:
- "WebRTC.Video.JitterBufferDelayInMs"
- "WebRTC.Video.TargetDelayInMs" (jitter delay + decode time + render delay)
- "WebRTC.Video.CurrentDelayInMs"

BUG=
==========

[commit-bot: I haz the power, 2016-04-28 06:42:39]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-01 22:01:04]

Description was changed from

==========
Add histogram stats for jitter buffer delay and current/target delay for
received video streams:
- "WebRTC.Video.JitterBufferDelayInMs"
- "WebRTC.Video.TargetDelayInMs" (jitter delay + decode time + render delay)
- "WebRTC.Video.CurrentDelayInMs"

BUG=
==========

to

==========
Add histogram stats for jitter buffer delay and current/target delay for
received video streams:
- "WebRTC.Video.JitterBufferDelayInMs"
- "WebRTC.Video.TargetDelayInMs" (jitter delay + decode time + render delay)
- "WebRTC.Video.CurrentDelayInMs"

BUG=

Committed: https://crrev.com/8688a4e2b53865cfd647a263f8685c3b6de8dacc
Cr-Commit-Position: refs/heads/master@{#12539}
==========

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

Patchset 4 (id:??) landed as
https://crrev.com/8688a4e2b53865cfd647a263f8685c3b6de8dacc
Cr-Commit-Position: refs/heads/master@{#12539}