commit eeafe94f28efb9848b528708334abe300c519795
author hbos <hbos@webrtc.org> Tue Nov 01 03:00:17 2016 -0700
committer Commit bot <commit-bot@chromium.org> Tue Nov 01 10:00:24 2016 +0000
tree b425a9302a220f61bb4488a8fd0f429ce1419e41
parent b84ad63b0a189a33abaf78a527b04f881d881e91

RTCInboundRTPStreamStats[1] added.

Not all stats are collected in this CL, this must be addressed before
closing the issue.

[1] https://w3c.github.io/webrtc-stats/#inboundrtpstats-dict*

  Re-landed after having to be reverted
  https://codereview.webrtc.org/2470683002/ due to depending on a CL
  that was reverted. Now that that has re-landed
  https://codereview.webrtc.org/2470703002/ this is ready to re-land.

BUG=chromium:627816, chromium:657855, chromium:657854
R=hta@webrtc.org
TBR=deadbeef@webrtc.org

Review-Url: https://codereview.webrtc.org/2465173003
Cr-Commit-Position: refs/heads/master@{#14868}

webrtc/api/stats/rtcstats_objects.h

diff --git a/webrtc/api/stats/rtcstats_objects.h b/webrtc/api/stats/rtcstats_objects.h
index 232e66b..4788a14 100644
--- a/webrtc/api/stats/rtcstats_objects.h
+++ b/webrtc/api/stats/rtcstats_objects.h
@@ -239,6 +239,48 @@
   RTCRTPStreamStats(std::string&& id, int64_t timestamp_us);
 };
 
+// https://w3c.github.io/webrtc-stats/#inboundrtpstats-dict*
+// TODO(hbos): Finish implementation and support the remote case
+// |is_remote = true|. Tracking bug crbug.com/657855
+class RTCInboundRTPStreamStats final : public RTCRTPStreamStats {
+ public:
+  WEBRTC_RTCSTATS_DECL();
+
+  RTCInboundRTPStreamStats(const std::string& id, int64_t timestamp_us);
+  RTCInboundRTPStreamStats(std::string&& id, int64_t timestamp_us);
+  RTCInboundRTPStreamStats(const RTCInboundRTPStreamStats& other);
+  ~RTCInboundRTPStreamStats() override;
+
+  RTCStatsMember<uint32_t> packets_received;
+  RTCStatsMember<uint64_t> bytes_received;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<uint32_t> packets_lost;
+  // TODO(hbos): Not collected in the "video" case by |RTCStatsCollector|.
+  // crbug.com/657855
+  RTCStatsMember<double> jitter;
+  RTCStatsMember<double> fraction_lost;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<uint32_t> packets_discarded;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<uint32_t> packets_repaired;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<uint32_t> burst_packets_lost;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<uint32_t> burst_packets_discarded;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<uint32_t> burst_loss_count;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<uint32_t> burst_discard_count;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<double> burst_loss_rate;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<double> burst_discard_rate;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<double> gap_loss_rate;
+  // TODO(hbos): Not collected by |RTCStatsCollector|. crbug.com/657855
+  RTCStatsMember<double> gap_discard_rate;
+};
+
 // https://w3c.github.io/webrtc-stats/#outboundrtpstats-dict*
 // TODO(hbos): Finish implementation and support the remote case
 // |is_remote = true|. Tracking bug crbug.com/657856