Add log to detect the out of order RTCP packets.

media/cast/net/cast_transport_sender_impl.cc

Index: media/cast/net/cast_transport_sender_impl.cc
diff --git a/media/cast/net/cast_transport_sender_impl.cc b/media/cast/net/cast_transport_sender_impl.cc
index 5f46089bd0a4c1c8132b4e50c021007aedb9d60b..ddc55b99aa977041b89caff7b697dc111a17d0f6 100644
--- a/media/cast/net/cast_transport_sender_impl.cc
+++ b/media/cast/net/cast_transport_sender_impl.cc
@@ -374,12 +374,10 @@ void CastTransportSenderImpl::OnReceivedCastMessage(
   }
 
   if (!cast_message.missing_frames_and_packets.empty()) {
-    // This call does two things.
-    // 1. Specifies that retransmissions for packets not listed in the set are
-    //    cancelled.
-    // 2. Specifies a deduplication window. For video this would be the most
-    //    recent RTT. For audio there is no deduplication.
-    ResendPackets(ssrc, cast_message.missing_frames_and_packets, true,
+    VLOG(2) << "feedback_count: " << cast_message.feedback_count;
+    // This call specifies a deduplication window. For video this would be the
+    // most recent RTT. For audio there is no deduplication.
+    ResendPackets(ssrc, cast_message.missing_frames_and_packets, false,

[miu, 2016/03/29 19:14:14]

Looks like all possible callers of ResendPackets will now pass |false| as the
3rd argument.  So, can you also remove the 3rd argument and the extra code in
CTSImpl::ResendPackets() (and RtpSender::ResendPackets())?

[xjz, 2016/03/29 19:33:49]

As discussed, this is not applicable any more.

                   dedup_info);
   }