Add received interframe delay UMA metrics

Add received interframe delay UMA metrics

BUG=webrtc:7420
Review-Url: https://codereview.webrtc.org/2966733002
Cr-Commit-Position: refs/heads/master@{#18875}
Committed: https://chromium.googlesource.com/external/webrtc/+/4257ab2e0281f248d25726b4917f8aa47b01547c

[ilnik, 2017-06-30 08:53:32]

ilnik@webrtc.org changed reviewers:
+ sprang@webrtc.org

[ilnik, 2017-06-30 08:53:54]

The CQ bit was checked by ilnik@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-30 08:54:04]

Dry run: 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, 2017-06-30 09:16:13]

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

[commit-bot: I haz the power, 2017-06-30 09:16:14]

Dry run: Try jobs failed on following builders:
  linux_msan on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/21699)

[sprang_webrtc, 2017-06-30 21:43:02]

lgtm with nit

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

https://codereview.webrtc.org/2966733002/diff/20001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy.cc:556: if
(interframe_delay_max_ms_screenshare_ == -1 ||
Skip this check and instead DCHECK that |interframe_delay_ms| >= 0

[ilnik, 2017-07-03 07:53:19]

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

https://codereview.webrtc.org/2966733002/diff/20001/webrtc/video/receive_stat...
webrtc/video/receive_statistics_proxy.cc:556: if
(interframe_delay_max_ms_screenshare_ == -1 ||
On 2017/06/30 21:43:02, sprang_webrtc wrote:
> Skip this check and instead DCHECK that |interframe_delay_ms| >= 0

Done.

[ilnik, 2017-07-03 07:53:25]

The CQ bit was checked by ilnik@webrtc.org

[ilnik, 2017-07-03 07:53:26]

The patchset sent to the CQ was uploaded after l-g-t-m from sprang@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2966733002/#ps40001 (title:
"Implement Sprang@ comments")

[commit-bot: I haz the power, 2017-07-03 07:53:32]

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, 2017-07-03 08:15:58]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1499068405769670,
"parent_rev": "ad515c459b817abd3c585d38181215e61ca2ccca", "commit_rev":
"4257ab2e0281f248d25726b4917f8aa47b01547c"}

[commit-bot: I haz the power, 2017-07-03 08:16:03]

Description was changed from

==========
Add received interframe delay UMA metrics

BUG=webrtc:7420
==========

to

==========
Add received interframe delay UMA metrics

BUG=webrtc:7420

Review-Url: https://codereview.webrtc.org/2966733002
Cr-Commit-Position: refs/heads/master@{#18875}
Committed:
https://chromium.googlesource.com/external/webrtc/+/4257ab2e0281f248d25726b49...
==========

[commit-bot: I haz the power, 2017-07-03 08:16:04]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/4257ab2e0281f248d25726b49...