RTCInboundRTPStreamStats.qpSum collected.

webrtc/pc/rtcstats_integrationtest.cc

Index: webrtc/pc/rtcstats_integrationtest.cc
diff --git a/webrtc/pc/rtcstats_integrationtest.cc b/webrtc/pc/rtcstats_integrationtest.cc
index 577504728786c476bad527e76bb3c54921859feb..26d45ed879e55f4bbc0313961c5b3c002dd9c184 100644
--- a/webrtc/pc/rtcstats_integrationtest.cc
+++ b/webrtc/pc/rtcstats_integrationtest.cc
@@ -525,7 +525,15 @@ class RTCStatsReportVerifier {
       const RTCInboundRTPStreamStats& inbound_stream) {
     RTCStatsVerifier verifier(report_, &inbound_stream);
     VerifyRTCRTPStreamStats(inbound_stream, &verifier);
-    verifier.TestMemberIsUndefined(inbound_stream.qp_sum);
+    // TODO(hbos): As soon as the encoders provide |qp_sum| values, this
+    // if-statement needs to be included.

[hta-webrtc, 2017/02/07 12:19:41]

Nit: Since this is the incoming stream, it's the decoder, not the encoder that
needs to provide the value.

Writing it like this is good, since it will trigger an error the first time
someone actually provides a value in test.

[hbos, 2017/02/07 15:37:04]

Done.

+    // https://bugs.chromium.org/p/webrtc/issues/detail?id=7065
+    // if (inbound_stream.media_type.is_defined() &&
+    //     *inbound_stream.media_type == "video") {
+    //   verifier.TestMemberIsNonNegative<uint64_t>(inbound_stream.qp_sum);
+    // } else {
+      verifier.TestMemberIsUndefined(inbound_stream.qp_sum);
+    // }
     verifier.TestMemberIsNonNegative<uint32_t>(inbound_stream.packets_received);
     verifier.TestMemberIsNonNegative<uint64_t>(inbound_stream.bytes_received);
     verifier.TestMemberIsNonNegative<uint32_t>(inbound_stream.packets_lost);