Measure video stats for WebRTC calls to ensure video works.

Measure video stats for WebRTC calls to ensure video works.

This will count the number of dropped frames and the number of decoded
frames. This provides a reasonable guarantee that the video pipeline is
working.

The next step would be to look at the actual pixels in the video.

BUG=434701
Committed: https://crrev.com/56c60209b09297951f2321d5b5818e1a97dca465
Cr-Commit-Position: refs/heads/master@{#305030}

[phoglund_chromium, 2014-11-19 13:41:25]

phoglund@chromium.org changed reviewers:
+ qyearsley@chromium.org

[qyearsley, 2014-11-19 21:15:33]

lgtm

https://codereview.chromium.org/738973005/diff/1/tools/perf/measurements/webr...
File tools/perf/measurements/webrtc.py (right):

https://codereview.chromium.org/738973005/diff/1/tools/perf/measurements/webr...
tools/perf/measurements/webrtc.py:61: self._media_metric.AddResults(tab,
results)
I'm wondering whether there's any particular reason to stop & report media
results last?

[phoglund_chromium, 2014-11-20 09:26:46]

Actually, Quinten, please take another look. I realized I was adding
decoded_*_bytes parameters, but those aren't hooked up for WebRTC video tags so
we may as well exclude them. I followed the pattern from the SystemMemory
metric, which has an exclude_metrics parameter in its AddResult.

https://codereview.chromium.org/738973005/diff/1/tools/perf/measurements/webr...
File tools/perf/measurements/webrtc.py (right):

https://codereview.chromium.org/738973005/diff/1/tools/perf/measurements/webr...
tools/perf/measurements/webrtc.py:61: self._media_metric.AddResults(tab,
results)
On 2014/11/19 21:15:33, qyearsley wrote:
> I'm wondering whether there's any particular reason to stop & report media
> results last?

Nope, none at all. Actually I should move it sorted on the metric name, like I
do in the other methods...

[qyearsley, 2014-11-20 15:49:46]

lgtm still :-)

[phoglund_chromium, 2014-11-20 15:56:29]

On 2014/11/20 15:49:46, qyearsley wrote:
> lgtm still :-)

Thank you sir. Committing.

[phoglund_chromium, 2014-11-20 15:56:33]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2014-11-20 15:57:51]

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

[commit-bot: I haz the power, 2014-11-20 17:39:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-20 17:40:50]

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