cast: Split Rtcp into two for sender and receiver

media/cast/net/rtcp/rtcp_unittest.cc

 // Copyright 2014 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 <stdint.h>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/test/simple_test_tick_clock.h"
 #include "media/cast/net/cast_transport_config.h"
 #include "media/cast/net/pacing/paced_sender.h"
-#include "media/cast/net/rtcp/rtcp.h"
+#include "media/cast/net/rtcp/receiver_rtcp_session.h"
+#include "media/cast/net/rtcp/rtcp_session.h"
 #include "media/cast/net/rtcp/rtcp_utility.h"
+#include "media/cast/net/rtcp/sender_rtcp_session.h"
 #include "media/cast/test/skewed_tick_clock.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 namespace media {
 namespace cast {
 
+namespace {
+
+media::cast::RtcpTimeData CreateRtcpTimeData(base::TimeTicks now) {
+  media::cast::RtcpTimeData ret;
+  ret.timestamp = now;
+  media::cast::ConvertTimeTicksToNtp(now, &ret.ntp_seconds, &ret.ntp_fraction);
+  return ret;
+}
+
+}  // namespace
+
 using testing::_;
 
 static const uint32 kSenderSsrc = 0x10203;
 static const uint32 kReceiverSsrc = 0x40506;
 static const int kInitialReceiverClockOffsetSeconds = -5;
 
 class FakeRtcpTransport : public PacedPacketSender {
  public:
   explicit FakeRtcpTransport(base::SimpleTestTickClock* clock)
       : clock_(clock),
         packet_delay_(base::TimeDelta::FromMilliseconds(42)),
         paused_(false) {}
 
-  void set_rtcp_destination(Rtcp* rtcp) { rtcp_ = rtcp; }
+  void set_rtcp_destination(RtcpSession* rtcp_session) {
+    rtcp_session_ = rtcp_session;
+  }
 
   base::TimeDelta packet_delay() const { return packet_delay_; }
   void set_packet_delay(base::TimeDelta delay) { packet_delay_ = delay; }
 
   bool SendRtcpPacket(uint32 ssrc, PacketRef packet) final {
     clock_->Advance(packet_delay_);
     if (paused_) {
       packet_queue_.push_back(packet);
     } else {
-      rtcp_->IncomingRtcpPacket(&packet->data[0], packet->data.size());
+      rtcp_session_->IncomingRtcpPacket(&packet->data[0], packet->data.size());
     }
     return true;
   }
 
   bool SendPackets(const SendPacketVector& packets) final { return false; }
 
   bool ResendPackets(const SendPacketVector& packets,
                      const DedupInfo& dedup_info) final {
     return false;
   }
 
   void CancelSendingPacket(const PacketKey& packet_key) final {}
 
   void Pause() {
     paused_ = true;
   }
 
   void Unpause() {
     paused_ = false;
     for (size_t i = 0; i < packet_queue_.size(); ++i) {
-      rtcp_->IncomingRtcpPacket(&packet_queue_[i]->data[0],
-                                packet_queue_[i]->data.size());
+      rtcp_session_->IncomingRtcpPacket(&packet_queue_[i]->data[0],
+                                        packet_queue_[i]->data.size());
     }
     packet_queue_.clear();
   }
 
   void ReversePacketQueue() {
     std::reverse(packet_queue_.begin(), packet_queue_.end());
   }
 
  private:
   base::SimpleTestTickClock* const clock_;
   base::TimeDelta packet_delay_;
-  Rtcp* rtcp_;
+  RtcpSession* rtcp_session_;  //  RTCP destination.
   bool paused_;
   std::vector<PacketRef> packet_queue_;
 
   DISALLOW_COPY_AND_ASSIGN(FakeRtcpTransport);
 };
 
 class MockFrameSender {
  public:
   MockFrameSender() {}
   virtual ~MockFrameSender() {}
@@ skipping 15 matching lines @@
         receiver_to_sender_(sender_clock_.get()),
         rtcp_for_sender_(base::Bind(&MockFrameSender::OnReceivedCastFeedback,
                                     base::Unretained(&mock_frame_sender_)),
                          base::Bind(&MockFrameSender::OnMeasuredRoundTripTime,
                                     base::Unretained(&mock_frame_sender_)),
                          RtcpLogMessageCallback(),
                          sender_clock_.get(),
                          &sender_to_receiver_,
                          kSenderSsrc,
                          kReceiverSsrc),
-        rtcp_for_receiver_(RtcpCastMessageCallback(),
-                           RtcpRttCallback(),
-                           RtcpLogMessageCallback(),
-                           receiver_clock_.get(),
+        rtcp_for_receiver_(receiver_clock_.get(),
                            &receiver_to_sender_,
                            kReceiverSsrc,
                            kSenderSsrc) {
     sender_clock_->Advance(base::TimeTicks::Now() - base::TimeTicks());
     receiver_clock_->SetSkew(
         1.0,  // No skew.
         base::TimeDelta::FromSeconds(kInitialReceiverClockOffsetSeconds));
 
     sender_to_receiver_.set_rtcp_destination(&rtcp_for_receiver_);
     receiver_to_sender_.set_rtcp_destination(&rtcp_for_sender_);
   }
 
   ~RtcpTest() override {}
 
   scoped_ptr<base::SimpleTestTickClock> sender_clock_;
   scoped_ptr<test::SkewedTickClock> receiver_clock_;
   FakeRtcpTransport sender_to_receiver_;
   FakeRtcpTransport receiver_to_sender_;
   MockFrameSender mock_frame_sender_;
-  Rtcp rtcp_for_sender_;
-  Rtcp rtcp_for_receiver_;
+  SenderRtcpSession rtcp_for_sender_;
+  ReceiverRtcpSession rtcp_for_receiver_;
 
+ private:
   DISALLOW_COPY_AND_ASSIGN(RtcpTest);
 };
 
 TEST_F(RtcpTest, LipSyncGleanedFromSenderReport) {
   // Initially, expect no lip-sync info receiver-side without having first
   // received a RTCP packet.
   base::TimeTicks reference_time;
   RtpTimeTicks rtp_timestamp;
   ASSERT_FALSE(rtcp_for_receiver_.GetLatestLipSyncTimes(&rtp_timestamp,
                                                         &reference_time));
 
   // Send a Sender Report to the receiver.
   const base::TimeTicks reference_time_sent = sender_clock_->NowTicks();
   const RtpTimeTicks rtp_timestamp_sent =
       RtpTimeTicks().Expand(UINT32_C(0xbee5));
-  rtcp_for_sender_.SendRtcpFromRtpSender(
-      reference_time_sent, rtp_timestamp_sent, 1, 1);
+  rtcp_for_sender_.SendRtcpReport(reference_time_sent, rtp_timestamp_sent, 1,
+                                  1);
 
   // Now the receiver should have lip-sync info.  Confirm that the lip-sync
   // reference time is the same as that sent.
   EXPECT_TRUE(rtcp_for_receiver_.GetLatestLipSyncTimes(&rtp_timestamp,
                                                        &reference_time));
   const base::TimeTicks rolled_back_time =
       (reference_time -
        // Roll-back relative clock offset:
        base::TimeDelta::FromSeconds(kInitialReceiverClockOffsetSeconds) -
        // Roll-back packet transmission time (because RTT is not yet known):
        sender_to_receiver_.packet_delay());
   EXPECT_NEAR(0, (reference_time_sent - rolled_back_time).InMicroseconds(), 5);
   EXPECT_EQ(rtp_timestamp_sent, rtp_timestamp);
 }
 
 // TODO(miu): There were a few tests here that didn't actually test anything
 // except that the code wouldn't crash and a callback method was invoked.  We
 // need to fill-in more testing of RTCP now that much of the refactoring work
 // has been completed.
 
 TEST_F(RtcpTest, RoundTripTimesDeterminedFromReportPingPong) {
   const int iterations = 12;
   EXPECT_CALL(mock_frame_sender_, OnMeasuredRoundTripTime(_))
       .Times(iterations);
 
-  // Initially, neither side knows the round trip time.
+  // Sender does not know the RTT yet.
   ASSERT_EQ(base::TimeDelta(), rtcp_for_sender_.current_round_trip_time());
-  ASSERT_EQ(base::TimeDelta(), rtcp_for_receiver_.current_round_trip_time());
 
   // Do a number of ping-pongs, checking how the round trip times are measured
-  // by the sender and receiver.
+  // by the sender.
   base::TimeDelta expected_rtt_according_to_sender;
-  base::TimeDelta expected_rtt_according_to_receiver;
   for (int i = 0; i < iterations; ++i) {
     const base::TimeDelta one_way_trip_time =
         base::TimeDelta::FromMilliseconds(1 << i);
     sender_to_receiver_.set_packet_delay(one_way_trip_time);
     receiver_to_sender_.set_packet_delay(one_way_trip_time);
 
     // Sender --> Receiver
     base::TimeTicks reference_time_sent = sender_clock_->NowTicks();
     const RtpTimeTicks rtp_timestamp_sent =
         RtpTimeTicks().Expand<uint32_t>(0xbee5) + RtpTimeDelta::FromTicks(i);
-    rtcp_for_sender_.SendRtcpFromRtpSender(
-        reference_time_sent, rtp_timestamp_sent, 1, 1);
+    rtcp_for_sender_.SendRtcpReport(reference_time_sent, rtp_timestamp_sent, 1,
+                                    1);
     EXPECT_EQ(expected_rtt_according_to_sender,
               rtcp_for_sender_.current_round_trip_time());
-#ifdef SENDER_PROVIDES_REPORT_BLOCK
-    EXPECT_EQ(expected_rtt_according_to_receiver,
-              rtcp_for_receiver_.current_round_trip_time());
-#endif
 
     // Receiver --> Sender
     RtpReceiverStatistics stats;
-    rtcp_for_receiver_.SendRtcpFromRtpReceiver(
-        rtcp_for_receiver_.ConvertToNTPAndSave(receiver_clock_->NowTicks()),
-        NULL, base::TimeDelta(), NULL, &stats);
+    rtcp_for_receiver_.SendRtcpReport(
+        CreateRtcpTimeData(receiver_clock_->NowTicks()), NULL,
+        base::TimeDelta(), NULL, &stats);
     expected_rtt_according_to_sender = one_way_trip_time * 2;
     EXPECT_EQ(expected_rtt_according_to_sender,
               rtcp_for_sender_.current_round_trip_time());
-#ifdef SENDER_PROVIDES_REPORT_BLOCK
-    EXPECT_EQ(expected_rtt_according_to_receiver,
-              rtcp_for_receiver_.current_round_trip_time());
-#endif
-
-    // In the next iteration of this loop, after the receiver gets the sender
-    // report, it will be measuring a round trip time consisting of two
-    // different one-way trip times.
-    expected_rtt_according_to_receiver =
-        (one_way_trip_time + one_way_trip_time * 2) / 2;
   }
 }
 
 TEST_F(RtcpTest, RejectOldRtcpPacket) {
   EXPECT_CALL(mock_frame_sender_, OnReceivedCastFeedback(_))
       .Times(1);
 
   // This is rejected.
   RtcpCastMessage cast_message(kSenderSsrc);
   cast_message.ack_frame_id = 1;
   receiver_to_sender_.Pause();
-  rtcp_for_receiver_.SendRtcpFromRtpReceiver(
-      rtcp_for_receiver_.ConvertToNTPAndSave(
-          receiver_clock_->NowTicks() - base::TimeDelta::FromSeconds(10)),
+  rtcp_for_receiver_.SendRtcpReport(
+      CreateRtcpTimeData(receiver_clock_->NowTicks() -
+                         base::TimeDelta::FromSeconds(10)),
       &cast_message, base::TimeDelta(), NULL, NULL);
 
   cast_message.ack_frame_id = 2;
-  rtcp_for_receiver_.SendRtcpFromRtpReceiver(
-      rtcp_for_receiver_.ConvertToNTPAndSave(receiver_clock_->NowTicks()),
-      &cast_message, base::TimeDelta(), NULL, NULL);
+  rtcp_for_receiver_.SendRtcpReport(
+      CreateRtcpTimeData(receiver_clock_->NowTicks()), &cast_message,
+      base::TimeDelta(), NULL, NULL);
 
   receiver_to_sender_.ReversePacketQueue();
   receiver_to_sender_.Unpause();
 }
 
 TEST_F(RtcpTest, NegativeTimeTicks) {
   EXPECT_CALL(mock_frame_sender_, OnReceivedCastFeedback(_))
       .Times(2);
 
   // Send a RRTR with NTP timestamp that translates to a very negative
   // value for TimeTicks.
   RtcpCastMessage cast_message(kSenderSsrc);
   cast_message.ack_frame_id = 2;
-  rtcp_for_receiver_.SendRtcpFromRtpReceiver(
-      rtcp_for_receiver_.ConvertToNTPAndSave(
-          base::TimeTicks() - base::TimeDelta::FromSeconds(5)),
+  rtcp_for_receiver_.SendRtcpReport(
+      CreateRtcpTimeData(base::TimeTicks() - base::TimeDelta::FromSeconds(5)),
       &cast_message, base::TimeDelta(), NULL, NULL);
 
   cast_message.ack_frame_id = 1;
-  rtcp_for_receiver_.SendRtcpFromRtpReceiver(
-      rtcp_for_receiver_.ConvertToNTPAndSave(base::TimeTicks()),
-      &cast_message, base::TimeDelta(), NULL, NULL);
+  rtcp_for_receiver_.SendRtcpReport(CreateRtcpTimeData(base::TimeTicks()),
+                                    &cast_message, base::TimeDelta(), NULL,
+                                    NULL);
 }
 
 // TODO(miu): Find a better home for this test.
 TEST(MisplacedCastTest, NtpAndTime) {
   const int64 kSecondsbetweenYear1900and2010 = INT64_C(40176 * 24 * 60 * 60);
   const int64 kSecondsbetweenYear1900and2030 = INT64_C(47481 * 24 * 60 * 60);
 
   uint32 ntp_seconds_1 = 0;
   uint32 ntp_fraction_1 = 0;
   base::TimeTicks input_time = base::TimeTicks::Now();
@@ skipping 31 matching lines @@
   base::TimeTicks out_3 = ConvertNtpToTimeTicks(ntp_seconds_3, ntp_fraction_3);
   EXPECT_EQ(input_time, out_3);  // Verify inverse.
 
   // Verify delta.
   EXPECT_EQ((out_3 - out_2), time_delta);
   EXPECT_NEAR((ntp_fraction_3 - ntp_fraction_2), 0xffffffff / 2, 1);
 }
 
 }  // namespace cast
 }  // namespace media