Stefan Holmer | e590416 | 2015-03-26 11:11:06 +0100 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. |
| 3 | * |
| 4 | * Use of this source code is governed by a BSD-style license |
| 5 | * that can be found in the LICENSE file in the root of the source |
| 6 | * tree. An additional intellectual property rights grant can be found |
| 7 | * in the file PATENTS. All contributing project authors may |
| 8 | * be found in the AUTHORS file in the root of the source tree. |
| 9 | */ |
| 10 | |
Jonas Olsson | a4d8737 | 2019-07-05 19:08:33 +0200 | [diff] [blame] | 11 | #include "modules/pacing/packet_router.h" |
| 12 | |
Yves Gerey | 3e70781 | 2018-11-28 16:47:49 +0100 | [diff] [blame] | 13 | #include <cstddef> |
| 14 | #include <cstdint> |
Mirko Bonadei | 317a1f0 | 2019-09-17 17:06:18 +0200 | [diff] [blame] | 15 | #include <memory> |
Erik Språng | f6468d2 | 2019-07-05 16:53:43 +0200 | [diff] [blame] | 16 | #include <utility> |
Stefan Holmer | e590416 | 2015-03-26 11:11:06 +0100 | [diff] [blame] | 17 | |
Yves Gerey | 3e70781 | 2018-11-28 16:47:49 +0100 | [diff] [blame] | 18 | #include "api/units/time_delta.h" |
Erik Språng | f6468d2 | 2019-07-05 16:53:43 +0200 | [diff] [blame] | 19 | #include "modules/rtp_rtcp/include/rtp_header_extension_map.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 20 | #include "modules/rtp_rtcp/mocks/mock_rtp_rtcp.h" |
| 21 | #include "modules/rtp_rtcp/source/rtcp_packet/transport_feedback.h" |
| 22 | #include "rtc_base/checks.h" |
Steve Anton | 10542f2 | 2019-01-11 09:11:00 -0800 | [diff] [blame] | 23 | #include "rtc_base/fake_clock.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 24 | #include "test/gmock.h" |
| 25 | #include "test/gtest.h" |
Stefan Holmer | e590416 | 2015-03-26 11:11:06 +0100 | [diff] [blame] | 26 | |
Stefan Holmer | e590416 | 2015-03-26 11:11:06 +0100 | [diff] [blame] | 27 | namespace webrtc { |
| 28 | |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 29 | // TODO(eladalon): Restructure and/or replace the existing monolithic tests |
| 30 | // (only some of the test are monolithic) according to the new |
| 31 | // guidelines - small tests for one thing at a time. |
| 32 | // (I'm not removing any tests during CL, so as to demonstrate no regressions.) |
| 33 | |
eladalon | b1338fe | 2017-08-01 09:36:19 -0700 | [diff] [blame] | 34 | namespace { |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 35 | |
| 36 | using ::testing::_; |
| 37 | using ::testing::AnyNumber; |
| 38 | using ::testing::AtLeast; |
| 39 | using ::testing::Field; |
| 40 | using ::testing::Gt; |
| 41 | using ::testing::Le; |
| 42 | using ::testing::NiceMock; |
Erik Språng | f6468d2 | 2019-07-05 16:53:43 +0200 | [diff] [blame] | 43 | using ::testing::Property; |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 44 | using ::testing::Return; |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 45 | using ::testing::SaveArg; |
| 46 | |
eladalon | b1338fe | 2017-08-01 09:36:19 -0700 | [diff] [blame] | 47 | constexpr int kProbeMinProbes = 5; |
| 48 | constexpr int kProbeMinBytes = 1000; |
| 49 | |
eladalon | b1338fe | 2017-08-01 09:36:19 -0700 | [diff] [blame] | 50 | } // namespace |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 51 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 52 | class PacketRouterTest : public ::testing::Test { |
| 53 | public: |
| 54 | PacketRouterTest() { |
Danil Chapovalov | a74e477 | 2019-09-12 17:53:04 +0200 | [diff] [blame] | 55 | extension_manager.Register<TransportSequenceNumber>(/*id=*/1); |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 56 | } |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 57 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 58 | protected: |
| 59 | std::unique_ptr<RtpPacketToSend> BuildRtpPacket(uint32_t ssrc) { |
| 60 | std::unique_ptr<RtpPacketToSend> packet = |
Mirko Bonadei | 317a1f0 | 2019-09-17 17:06:18 +0200 | [diff] [blame] | 61 | std::make_unique<RtpPacketToSend>(&extension_manager); |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 62 | packet->SetSsrc(ssrc); |
| 63 | return packet; |
| 64 | } |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 65 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 66 | PacketRouter packet_router_; |
| 67 | RtpHeaderExtensionMap extension_manager; |
| 68 | }; |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 69 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 70 | TEST_F(PacketRouterTest, Sanity_NoModuleRegistered_GeneratePadding) { |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 71 | constexpr size_t bytes = 300; |
| 72 | const PacedPacketInfo paced_info(1, kProbeMinProbes, kProbeMinBytes); |
| 73 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 74 | EXPECT_TRUE(packet_router_.GeneratePadding(bytes).empty()); |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 75 | } |
| 76 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 77 | TEST_F(PacketRouterTest, Sanity_NoModuleRegistered_OnReceiveBitrateChanged) { |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 78 | const std::vector<uint32_t> ssrcs = {1, 2, 3}; |
| 79 | constexpr uint32_t bitrate_bps = 10000; |
| 80 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 81 | packet_router_.OnReceiveBitrateChanged(ssrcs, bitrate_bps); |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 82 | } |
| 83 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 84 | TEST_F(PacketRouterTest, Sanity_NoModuleRegistered_SendRemb) { |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 85 | const std::vector<uint32_t> ssrcs = {1, 2, 3}; |
| 86 | constexpr uint32_t bitrate_bps = 10000; |
| 87 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 88 | EXPECT_FALSE(packet_router_.SendRemb(bitrate_bps, ssrcs)); |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 89 | } |
| 90 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 91 | TEST_F(PacketRouterTest, Sanity_NoModuleRegistered_SendTransportFeedback) { |
Per Kjellander | ee153c9 | 2019-10-10 16:43:46 +0200 | [diff] [blame^] | 92 | std::vector<std::unique_ptr<rtcp::RtcpPacket>> feedback; |
| 93 | feedback.push_back(std::make_unique<rtcp::TransportFeedback>()); |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 94 | |
Per Kjellander | ee153c9 | 2019-10-10 16:43:46 +0200 | [diff] [blame^] | 95 | EXPECT_FALSE(packet_router_.SendCombinedRtcpPacket(std::move(feedback))); |
eladalon | 32040ef | 2017-08-02 06:29:00 -0700 | [diff] [blame] | 96 | } |
| 97 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 98 | TEST_F(PacketRouterTest, GeneratePaddingPicksCorrectModule) { |
Erik Språng | 478cb46 | 2019-06-26 15:49:27 +0200 | [diff] [blame] | 99 | // Two RTP modules. The first (prioritized due to rtx) isn't sending media so |
| 100 | // should not be called. |
| 101 | const uint16_t kSsrc1 = 1234; |
| 102 | const uint16_t kSsrc2 = 4567; |
| 103 | |
| 104 | NiceMock<MockRtpRtcp> rtp_1; |
| 105 | ON_CALL(rtp_1, RtxSendStatus()).WillByDefault(Return(kRtxRedundantPayloads)); |
| 106 | ON_CALL(rtp_1, SSRC()).WillByDefault(Return(kSsrc1)); |
Mirko Bonadei | 999a72a | 2019-07-12 17:33:46 +0000 | [diff] [blame] | 107 | ON_CALL(rtp_1, SupportsPadding).WillByDefault(Return(false)); |
Erik Språng | 478cb46 | 2019-06-26 15:49:27 +0200 | [diff] [blame] | 108 | |
| 109 | NiceMock<MockRtpRtcp> rtp_2; |
| 110 | ON_CALL(rtp_2, RtxSendStatus()).WillByDefault(Return(kRtxOff)); |
| 111 | ON_CALL(rtp_2, SSRC()).WillByDefault(Return(kSsrc2)); |
Mirko Bonadei | 999a72a | 2019-07-12 17:33:46 +0000 | [diff] [blame] | 112 | ON_CALL(rtp_2, SupportsPadding).WillByDefault(Return(true)); |
Erik Språng | 478cb46 | 2019-06-26 15:49:27 +0200 | [diff] [blame] | 113 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 114 | packet_router_.AddSendRtpModule(&rtp_1, false); |
| 115 | packet_router_.AddSendRtpModule(&rtp_2, false); |
Erik Språng | 478cb46 | 2019-06-26 15:49:27 +0200 | [diff] [blame] | 116 | |
| 117 | const size_t kPaddingSize = 123; |
Erik Språng | f6468d2 | 2019-07-05 16:53:43 +0200 | [diff] [blame] | 118 | const size_t kExpectedPaddingPackets = 1; |
Erik Språng | 478cb46 | 2019-06-26 15:49:27 +0200 | [diff] [blame] | 119 | EXPECT_CALL(rtp_1, GeneratePadding(_)).Times(0); |
Erik Språng | f6468d2 | 2019-07-05 16:53:43 +0200 | [diff] [blame] | 120 | EXPECT_CALL(rtp_2, GeneratePadding(kPaddingSize)) |
| 121 | .WillOnce([&](size_t padding_size) { |
| 122 | return std::vector<std::unique_ptr<RtpPacketToSend>>( |
| 123 | kExpectedPaddingPackets); |
| 124 | }); |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 125 | auto generated_padding = packet_router_.GeneratePadding(kPaddingSize); |
Erik Språng | f6468d2 | 2019-07-05 16:53:43 +0200 | [diff] [blame] | 126 | EXPECT_EQ(generated_padding.size(), kExpectedPaddingPackets); |
Erik Språng | 478cb46 | 2019-06-26 15:49:27 +0200 | [diff] [blame] | 127 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 128 | packet_router_.RemoveSendRtpModule(&rtp_1); |
| 129 | packet_router_.RemoveSendRtpModule(&rtp_2); |
Erik Språng | 478cb46 | 2019-06-26 15:49:27 +0200 | [diff] [blame] | 130 | } |
| 131 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 132 | TEST_F(PacketRouterTest, PadsOnLastActiveMediaStream) { |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 133 | const uint16_t kSsrc1 = 1234; |
| 134 | const uint16_t kSsrc2 = 4567; |
| 135 | const uint16_t kSsrc3 = 8901; |
| 136 | |
| 137 | // First two rtp modules send media and have rtx. |
| 138 | NiceMock<MockRtpRtcp> rtp_1; |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 139 | EXPECT_CALL(rtp_1, SSRC()).WillRepeatedly(Return(kSsrc1)); |
Mirko Bonadei | 999a72a | 2019-07-12 17:33:46 +0000 | [diff] [blame] | 140 | EXPECT_CALL(rtp_1, SupportsPadding).WillRepeatedly(Return(true)); |
| 141 | EXPECT_CALL(rtp_1, SupportsRtxPayloadPadding).WillRepeatedly(Return(true)); |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 142 | EXPECT_CALL(rtp_1, TrySendPacket).WillRepeatedly(Return(false)); |
| 143 | EXPECT_CALL( |
| 144 | rtp_1, |
| 145 | TrySendPacket( |
| 146 | ::testing::Pointee(Property(&RtpPacketToSend::Ssrc, kSsrc1)), _)) |
| 147 | .WillRepeatedly(Return(true)); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 148 | |
| 149 | NiceMock<MockRtpRtcp> rtp_2; |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 150 | EXPECT_CALL(rtp_2, SSRC()).WillRepeatedly(Return(kSsrc2)); |
Mirko Bonadei | 999a72a | 2019-07-12 17:33:46 +0000 | [diff] [blame] | 151 | EXPECT_CALL(rtp_2, SupportsPadding).WillRepeatedly(Return(true)); |
| 152 | EXPECT_CALL(rtp_2, SupportsRtxPayloadPadding).WillRepeatedly(Return(true)); |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 153 | EXPECT_CALL(rtp_2, TrySendPacket).WillRepeatedly(Return(false)); |
| 154 | EXPECT_CALL( |
| 155 | rtp_2, |
| 156 | TrySendPacket( |
| 157 | ::testing::Pointee(Property(&RtpPacketToSend::Ssrc, kSsrc2)), _)) |
| 158 | .WillRepeatedly(Return(true)); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 159 | |
| 160 | // Third module is sending media, but does not support rtx. |
| 161 | NiceMock<MockRtpRtcp> rtp_3; |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 162 | EXPECT_CALL(rtp_3, SSRC()).WillRepeatedly(Return(kSsrc3)); |
Mirko Bonadei | 999a72a | 2019-07-12 17:33:46 +0000 | [diff] [blame] | 163 | EXPECT_CALL(rtp_3, SupportsPadding).WillRepeatedly(Return(true)); |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 164 | EXPECT_CALL(rtp_3, SupportsRtxPayloadPadding).WillRepeatedly(Return(false)); |
| 165 | EXPECT_CALL(rtp_3, TrySendPacket).WillRepeatedly(Return(false)); |
| 166 | EXPECT_CALL( |
| 167 | rtp_3, |
| 168 | TrySendPacket( |
| 169 | ::testing::Pointee(Property(&RtpPacketToSend::Ssrc, kSsrc3)), _)) |
| 170 | .WillRepeatedly(Return(true)); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 171 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 172 | packet_router_.AddSendRtpModule(&rtp_1, false); |
| 173 | packet_router_.AddSendRtpModule(&rtp_2, false); |
| 174 | packet_router_.AddSendRtpModule(&rtp_3, false); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 175 | |
| 176 | const size_t kPaddingBytes = 100; |
| 177 | |
| 178 | // Initially, padding will be sent on last added rtp module that sends media |
| 179 | // and supports rtx. |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 180 | EXPECT_CALL(rtp_2, GeneratePadding(kPaddingBytes)) |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 181 | .Times(1) |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 182 | .WillOnce([](size_t target_size_bytes) { |
| 183 | return std::vector<std::unique_ptr<RtpPacketToSend>>(); |
| 184 | }); |
| 185 | packet_router_.GeneratePadding(kPaddingBytes); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 186 | |
| 187 | // Send media on first module. Padding should be sent on that module. |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 188 | packet_router_.SendPacket(BuildRtpPacket(kSsrc1), PacedPacketInfo()); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 189 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 190 | EXPECT_CALL(rtp_1, GeneratePadding(kPaddingBytes)) |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 191 | .Times(1) |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 192 | .WillOnce([](size_t target_size_bytes) { |
| 193 | return std::vector<std::unique_ptr<RtpPacketToSend>>(); |
| 194 | }); |
| 195 | packet_router_.GeneratePadding(kPaddingBytes); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 196 | |
| 197 | // Send media on second module. Padding should be sent there. |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 198 | packet_router_.SendPacket(BuildRtpPacket(kSsrc2), PacedPacketInfo()); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 199 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 200 | EXPECT_CALL(rtp_2, GeneratePadding(kPaddingBytes)) |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 201 | .Times(1) |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 202 | .WillOnce([](size_t target_size_bytes) { |
| 203 | return std::vector<std::unique_ptr<RtpPacketToSend>>(); |
| 204 | }); |
| 205 | packet_router_.GeneratePadding(kPaddingBytes); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 206 | |
| 207 | // Remove second module, padding should now fall back to first module. |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 208 | packet_router_.RemoveSendRtpModule(&rtp_2); |
| 209 | EXPECT_CALL(rtp_1, GeneratePadding(kPaddingBytes)) |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 210 | .Times(1) |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 211 | .WillOnce([](size_t target_size_bytes) { |
| 212 | return std::vector<std::unique_ptr<RtpPacketToSend>>(); |
| 213 | }); |
| 214 | packet_router_.GeneratePadding(kPaddingBytes); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 215 | |
| 216 | // Remove first module too, leaving only the one without rtx. |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 217 | packet_router_.RemoveSendRtpModule(&rtp_1); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 218 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 219 | EXPECT_CALL(rtp_3, GeneratePadding(kPaddingBytes)) |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 220 | .Times(1) |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 221 | .WillOnce([](size_t target_size_bytes) { |
| 222 | return std::vector<std::unique_ptr<RtpPacketToSend>>(); |
| 223 | }); |
| 224 | packet_router_.GeneratePadding(kPaddingBytes); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 225 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 226 | packet_router_.RemoveSendRtpModule(&rtp_3); |
Erik Språng | 8b7ca4a | 2018-05-17 13:43:35 +0200 | [diff] [blame] | 227 | } |
| 228 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 229 | TEST_F(PacketRouterTest, AllocateSequenceNumbers) { |
sprang | 867fb52 | 2015-08-03 04:38:41 -0700 | [diff] [blame] | 230 | const uint16_t kStartSeq = 0xFFF0; |
| 231 | const size_t kNumPackets = 32; |
| 232 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 233 | packet_router_.SetTransportWideSequenceNumber(kStartSeq - 1); |
sprang | 867fb52 | 2015-08-03 04:38:41 -0700 | [diff] [blame] | 234 | |
| 235 | for (size_t i = 0; i < kNumPackets; ++i) { |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 236 | uint16_t seq = packet_router_.AllocateSequenceNumber(); |
sprang | 867fb52 | 2015-08-03 04:38:41 -0700 | [diff] [blame] | 237 | uint32_t expected_unwrapped_seq = static_cast<uint32_t>(kStartSeq) + i; |
| 238 | EXPECT_EQ(static_cast<uint16_t>(expected_unwrapped_seq & 0xFFFF), seq); |
| 239 | } |
| 240 | } |
stefan | bba9dec | 2016-02-01 04:39:55 -0800 | [diff] [blame] | 241 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 242 | TEST_F(PacketRouterTest, SendTransportFeedback) { |
eladalon | 6c9556e | 2017-07-10 03:33:00 -0700 | [diff] [blame] | 243 | NiceMock<MockRtpRtcp> rtp_1; |
| 244 | NiceMock<MockRtpRtcp> rtp_2; |
eladalon | b1338fe | 2017-08-01 09:36:19 -0700 | [diff] [blame] | 245 | |
Per Kjellander | ee153c9 | 2019-10-10 16:43:46 +0200 | [diff] [blame^] | 246 | ON_CALL(rtp_1, RTCP()).WillByDefault(Return(RtcpMode::kCompound)); |
| 247 | ON_CALL(rtp_2, RTCP()).WillByDefault(Return(RtcpMode::kCompound)); |
| 248 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 249 | packet_router_.AddSendRtpModule(&rtp_1, false); |
| 250 | packet_router_.AddReceiveRtpModule(&rtp_2, false); |
stefan | bba9dec | 2016-02-01 04:39:55 -0800 | [diff] [blame] | 251 | |
Per Kjellander | ee153c9 | 2019-10-10 16:43:46 +0200 | [diff] [blame^] | 252 | std::vector<std::unique_ptr<rtcp::RtcpPacket>> feedback; |
| 253 | feedback.push_back(std::make_unique<rtcp::TransportFeedback>()); |
| 254 | EXPECT_CALL(rtp_1, SendCombinedRtcpPacket).Times(1); |
| 255 | packet_router_.SendCombinedRtcpPacket(std::move(feedback)); |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 256 | packet_router_.RemoveSendRtpModule(&rtp_1); |
Per Kjellander | ee153c9 | 2019-10-10 16:43:46 +0200 | [diff] [blame^] | 257 | EXPECT_CALL(rtp_2, SendCombinedRtcpPacket).Times(1); |
| 258 | std::vector<std::unique_ptr<rtcp::RtcpPacket>> new_feedback; |
| 259 | new_feedback.push_back(std::make_unique<rtcp::TransportFeedback>()); |
| 260 | packet_router_.SendCombinedRtcpPacket(std::move(new_feedback)); |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 261 | packet_router_.RemoveReceiveRtpModule(&rtp_2); |
| 262 | } |
| 263 | |
| 264 | TEST_F(PacketRouterTest, SendPacketWithoutTransportSequenceNumbers) { |
| 265 | NiceMock<MockRtpRtcp> rtp_1; |
| 266 | packet_router_.AddSendRtpModule(&rtp_1, false); |
| 267 | |
| 268 | const uint16_t kSsrc1 = 1234; |
| 269 | ON_CALL(rtp_1, SendingMedia).WillByDefault(Return(true)); |
| 270 | ON_CALL(rtp_1, SSRC).WillByDefault(Return(kSsrc1)); |
| 271 | |
| 272 | // Send a packet without TransportSequenceNumber extension registered, |
| 273 | // packets sent should not have the extension set. |
| 274 | RtpHeaderExtensionMap extension_manager; |
Mirko Bonadei | 317a1f0 | 2019-09-17 17:06:18 +0200 | [diff] [blame] | 275 | auto packet = std::make_unique<RtpPacketToSend>(&extension_manager); |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 276 | packet->SetSsrc(kSsrc1); |
| 277 | EXPECT_CALL( |
| 278 | rtp_1, |
| 279 | TrySendPacket( |
| 280 | Property(&RtpPacketToSend::HasExtension<TransportSequenceNumber>, |
| 281 | false), |
| 282 | _)) |
| 283 | .WillOnce(Return(true)); |
| 284 | packet_router_.SendPacket(std::move(packet), PacedPacketInfo()); |
| 285 | |
| 286 | packet_router_.RemoveSendRtpModule(&rtp_1); |
| 287 | } |
| 288 | |
| 289 | TEST_F(PacketRouterTest, SendPacketAssignsTransportSequenceNumbers) { |
| 290 | NiceMock<MockRtpRtcp> rtp_1; |
| 291 | NiceMock<MockRtpRtcp> rtp_2; |
| 292 | |
| 293 | packet_router_.AddSendRtpModule(&rtp_1, false); |
| 294 | packet_router_.AddSendRtpModule(&rtp_2, false); |
| 295 | |
| 296 | const uint16_t kSsrc1 = 1234; |
| 297 | const uint16_t kSsrc2 = 2345; |
| 298 | |
| 299 | ON_CALL(rtp_1, SSRC).WillByDefault(Return(kSsrc1)); |
| 300 | ON_CALL(rtp_2, SSRC).WillByDefault(Return(kSsrc2)); |
| 301 | |
| 302 | // Transport sequence numbers start at 1, for historical reasons. |
| 303 | uint16_t transport_sequence_number = 1; |
| 304 | |
| 305 | auto packet = BuildRtpPacket(kSsrc1); |
| 306 | EXPECT_TRUE(packet->ReserveExtension<TransportSequenceNumber>()); |
| 307 | EXPECT_CALL( |
| 308 | rtp_1, |
| 309 | TrySendPacket( |
| 310 | Property(&RtpPacketToSend::GetExtension<TransportSequenceNumber>, |
| 311 | transport_sequence_number), |
| 312 | _)) |
| 313 | .WillOnce(Return(true)); |
| 314 | packet_router_.SendPacket(std::move(packet), PacedPacketInfo()); |
| 315 | |
| 316 | ++transport_sequence_number; |
| 317 | packet = BuildRtpPacket(kSsrc2); |
| 318 | EXPECT_TRUE(packet->ReserveExtension<TransportSequenceNumber>()); |
| 319 | |
| 320 | // There will be a failed attempt to send on kSsrc1 before trying |
| 321 | // the correct RTP module. |
| 322 | EXPECT_CALL(rtp_1, TrySendPacket).WillOnce(Return(false)); |
| 323 | EXPECT_CALL( |
| 324 | rtp_2, |
| 325 | TrySendPacket( |
| 326 | Property(&RtpPacketToSend::GetExtension<TransportSequenceNumber>, |
| 327 | transport_sequence_number), |
| 328 | _)) |
| 329 | .WillOnce(Return(true)); |
| 330 | packet_router_.SendPacket(std::move(packet), PacedPacketInfo()); |
| 331 | |
| 332 | packet_router_.RemoveSendRtpModule(&rtp_1); |
| 333 | packet_router_.RemoveSendRtpModule(&rtp_2); |
stefan | bba9dec | 2016-02-01 04:39:55 -0800 | [diff] [blame] | 334 | } |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 335 | |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 336 | #if RTC_DCHECK_IS_ON && GTEST_HAS_DEATH_TEST && !defined(WEBRTC_ANDROID) |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 337 | TEST_F(PacketRouterTest, DoubleRegistrationOfSendModuleDisallowed) { |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 338 | NiceMock<MockRtpRtcp> module; |
| 339 | |
| 340 | constexpr bool remb_candidate = false; // Value irrelevant. |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 341 | packet_router_.AddSendRtpModule(&module, remb_candidate); |
| 342 | EXPECT_DEATH(packet_router_.AddSendRtpModule(&module, remb_candidate), ""); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 343 | |
| 344 | // Test tear-down |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 345 | packet_router_.RemoveSendRtpModule(&module); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 346 | } |
| 347 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 348 | TEST_F(PacketRouterTest, DoubleRegistrationOfReceiveModuleDisallowed) { |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 349 | NiceMock<MockRtpRtcp> module; |
| 350 | |
| 351 | constexpr bool remb_candidate = false; // Value irrelevant. |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 352 | packet_router_.AddReceiveRtpModule(&module, remb_candidate); |
| 353 | EXPECT_DEATH(packet_router_.AddReceiveRtpModule(&module, remb_candidate), ""); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 354 | |
| 355 | // Test tear-down |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 356 | packet_router_.RemoveReceiveRtpModule(&module); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 357 | } |
| 358 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 359 | TEST_F(PacketRouterTest, RemovalOfNeverAddedSendModuleDisallowed) { |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 360 | NiceMock<MockRtpRtcp> module; |
| 361 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 362 | EXPECT_DEATH(packet_router_.RemoveSendRtpModule(&module), ""); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 363 | } |
| 364 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 365 | TEST_F(PacketRouterTest, RemovalOfNeverAddedReceiveModuleDisallowed) { |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 366 | NiceMock<MockRtpRtcp> module; |
| 367 | |
Erik Språng | 4208a13 | 2019-08-26 08:58:45 +0200 | [diff] [blame] | 368 | EXPECT_DEATH(packet_router_.RemoveReceiveRtpModule(&module), ""); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 369 | } |
| 370 | #endif // RTC_DCHECK_IS_ON && GTEST_HAS_DEATH_TEST && !defined(WEBRTC_ANDROID) |
| 371 | |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 372 | TEST(PacketRouterRembTest, LowerEstimateToSendRemb) { |
| 373 | rtc::ScopedFakeClock clock; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 374 | NiceMock<MockRtpRtcp> rtp; |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 375 | PacketRouter packet_router; |
| 376 | |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 377 | packet_router.AddSendRtpModule(&rtp, true); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 378 | |
| 379 | uint32_t bitrate_estimate = 456; |
| 380 | const std::vector<uint32_t> ssrcs = {1234}; |
| 381 | |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 382 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 383 | |
| 384 | // Call OnReceiveBitrateChanged twice to get a first estimate. |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 385 | clock.AdvanceTime(TimeDelta::ms(1000)); |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 386 | EXPECT_CALL(rtp, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 387 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 388 | |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 389 | // Lower the estimate with more than 3% to trigger a call to SetRemb right |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 390 | // away. |
| 391 | bitrate_estimate = bitrate_estimate - 100; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 392 | EXPECT_CALL(rtp, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 393 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 394 | |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 395 | packet_router.RemoveSendRtpModule(&rtp); |
| 396 | } |
| 397 | |
| 398 | TEST(PacketRouterRembTest, VerifyIncreasingAndDecreasing) { |
| 399 | rtc::ScopedFakeClock clock; |
eladalon | 6c9556e | 2017-07-10 03:33:00 -0700 | [diff] [blame] | 400 | NiceMock<MockRtpRtcp> rtp; |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 401 | PacketRouter packet_router; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 402 | packet_router.AddSendRtpModule(&rtp, true); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 403 | |
| 404 | uint32_t bitrate_estimate[] = {456, 789}; |
| 405 | std::vector<uint32_t> ssrcs = {1234, 5678}; |
| 406 | |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 407 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate[0]); |
| 408 | |
| 409 | // Call OnReceiveBitrateChanged twice to get a first estimate. |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 410 | EXPECT_CALL(rtp, SetRemb(bitrate_estimate[0], ssrcs)).Times(1); |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 411 | clock.AdvanceTime(TimeDelta::ms(1000)); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 412 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate[0]); |
| 413 | |
| 414 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate[1] + 100); |
| 415 | |
| 416 | // Lower the estimate to trigger a callback. |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 417 | EXPECT_CALL(rtp, SetRemb(bitrate_estimate[1], ssrcs)).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 418 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate[1]); |
| 419 | |
| 420 | packet_router.RemoveSendRtpModule(&rtp); |
| 421 | } |
| 422 | |
| 423 | TEST(PacketRouterRembTest, NoRembForIncreasedBitrate) { |
| 424 | rtc::ScopedFakeClock clock; |
eladalon | 6c9556e | 2017-07-10 03:33:00 -0700 | [diff] [blame] | 425 | NiceMock<MockRtpRtcp> rtp; |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 426 | PacketRouter packet_router; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 427 | packet_router.AddSendRtpModule(&rtp, true); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 428 | |
| 429 | uint32_t bitrate_estimate = 456; |
| 430 | std::vector<uint32_t> ssrcs = {1234, 5678}; |
| 431 | |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 432 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 433 | |
| 434 | // Call OnReceiveBitrateChanged twice to get a first estimate. |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 435 | EXPECT_CALL(rtp, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 436 | clock.AdvanceTime(TimeDelta::ms(1000)); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 437 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 438 | |
| 439 | // Increased estimate shouldn't trigger a callback right away. |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 440 | EXPECT_CALL(rtp, SetRemb(_, _)).Times(0); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 441 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate + 1); |
| 442 | |
| 443 | // Decreasing the estimate less than 3% shouldn't trigger a new callback. |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 444 | EXPECT_CALL(rtp, SetRemb(_, _)).Times(0); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 445 | int lower_estimate = bitrate_estimate * 98 / 100; |
| 446 | packet_router.OnReceiveBitrateChanged(ssrcs, lower_estimate); |
| 447 | |
| 448 | packet_router.RemoveSendRtpModule(&rtp); |
| 449 | } |
| 450 | |
| 451 | TEST(PacketRouterRembTest, ChangeSendRtpModule) { |
| 452 | rtc::ScopedFakeClock clock; |
eladalon | 6c9556e | 2017-07-10 03:33:00 -0700 | [diff] [blame] | 453 | NiceMock<MockRtpRtcp> rtp_send; |
| 454 | NiceMock<MockRtpRtcp> rtp_recv; |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 455 | PacketRouter packet_router; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 456 | packet_router.AddSendRtpModule(&rtp_send, true); |
| 457 | packet_router.AddReceiveRtpModule(&rtp_recv, true); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 458 | |
| 459 | uint32_t bitrate_estimate = 456; |
| 460 | std::vector<uint32_t> ssrcs = {1234, 5678}; |
| 461 | |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 462 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 463 | |
| 464 | // Call OnReceiveBitrateChanged twice to get a first estimate. |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 465 | clock.AdvanceTime(TimeDelta::ms(1000)); |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 466 | EXPECT_CALL(rtp_send, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 467 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 468 | |
| 469 | // Decrease estimate to trigger a REMB. |
| 470 | bitrate_estimate = bitrate_estimate - 100; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 471 | EXPECT_CALL(rtp_send, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 472 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 473 | |
| 474 | // Remove the sending module -> should get remb on the second module. |
| 475 | packet_router.RemoveSendRtpModule(&rtp_send); |
| 476 | |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 477 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 478 | |
| 479 | bitrate_estimate = bitrate_estimate - 100; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 480 | EXPECT_CALL(rtp_recv, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 481 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 482 | |
| 483 | packet_router.RemoveReceiveRtpModule(&rtp_recv); |
| 484 | } |
| 485 | |
| 486 | TEST(PacketRouterRembTest, OnlyOneRembForRepeatedOnReceiveBitrateChanged) { |
| 487 | rtc::ScopedFakeClock clock; |
eladalon | 6c9556e | 2017-07-10 03:33:00 -0700 | [diff] [blame] | 488 | NiceMock<MockRtpRtcp> rtp; |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 489 | PacketRouter packet_router; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 490 | packet_router.AddSendRtpModule(&rtp, true); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 491 | |
| 492 | uint32_t bitrate_estimate = 456; |
| 493 | const std::vector<uint32_t> ssrcs = {1234}; |
| 494 | |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 495 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 496 | |
| 497 | // Call OnReceiveBitrateChanged twice to get a first estimate. |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 498 | clock.AdvanceTime(TimeDelta::ms(1000)); |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 499 | EXPECT_CALL(rtp, SetRemb(_, _)).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 500 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 501 | |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 502 | // Lower the estimate, should trigger a call to SetRemb right away. |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 503 | bitrate_estimate = bitrate_estimate - 100; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 504 | EXPECT_CALL(rtp, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 505 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 506 | |
| 507 | // Call OnReceiveBitrateChanged again, this should not trigger a new callback. |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 508 | EXPECT_CALL(rtp, SetRemb(_, _)).Times(0); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 509 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 510 | packet_router.RemoveSendRtpModule(&rtp); |
| 511 | } |
| 512 | |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 513 | TEST(PacketRouterRembTest, SetMaxDesiredReceiveBitrateLimitsSetRemb) { |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 514 | rtc::ScopedFakeClock clock; |
| 515 | PacketRouter packet_router; |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 516 | clock.AdvanceTime(TimeDelta::ms(1000)); |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 517 | NiceMock<MockRtpRtcp> remb_sender; |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 518 | constexpr bool remb_candidate = true; |
| 519 | packet_router.AddSendRtpModule(&remb_sender, remb_candidate); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 520 | |
Danil Chapovalov | 1de4b62 | 2017-12-13 13:35:10 +0100 | [diff] [blame] | 521 | const int64_t cap_bitrate = 100000; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 522 | EXPECT_CALL(remb_sender, SetRemb(Le(cap_bitrate), _)).Times(AtLeast(1)); |
| 523 | EXPECT_CALL(remb_sender, SetRemb(Gt(cap_bitrate), _)).Times(0); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 524 | |
| 525 | const std::vector<uint32_t> ssrcs = {1234}; |
| 526 | packet_router.SetMaxDesiredReceiveBitrate(cap_bitrate); |
| 527 | packet_router.OnReceiveBitrateChanged(ssrcs, cap_bitrate + 5000); |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 528 | clock.AdvanceTime(TimeDelta::ms(1000)); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 529 | packet_router.OnReceiveBitrateChanged(ssrcs, cap_bitrate - 5000); |
| 530 | |
| 531 | // Test tear-down. |
| 532 | packet_router.RemoveSendRtpModule(&remb_sender); |
| 533 | } |
| 534 | |
| 535 | TEST(PacketRouterRembTest, |
| 536 | SetMaxDesiredReceiveBitrateTriggersRembWhenMoreRestrictive) { |
| 537 | rtc::ScopedFakeClock clock; |
| 538 | PacketRouter packet_router; |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 539 | clock.AdvanceTime(TimeDelta::ms(1000)); |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 540 | NiceMock<MockRtpRtcp> remb_sender; |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 541 | constexpr bool remb_candidate = true; |
| 542 | packet_router.AddSendRtpModule(&remb_sender, remb_candidate); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 543 | |
Danil Chapovalov | 1de4b62 | 2017-12-13 13:35:10 +0100 | [diff] [blame] | 544 | const int64_t measured_bitrate_bps = 150000; |
| 545 | const int64_t cap_bitrate_bps = measured_bitrate_bps - 5000; |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 546 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 547 | EXPECT_CALL(remb_sender, SetRemb(measured_bitrate_bps, _)); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 548 | packet_router.OnReceiveBitrateChanged(ssrcs, measured_bitrate_bps); |
| 549 | |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 550 | EXPECT_CALL(remb_sender, SetRemb(cap_bitrate_bps, _)); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 551 | packet_router.SetMaxDesiredReceiveBitrate(cap_bitrate_bps); |
| 552 | |
| 553 | // Test tear-down. |
| 554 | packet_router.RemoveSendRtpModule(&remb_sender); |
| 555 | } |
| 556 | |
| 557 | TEST(PacketRouterRembTest, |
| 558 | SetMaxDesiredReceiveBitrateDoesNotTriggerRembWhenAsRestrictive) { |
| 559 | rtc::ScopedFakeClock clock; |
| 560 | PacketRouter packet_router; |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 561 | clock.AdvanceTime(TimeDelta::ms(1000)); |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 562 | NiceMock<MockRtpRtcp> remb_sender; |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 563 | constexpr bool remb_candidate = true; |
| 564 | packet_router.AddSendRtpModule(&remb_sender, remb_candidate); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 565 | |
| 566 | const uint32_t measured_bitrate_bps = 150000; |
| 567 | const uint32_t cap_bitrate_bps = measured_bitrate_bps; |
| 568 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 569 | EXPECT_CALL(remb_sender, SetRemb(measured_bitrate_bps, _)); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 570 | packet_router.OnReceiveBitrateChanged(ssrcs, measured_bitrate_bps); |
| 571 | |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 572 | EXPECT_CALL(remb_sender, SetRemb(_, _)).Times(0); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 573 | packet_router.SetMaxDesiredReceiveBitrate(cap_bitrate_bps); |
| 574 | |
| 575 | // Test tear-down. |
| 576 | packet_router.RemoveSendRtpModule(&remb_sender); |
| 577 | } |
| 578 | |
| 579 | TEST(PacketRouterRembTest, |
| 580 | SetMaxDesiredReceiveBitrateDoesNotTriggerRembWhenLessRestrictive) { |
| 581 | rtc::ScopedFakeClock clock; |
| 582 | PacketRouter packet_router; |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 583 | clock.AdvanceTime(TimeDelta::ms(1000)); |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 584 | NiceMock<MockRtpRtcp> remb_sender; |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 585 | constexpr bool remb_candidate = true; |
| 586 | packet_router.AddSendRtpModule(&remb_sender, remb_candidate); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 587 | |
| 588 | const uint32_t measured_bitrate_bps = 150000; |
| 589 | const uint32_t cap_bitrate_bps = measured_bitrate_bps + 500; |
| 590 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 591 | EXPECT_CALL(remb_sender, SetRemb(measured_bitrate_bps, _)); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 592 | packet_router.OnReceiveBitrateChanged(ssrcs, measured_bitrate_bps); |
| 593 | |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 594 | EXPECT_CALL(remb_sender, SetRemb(_, _)).Times(0); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 595 | packet_router.SetMaxDesiredReceiveBitrate(cap_bitrate_bps); |
| 596 | |
| 597 | // Test tear-down. |
| 598 | packet_router.RemoveSendRtpModule(&remb_sender); |
| 599 | } |
| 600 | |
| 601 | TEST(PacketRouterRembTest, |
| 602 | SetMaxDesiredReceiveBitrateTriggersRembWhenNoRecentMeasure) { |
| 603 | rtc::ScopedFakeClock clock; |
| 604 | PacketRouter packet_router; |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 605 | clock.AdvanceTime(TimeDelta::ms(1000)); |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 606 | NiceMock<MockRtpRtcp> remb_sender; |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 607 | constexpr bool remb_candidate = true; |
| 608 | packet_router.AddSendRtpModule(&remb_sender, remb_candidate); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 609 | |
| 610 | const uint32_t measured_bitrate_bps = 150000; |
| 611 | const uint32_t cap_bitrate_bps = measured_bitrate_bps + 5000; |
| 612 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 613 | EXPECT_CALL(remb_sender, SetRemb(measured_bitrate_bps, _)); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 614 | packet_router.OnReceiveBitrateChanged(ssrcs, measured_bitrate_bps); |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 615 | clock.AdvanceTime(TimeDelta::ms(1000)); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 616 | |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 617 | EXPECT_CALL(remb_sender, SetRemb(cap_bitrate_bps, _)); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 618 | packet_router.SetMaxDesiredReceiveBitrate(cap_bitrate_bps); |
| 619 | |
| 620 | // Test tear-down. |
| 621 | packet_router.RemoveSendRtpModule(&remb_sender); |
| 622 | } |
| 623 | |
| 624 | TEST(PacketRouterRembTest, |
| 625 | SetMaxDesiredReceiveBitrateTriggersRembWhenNoMeasures) { |
| 626 | rtc::ScopedFakeClock clock; |
| 627 | PacketRouter packet_router; |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 628 | clock.AdvanceTime(TimeDelta::ms(1000)); |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 629 | NiceMock<MockRtpRtcp> remb_sender; |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 630 | constexpr bool remb_candidate = true; |
| 631 | packet_router.AddSendRtpModule(&remb_sender, remb_candidate); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 632 | |
| 633 | // Set cap. |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 634 | EXPECT_CALL(remb_sender, SetRemb(100000, _)).Times(1); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 635 | packet_router.SetMaxDesiredReceiveBitrate(100000); |
| 636 | // Increase cap. |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 637 | EXPECT_CALL(remb_sender, SetRemb(200000, _)).Times(1); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 638 | packet_router.SetMaxDesiredReceiveBitrate(200000); |
| 639 | // Decrease cap. |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 640 | EXPECT_CALL(remb_sender, SetRemb(150000, _)).Times(1); |
danilchap | 4708537 | 2017-08-10 06:03:57 -0700 | [diff] [blame] | 641 | packet_router.SetMaxDesiredReceiveBitrate(150000); |
| 642 | |
| 643 | // Test tear-down. |
| 644 | packet_router.RemoveSendRtpModule(&remb_sender); |
| 645 | } |
| 646 | |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 647 | // Only register receiving modules and make sure we fallback to trigger a REMB |
| 648 | // packet on this one. |
| 649 | TEST(PacketRouterRembTest, NoSendingRtpModule) { |
| 650 | rtc::ScopedFakeClock clock; |
eladalon | 6c9556e | 2017-07-10 03:33:00 -0700 | [diff] [blame] | 651 | NiceMock<MockRtpRtcp> rtp; |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 652 | PacketRouter packet_router; |
| 653 | |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 654 | packet_router.AddReceiveRtpModule(&rtp, true); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 655 | |
| 656 | uint32_t bitrate_estimate = 456; |
| 657 | const std::vector<uint32_t> ssrcs = {1234}; |
| 658 | |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 659 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 660 | |
| 661 | // Call OnReceiveBitrateChanged twice to get a first estimate. |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 662 | clock.AdvanceTime(TimeDelta::ms(1000)); |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 663 | EXPECT_CALL(rtp, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 664 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 665 | |
| 666 | // Lower the estimate to trigger a new packet REMB packet. |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 667 | EXPECT_CALL(rtp, SetRemb(bitrate_estimate - 100, ssrcs)).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 668 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate - 100); |
| 669 | |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 670 | EXPECT_CALL(rtp, UnsetRemb()).Times(1); |
nisse | 0584331 | 2017-04-18 23:38:35 -0700 | [diff] [blame] | 671 | packet_router.RemoveReceiveRtpModule(&rtp); |
| 672 | } |
| 673 | |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 674 | TEST(PacketRouterRembTest, NonCandidateSendRtpModuleNotUsedForRemb) { |
| 675 | rtc::ScopedFakeClock clock; |
| 676 | PacketRouter packet_router; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 677 | NiceMock<MockRtpRtcp> module; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 678 | |
| 679 | constexpr bool remb_candidate = false; |
| 680 | |
| 681 | packet_router.AddSendRtpModule(&module, remb_candidate); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 682 | |
| 683 | constexpr uint32_t bitrate_estimate = 456; |
| 684 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 685 | EXPECT_CALL(module, SetRemb(_, _)).Times(0); |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 686 | clock.AdvanceTime(TimeDelta::ms(1000)); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 687 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 688 | |
| 689 | // Test tear-down |
| 690 | packet_router.RemoveSendRtpModule(&module); |
| 691 | } |
| 692 | |
| 693 | TEST(PacketRouterRembTest, CandidateSendRtpModuleUsedForRemb) { |
| 694 | rtc::ScopedFakeClock clock; |
| 695 | PacketRouter packet_router; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 696 | NiceMock<MockRtpRtcp> module; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 697 | |
| 698 | constexpr bool remb_candidate = true; |
| 699 | |
| 700 | packet_router.AddSendRtpModule(&module, remb_candidate); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 701 | |
| 702 | constexpr uint32_t bitrate_estimate = 456; |
| 703 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 704 | EXPECT_CALL(module, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 705 | clock.AdvanceTime(TimeDelta::ms(1000)); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 706 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 707 | |
| 708 | // Test tear-down |
| 709 | packet_router.RemoveSendRtpModule(&module); |
| 710 | } |
| 711 | |
| 712 | TEST(PacketRouterRembTest, NonCandidateReceiveRtpModuleNotUsedForRemb) { |
| 713 | rtc::ScopedFakeClock clock; |
| 714 | PacketRouter packet_router; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 715 | NiceMock<MockRtpRtcp> module; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 716 | |
| 717 | constexpr bool remb_candidate = false; |
| 718 | |
| 719 | packet_router.AddReceiveRtpModule(&module, remb_candidate); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 720 | |
| 721 | constexpr uint32_t bitrate_estimate = 456; |
| 722 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 723 | EXPECT_CALL(module, SetRemb(_, _)).Times(0); |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 724 | clock.AdvanceTime(TimeDelta::ms(1000)); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 725 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 726 | |
| 727 | // Test tear-down |
| 728 | packet_router.RemoveReceiveRtpModule(&module); |
| 729 | } |
| 730 | |
| 731 | TEST(PacketRouterRembTest, CandidateReceiveRtpModuleUsedForRemb) { |
| 732 | rtc::ScopedFakeClock clock; |
| 733 | PacketRouter packet_router; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 734 | NiceMock<MockRtpRtcp> module; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 735 | |
| 736 | constexpr bool remb_candidate = true; |
| 737 | |
| 738 | packet_router.AddReceiveRtpModule(&module, remb_candidate); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 739 | |
| 740 | constexpr uint32_t bitrate_estimate = 456; |
| 741 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 742 | EXPECT_CALL(module, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 743 | clock.AdvanceTime(TimeDelta::ms(1000)); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 744 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 745 | |
| 746 | // Test tear-down |
| 747 | packet_router.RemoveReceiveRtpModule(&module); |
| 748 | } |
| 749 | |
| 750 | TEST(PacketRouterRembTest, |
| 751 | SendCandidatePreferredOverReceiveCandidate_SendModuleAddedFirst) { |
| 752 | rtc::ScopedFakeClock clock; |
| 753 | PacketRouter packet_router; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 754 | NiceMock<MockRtpRtcp> send_module; |
| 755 | NiceMock<MockRtpRtcp> receive_module; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 756 | |
| 757 | constexpr bool remb_candidate = true; |
| 758 | |
| 759 | // Send module added - activated. |
| 760 | packet_router.AddSendRtpModule(&send_module, remb_candidate); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 761 | |
| 762 | // Receive module added - the send module remains the active one. |
| 763 | packet_router.AddReceiveRtpModule(&receive_module, remb_candidate); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 764 | |
| 765 | constexpr uint32_t bitrate_estimate = 456; |
| 766 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 767 | EXPECT_CALL(send_module, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
| 768 | EXPECT_CALL(receive_module, SetRemb(_, _)).Times(0); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 769 | |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 770 | clock.AdvanceTime(TimeDelta::ms(1000)); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 771 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 772 | |
| 773 | // Test tear-down |
| 774 | packet_router.RemoveReceiveRtpModule(&receive_module); |
| 775 | packet_router.RemoveSendRtpModule(&send_module); |
| 776 | } |
| 777 | |
| 778 | TEST(PacketRouterRembTest, |
| 779 | SendCandidatePreferredOverReceiveCandidate_ReceiveModuleAddedFirst) { |
| 780 | rtc::ScopedFakeClock clock; |
| 781 | PacketRouter packet_router; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 782 | NiceMock<MockRtpRtcp> send_module; |
| 783 | NiceMock<MockRtpRtcp> receive_module; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 784 | |
| 785 | constexpr bool remb_candidate = true; |
| 786 | |
| 787 | // Receive module added - activated. |
| 788 | packet_router.AddReceiveRtpModule(&receive_module, remb_candidate); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 789 | |
| 790 | // Send module added - replaces receive module as active. |
| 791 | packet_router.AddSendRtpModule(&send_module, remb_candidate); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 792 | |
| 793 | constexpr uint32_t bitrate_estimate = 456; |
| 794 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 795 | EXPECT_CALL(send_module, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
| 796 | EXPECT_CALL(receive_module, SetRemb(_, _)).Times(0); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 797 | |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 798 | clock.AdvanceTime(TimeDelta::ms(1000)); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 799 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 800 | |
| 801 | // Test tear-down |
| 802 | packet_router.RemoveReceiveRtpModule(&receive_module); |
| 803 | packet_router.RemoveSendRtpModule(&send_module); |
| 804 | } |
| 805 | |
| 806 | TEST(PacketRouterRembTest, ReceiveModuleTakesOverWhenLastSendModuleRemoved) { |
| 807 | rtc::ScopedFakeClock clock; |
| 808 | PacketRouter packet_router; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 809 | NiceMock<MockRtpRtcp> send_module; |
| 810 | NiceMock<MockRtpRtcp> receive_module; |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 811 | |
| 812 | constexpr bool remb_candidate = true; |
| 813 | |
| 814 | // Send module active, receive module inactive. |
| 815 | packet_router.AddSendRtpModule(&send_module, remb_candidate); |
| 816 | packet_router.AddReceiveRtpModule(&receive_module, remb_candidate); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 817 | |
| 818 | // Send module removed - receive module becomes active. |
| 819 | packet_router.RemoveSendRtpModule(&send_module); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 820 | constexpr uint32_t bitrate_estimate = 456; |
| 821 | const std::vector<uint32_t> ssrcs = {1234}; |
Danil Chapovalov | 51e21aa | 2017-10-10 17:46:26 +0200 | [diff] [blame] | 822 | EXPECT_CALL(send_module, SetRemb(_, _)).Times(0); |
| 823 | EXPECT_CALL(receive_module, SetRemb(bitrate_estimate, ssrcs)).Times(1); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 824 | |
Sebastian Jansson | 5f83cf0 | 2018-05-08 14:52:22 +0200 | [diff] [blame] | 825 | clock.AdvanceTime(TimeDelta::ms(1000)); |
eladalon | 822ff2b | 2017-08-01 06:30:28 -0700 | [diff] [blame] | 826 | packet_router.OnReceiveBitrateChanged(ssrcs, bitrate_estimate); |
| 827 | |
| 828 | // Test tear-down |
| 829 | packet_router.RemoveReceiveRtpModule(&receive_module); |
| 830 | } |
| 831 | |
Stefan Holmer | e590416 | 2015-03-26 11:11:06 +0100 | [diff] [blame] | 832 | } // namespace webrtc |