Distinguishing between audio and video stats in WebRTC telemetry.

Distinguishing between audio and video stats in WebRTC telemetry.

We used to not be able to tell audio packetsSent from video packetsSent
in the WebRTC stats reports, but with this patch we can. I use the fact
that all video-send metrics will come with other video-send metrics, so
we can classify a report based on what's in it (for instance, a report
with audioInputLevel in it will be an audio-send report).

This fixes buggy output where the code would merge data from send
and receive metrics, for instance leading to packets received sequences
like 10, 20, 15, 30, which doesn't make sense.

BUG=412787
Committed: https://crrev.com/2e727fd948c84a61e5158ab3a53528a1a94fe151
Cr-Commit-Position: refs/heads/master@{#300653}

[phoglund_chromium, 2014-10-21 15:58:22]

phoglund@chromium.org changed reviewers:
+ skyostil@google.com

[phoglund_chromium, 2014-10-21 15:58:23]

phoglund@chromium.org changed required reviewers:
+ skyostil@google.com

[phoglund_chromium, 2014-10-21 15:58:23]

Hi!

tonyg suggested you could review changes to metrics. Please review this one
(which is really a bug fix) and let me know if you have any questions.

[Sami, 2014-10-21 16:23:24]

skyostil@chromium.org changed reviewers:
+ skyostil@chromium.org

[Sami, 2014-10-21 16:23:25]

lgtm. Thanks for extending the test too.

[phoglund_chromium, 2014-10-22 07:05:43]

The CQ bit was checked by phoglund@chromium.org

[phoglund_chromium, 2014-10-22 07:06:05]

On 2014/10/21 16:23:25, Sami wrote:
> lgtm. Thanks for extending the test too.

Of course :)

[commit-bot: I haz the power, 2014-10-22 07:06:17]

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

[commit-bot: I haz the power, 2014-10-22 07:06:18]

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

[commit-bot: I haz the power, 2014-10-22 07:06:19]

All required reviewers (with asterisk prefixes) have not yet approved this CL.

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[phoglund_chromium, 2014-10-22 07:09:29]

phoglund@chromium.org changed required reviewers:
- skyostil@google.com

[phoglund_chromium, 2014-10-22 07:09:35]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2014-10-22 07:10:34]

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

[commit-bot: I haz the power, 2014-10-22 08:52:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-22 08:53:25]

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