Cast: Make receiver use cast_transport

media/cast/net/rtcp/rtcp_defines.h

Index: media/cast/net/rtcp/rtcp_defines.h
diff --git a/media/cast/net/rtcp/rtcp_defines.h b/media/cast/net/rtcp/rtcp_defines.h
index 3dd23ef6a4cbf28d6eae2602439fc0dc53b254e1..aa87eb6084f68b843439ff6647fbb716492dc75c 100644
--- a/media/cast/net/rtcp/rtcp_defines.h
+++ b/media/cast/net/rtcp/rtcp_defines.h
@@ -109,6 +109,35 @@ typedef base::Callback<void(base::TimeDelta)> RtcpRttCallback;
 typedef
 base::Callback<void(const RtcpReceiverLogMessage&)> RtcpLogMessageCallback;
 
+
+struct RtpReceiverStatistics {
+  RtpReceiverStatistics();
+  uint8 fraction_lost;

[miu, 2014/12/04 04:18:54]

IMO, now's a good time to replace these data members with more appropriate data
types.  Meaning, we shouldn't force the whole implementation to know of the RTCP
wire format.

For example, fraction_lost takes a value from 0 to 255, but that's not intuitive
at all.  Should it be a float, or remain an int that contains a percentage
value?

Also, it would be nice if each of the members of this struct had a short comment
for documentation purposes.

[hubbe, 2014/12/05 23:57:20]

I'm just moving this struct, I'd prefer not to change it.
However, I'll add a TODO for documenting the members.

+  uint32 cumulative_lost;  // 24 bits valid.
+  uint32 extended_high_sequence_number;
+  uint32 jitter;
+};
+
+// These are intended to only be created using Rtcp::ConvertToNTPAndSave.
+struct RtcpTimeData {
+  uint32 ntp_seconds;
+  uint32 ntp_fraction;
+  base::TimeTicks t;

[miu, 2014/12/04 04:18:54]

naming: Can we do a little better than |t|, like |timestamp| or |report_time|?

[hubbe, 2014/12/05 23:57:20]

Done.

+};
+
+struct SendRtcpFromRtpReceiver_Params {
+  SendRtcpFromRtpReceiver_Params();
+  ~SendRtcpFromRtpReceiver_Params();
+  uint32 ssrc;
+  uint32 sender_ssrc;
+  RtcpTimeData time_data;
+  scoped_ptr<RtcpCastMessage> cast_message;
+  base::TimeDelta target_delay;
+  scoped_ptr<std::vector<std::pair<RtpTimestamp, RtcpEvent> > > rtcp_events;
+  scoped_ptr<RtpReceiverStatistics> rtp_receiver_statistics;
+};
+
+
 }  // namespace cast
 }  // namespace media