Cast: end2end bug fixes

media/cast/transport/rtp_sender/rtp_sender.cc

Index: media/cast/transport/rtp_sender/rtp_sender.cc
diff --git a/media/cast/transport/rtp_sender/rtp_sender.cc b/media/cast/transport/rtp_sender/rtp_sender.cc
index 993cdc4dad8c69f183bf0a03446e5820a7b153ae..0f232610cec0e1b1a63d8254941608bfd871f7e1 100644
--- a/media/cast/transport/rtp_sender/rtp_sender.cc
+++ b/media/cast/transport/rtp_sender/rtp_sender.cc
@@ -14,8 +14,8 @@ namespace media {
 namespace cast {
 namespace transport {
 
-// Schedule the RTP statistics callback every 100mS.
-static const int kStatsCallbackIntervalMs = 100;
+// Schedule the RTP statistics callback every 33mS.

[pwestin, 2014/02/12 21:16:43]

explain why 33 

[mikhal1, 2014/02/12 23:21:05]

Done.

+static const int kStatsCallbackIntervalMs = 33;
 
 RtpSender::RtpSender(
     base::TickClock* clock,
@@ -23,8 +23,7 @@ RtpSender::RtpSender(
     bool is_audio,
     const scoped_refptr<base::TaskRunner>& transport_task_runner,
     PacedSender* const transport)
-    : clock_(clock),
-      config_(),
+    : config_(),
       transport_(transport),
       stats_callback_(),
       transport_task_runner_(transport_task_runner) {
@@ -48,7 +47,8 @@ RtpSender::RtpSender(
   }
   // Randomly set start values.
   config_.sequence_number = base::RandInt(0, 65535);
-  packetizer_.reset(new RtpPacketizer(transport, storage_.get(), config_));
+  packetizer_.reset(
+      new RtpPacketizer(transport, storage_.get(), config_));
 }
 
 RtpSender::~RtpSender() {}
@@ -138,12 +138,6 @@ void RtpSender::ScheduleNextStatsReport() {
 
 void RtpSender::RtpStatistics() {
   RtcpSenderInfo sender_info;
-  uint32 ntp_seconds = 0;
-  uint32 ntp_fraction = 0;
-  ConvertTimeTicksToNtp(clock_->NowTicks(), &ntp_seconds, &ntp_fraction);
-  sender_info.ntp_seconds = ntp_seconds;
-  sender_info.ntp_fraction = ntp_fraction;
-
   base::TimeTicks time_sent;
   uint32 rtp_timestamp = 0;
   packetizer_->LastSentTimestamp(&time_sent, &rtp_timestamp);