Revert of RTCInboundRTPStreamStats.qpSum collected.

Revert of RTCInboundRTPStreamStats.qpSum collected. (patchset #4 id:80001 of https://codereview.webrtc.org/2675943002/ )

Reason for revert:
Breaks downstream build.

Original issue's description:
> 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.
> 
> BUG=webrtc:7065
> 
> Review-Url: https://codereview.webrtc.org/2675943002
> Cr-Commit-Position: refs/heads/master@{#16477}
> Committed: https://chromium.googlesource.com/external/webrtc/+/cd195bea5e38a61e73b08f82445560d9136cd9da

TBR=sakal@webrtc.org,hta@webrtc.org,hbos@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7065
Review-Url: https://codereview.webrtc.org/2687483002 .
Cr-Commit-Position: refs/heads/master@{#16479}
Committed: https://chromium.googlesource.com/external/webrtc/+/ed02c6d68f31baed3cfb7656062895a4d1cb7e65

[skvlad, 2017-02-07 18:44:22]

The CQ bit was checked by skvlad@webrtc.org

[skvlad, 2017-02-07 18:44:22]

Created Revert of RTCInboundRTPStreamStats.qpSum collected.

[commit-bot: I haz the power, 2017-02-07 18:44:31]

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-07 18:44:32]

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

[commit-bot: I haz the power, 2017-02-07 18:44:33]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-webrtc-committers".
Note that this has nothing to do with OWNERS files.

[skvlad, 2017-02-07 18:45:33]

Description was changed from

==========
Revert of RTCInboundRTPStreamStats.qpSum collected. (patchset #4 id:80001 of
https://codereview.webrtc.org/2675943002/ )

Reason for revert:
Breaks downstream build.

Original issue's description:
> 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.
> 
> BUG=webrtc:7065
> 
> Review-Url: https://codereview.webrtc.org/2675943002
> Cr-Commit-Position: refs/heads/master@{#16477}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/cd195bea5e38a61e73b08f824...

TBR=sakal@webrtc.org,hta@webrtc.org,hbos@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7065
==========

to

==========
Revert of RTCInboundRTPStreamStats.qpSum collected. (patchset #4 id:80001 of
https://codereview.webrtc.org/2675943002/ )

Reason for revert:
Breaks downstream build.

Original issue's description:
> 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.
> 
> BUG=webrtc:7065
> 
> Review-Url: https://codereview.webrtc.org/2675943002
> Cr-Commit-Position: refs/heads/master@{#16477}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/cd195bea5e38a61e73b08f824...

TBR=sakal@webrtc.org,hta@webrtc.org,hbos@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7065

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

[skvlad, 2017-02-07 18:45:34]

Committed patchset #1 (id:1) manually as
ed02c6d68f31baed3cfb7656062895a4d1cb7e65 (presubmit successful).

[hbos, 2017-02-07 18:45:59]

lgtm

[hbos, 2017-02-07 18:46:47]

Does this break downstream or is it a previous CL that this depends on that
breaks it?

[skvlad, 2017-02-07 19:00:30]

On 2017/02/07 18:46:47, hbos wrote:
> Does this break downstream or is it a previous CL that this depends on that
> breaks it?

This CL depends on https://codereview.webrtc.org/2680893002/, which breaks the
build. Unfortunately, it appears that both need to be reverted.