Re-land of RTCInboundRTPStreamStats.qpSum collected.

Re-land of RTCInboundRTPStreamStats.qpSum collected.

This was previously only collected for local tracks
(RTCOutboundRTPStreamStats.qpSum).

Spec: https://w3c.github.io/webrtc-stats/#dom-rtcrtpstreamstats-qpsum

This CL also improves some testing in rtcstatscollector_unittest.cc.
Default and non-default values are tested in the same unittests,
removing the test that was specific to default-values, which was
otherwise code duplication.

This is a re-land of https://codereview.webrtc.org/2675943002 after
dependent CL that was re-landed.

BUG=webrtc:7065
TBR=hta@webrtc.org, sakal@webrtc.org

Review-Url: https://codereview.webrtc.org/2703503003
Cr-Commit-Position: refs/heads/master@{#16642}
Committed: https://chromium.googlesource.com/external/webrtc/+/a51d4f34d950b4f31bb0db19cbbb936620e334ab

[hbos, 2017-02-16 12:49:06]

Description was changed from

==========
RTCInboundRTPStreamStats.qpSum collected.

This was previously only collected for local tracks
(RTCOutboundRTPStreamStats.qpSum).

Spec: https://w3c.github.io/webrtc-stats/#dom-rtcrtpstreamstats-qpsum

This CL also improves some testing in rtcstatscollector_unittest.cc.
Default and non-default values are tested in the same unittests,
removing the test that was specific to default-values, which was
otherwise code duplication.

This is a re-land of https://codereview.webrtc.org/2675943002 after
dependent CL that was re-landed.

BUG=webrtc:7065
TBR=hta@webrtc.org, sakal@webrtc.org
==========

to

==========
Re-land of RTCInboundRTPStreamStats.qpSum collected.

This was previously only collected for local tracks
(RTCOutboundRTPStreamStats.qpSum).

Spec: https://w3c.github.io/webrtc-stats/#dom-rtcrtpstreamstats-qpsum

This CL also improves some testing in rtcstatscollector_unittest.cc.
Default and non-default values are tested in the same unittests,
removing the test that was specific to default-values, which was
otherwise code duplication.

This is a re-land of https://codereview.webrtc.org/2675943002 after
dependent CL that was re-landed.

BUG=webrtc:7065
TBR=hta@webrtc.org, sakal@webrtc.org
==========

[hbos, 2017-02-16 12:49:12]

The CQ bit was checked by hbos@webrtc.org

[commit-bot: I haz the power, 2017-02-16 12:49:22]

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-02-16 13:34:48]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487249352938050, "parent_rev":
"454c1d6a2305b7f0c8395b8e76b9a8aaaf8ee258", "commit_rev":
"a51d4f34d950b4f31bb0db19cbbb936620e334ab"}

[commit-bot: I haz the power, 2017-02-16 13:34:53]

Description was changed from

==========
Re-land of RTCInboundRTPStreamStats.qpSum collected.

This was previously only collected for local tracks
(RTCOutboundRTPStreamStats.qpSum).

Spec: https://w3c.github.io/webrtc-stats/#dom-rtcrtpstreamstats-qpsum

This CL also improves some testing in rtcstatscollector_unittest.cc.
Default and non-default values are tested in the same unittests,
removing the test that was specific to default-values, which was
otherwise code duplication.

This is a re-land of https://codereview.webrtc.org/2675943002 after
dependent CL that was re-landed.

BUG=webrtc:7065
TBR=hta@webrtc.org, sakal@webrtc.org
==========

to

==========
Re-land of RTCInboundRTPStreamStats.qpSum collected.

This was previously only collected for local tracks
(RTCOutboundRTPStreamStats.qpSum).

Spec: https://w3c.github.io/webrtc-stats/#dom-rtcrtpstreamstats-qpsum

This CL also improves some testing in rtcstatscollector_unittest.cc.
Default and non-default values are tested in the same unittests,
removing the test that was specific to default-values, which was
otherwise code duplication.

This is a re-land of https://codereview.webrtc.org/2675943002 after
dependent CL that was re-landed.

BUG=webrtc:7065
TBR=hta@webrtc.org, sakal@webrtc.org

Review-Url: https://codereview.webrtc.org/2703503003
Cr-Commit-Position: refs/heads/master@{#16642}
Committed:
https://chromium.googlesource.com/external/webrtc/+/a51d4f34d950b4f31bb0db19c...
==========

[commit-bot: I haz the power, 2017-02-16 13:34:54]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/a51d4f34d950b4f31bb0db19c...

[sakal, 2017-02-16 13:36:33]

lgtm