Cast: end2end bug fixes

media/cast/video_sender/video_sender.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/cast/video_sender/video_sender.h"
 
 #include <list>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 37 matching lines @@
       transport::CastTransportSender* const transport_sender)
       : transport_sender_(transport_sender), sender_info_(), rtp_timestamp_(0) {
     transport_sender_->SubscribeVideoRtpStatsCallback(
         base::Bind(&LocalRtpVideoSenderStatistics::StoreStatistics,
                    base::Unretained(this)));
   }
 
   virtual void GetStatistics(const base::TimeTicks& now,
                              transport::RtcpSenderInfo* sender_info) OVERRIDE {
     // Update RTP timestamp and return last stored statistics.
-    if (rtp_timestamp_) {
+    uint32 ntp_seconds = 0;
+    uint32 ntp_fraction = 0;
+    uint32 rtp_timestamp = 0;
+    if (rtp_timestamp_ > 0) {
       base::TimeDelta time_since_last_send = now - time_sent_;
-      sender_info_.rtp_timestamp =
-          rtp_timestamp_ +
-          time_since_last_send.InMilliseconds() * (kVideoFrequency / 1000);
-    } else {
-      sender_info_.rtp_timestamp = 0;
+      rtp_timestamp = rtp_timestamp_ + time_since_last_send.InMilliseconds() *
+                                           (kVideoFrequency / 1000);
+      // Update NTP time to current time.
+      ConvertTimeTicksToNtp(now, &ntp_seconds, &ntp_fraction);
     }
-    memcpy(sender_info, &sender_info_, sizeof(transport::RtcpSenderInfo));
+    // Populate sender info.
+    sender_info->rtp_timestamp = rtp_timestamp;
+    sender_info->ntp_seconds = ntp_seconds;
+    sender_info->ntp_fraction = ntp_fraction;
+    sender_info->send_packet_count = sender_info_.send_packet_count;
+    sender_info->send_octet_count = sender_info_.send_octet_count;
   }
 
   void StoreStatistics(transport::RtcpSenderInfo& sender_info,
                        base::TimeTicks time_sent,
                        uint32 rtp_timestamp) {
     sender_info_ = sender_info;
     time_sent_ = time_sent;
     rtp_timestamp_ = rtp_timestamp;
   }
 
@@ skipping 399 matching lines @@
 
 void VideoSender::ResendPacketsOnTransportThread(
     const transport::MissingFramesAndPacketsMap& missing_packets) {
   DCHECK(cast_environment_->CurrentlyOn(CastEnvironment::TRANSPORT));
   last_send_time_ = cast_environment_->Clock()->NowTicks();
   transport_sender_->ResendPackets(false, missing_packets);
 }
 
 }  // namespace cast
 }  // namespace media