mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [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 "call/rtp_video_sender.h" |
| 12 | |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 13 | #include <atomic> |
kwiberg | 27f982b | 2016-03-01 11:52:33 -0800 | [diff] [blame] | 14 | #include <memory> |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 15 | #include <string> |
Tommi | 6fba6b7 | 2022-01-28 09:00:01 +0100 | [diff] [blame] | 16 | #include <utility> |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 17 | |
Danil Chapovalov | b7128ed | 2022-07-06 18:35:01 +0200 | [diff] [blame] | 18 | #include "absl/functional/any_invocable.h" |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 19 | #include "call/rtp_transport_controller_send.h" |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 20 | #include "modules/rtp_rtcp/include/rtp_rtcp_defines.h" |
| 21 | #include "modules/rtp_rtcp/source/byte_io.h" |
| 22 | #include "modules/rtp_rtcp/source/rtcp_packet/nack.h" |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 23 | #include "modules/rtp_rtcp/source/rtp_dependency_descriptor_extension.h" |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 24 | #include "modules/rtp_rtcp/source/rtp_packet.h" |
Stefan Holmer | 64be7fa | 2018-10-04 15:21:55 +0200 | [diff] [blame] | 25 | #include "modules/video_coding/fec_controller_default.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 26 | #include "modules/video_coding/include/video_codec_interface.h" |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 27 | #include "rtc_base/rate_limiter.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 06:47:31 +0200 | [diff] [blame] | 28 | #include "test/gmock.h" |
| 29 | #include "test/gtest.h" |
Marina Ciocea | 1148fd5 | 2020-04-29 22:39:13 +0200 | [diff] [blame] | 30 | #include "test/mock_frame_transformer.h" |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 31 | #include "test/mock_transport.h" |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 32 | #include "test/scenario/scenario.h" |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 33 | #include "test/scoped_key_value_config.h" |
Erik Språng | 00cc836 | 2019-11-25 12:21:46 +0100 | [diff] [blame] | 34 | #include "test/time_controller/simulated_time_controller.h" |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 35 | #include "video/send_delay_stats.h" |
| 36 | #include "video/send_statistics_proxy.h" |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 37 | |
Emil Lundmark | 6c81a42 | 2022-05-18 17:13:34 +0200 | [diff] [blame] | 38 | namespace webrtc { |
| 39 | namespace { |
| 40 | |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 41 | using ::testing::_; |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 42 | using ::testing::NiceMock; |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 43 | using ::testing::SaveArg; |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 44 | using ::testing::SizeIs; |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 45 | |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 46 | const int8_t kPayloadType = 96; |
| 47 | const uint32_t kSsrc1 = 12345; |
| 48 | const uint32_t kSsrc2 = 23456; |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 49 | const uint32_t kRtxSsrc1 = 34567; |
| 50 | const uint32_t kRtxSsrc2 = 45678; |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 51 | const int16_t kInitialPictureId1 = 222; |
| 52 | const int16_t kInitialPictureId2 = 44; |
Niels Möller | bb894ff | 2018-03-15 12:28:53 +0100 | [diff] [blame] | 53 | const int16_t kInitialTl0PicIdx1 = 99; |
| 54 | const int16_t kInitialTl0PicIdx2 = 199; |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 55 | const int64_t kRetransmitWindowSizeMs = 500; |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 56 | const int kTransportsSequenceExtensionId = 7; |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 57 | const int kDependencyDescriptorExtensionId = 8; |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 58 | |
| 59 | class MockRtcpIntraFrameObserver : public RtcpIntraFrameObserver { |
| 60 | public: |
Danil Chapovalov | 9a5efe9 | 2020-05-14 22:06:18 +0200 | [diff] [blame] | 61 | MOCK_METHOD(void, OnReceivedIntraFrameRequest, (uint32_t), (override)); |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 62 | }; |
| 63 | |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 64 | RtpSenderObservers CreateObservers( |
| 65 | RtcpRttStats* rtcp_rtt_stats, |
| 66 | RtcpIntraFrameObserver* intra_frame_callback, |
Henrik Boström | 87e3f9d | 2019-05-27 10:44:24 +0200 | [diff] [blame] | 67 | ReportBlockDataObserver* report_block_data_observer, |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 68 | StreamDataCountersCallback* rtp_stats, |
| 69 | BitrateStatisticsObserver* bitrate_observer, |
| 70 | FrameCountObserver* frame_count_observer, |
| 71 | RtcpPacketTypeCounterObserver* rtcp_type_observer, |
| 72 | SendSideDelayObserver* send_delay_observer, |
Stefan Holmer | 64be7fa | 2018-10-04 15:21:55 +0200 | [diff] [blame] | 73 | SendPacketObserver* send_packet_observer) { |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 74 | RtpSenderObservers observers; |
| 75 | observers.rtcp_rtt_stats = rtcp_rtt_stats; |
| 76 | observers.intra_frame_callback = intra_frame_callback; |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 77 | observers.rtcp_loss_notification_observer = nullptr; |
Henrik Boström | 87e3f9d | 2019-05-27 10:44:24 +0200 | [diff] [blame] | 78 | observers.report_block_data_observer = report_block_data_observer; |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 79 | observers.rtp_stats = rtp_stats; |
| 80 | observers.bitrate_observer = bitrate_observer; |
| 81 | observers.frame_count_observer = frame_count_observer; |
| 82 | observers.rtcp_type_observer = rtcp_type_observer; |
| 83 | observers.send_delay_observer = send_delay_observer; |
| 84 | observers.send_packet_observer = send_packet_observer; |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 85 | return observers; |
| 86 | } |
| 87 | |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 88 | BitrateConstraints GetBitrateConfig() { |
| 89 | BitrateConstraints bitrate_config; |
| 90 | bitrate_config.min_bitrate_bps = 30000; |
| 91 | bitrate_config.start_bitrate_bps = 300000; |
| 92 | bitrate_config.max_bitrate_bps = 3000000; |
| 93 | return bitrate_config; |
| 94 | } |
| 95 | |
| 96 | VideoSendStream::Config CreateVideoSendStreamConfig( |
| 97 | Transport* transport, |
| 98 | const std::vector<uint32_t>& ssrcs, |
| 99 | const std::vector<uint32_t>& rtx_ssrcs, |
| 100 | int payload_type) { |
| 101 | VideoSendStream::Config config(transport); |
| 102 | config.rtp.ssrcs = ssrcs; |
| 103 | config.rtp.rtx.ssrcs = rtx_ssrcs; |
| 104 | config.rtp.payload_type = payload_type; |
| 105 | config.rtp.rtx.payload_type = payload_type + 1; |
| 106 | config.rtp.nack.rtp_history_ms = 1000; |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 107 | config.rtp.extensions.emplace_back(RtpExtension::kTransportSequenceNumberUri, |
| 108 | kTransportsSequenceExtensionId); |
Danil Chapovalov | d0321c5 | 2021-09-14 12:58:51 +0200 | [diff] [blame] | 109 | config.rtp.extensions.emplace_back(RtpDependencyDescriptorExtension::Uri(), |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 110 | kDependencyDescriptorExtensionId); |
Danil Chapovalov | 748550d | 2021-04-29 11:42:54 +0200 | [diff] [blame] | 111 | config.rtp.extmap_allow_mixed = true; |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 112 | return config; |
| 113 | } |
| 114 | |
Stefan Holmer | 9416ef8 | 2018-07-19 10:34:38 +0200 | [diff] [blame] | 115 | class RtpVideoSenderTestFixture { |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 116 | public: |
Stefan Holmer | 9416ef8 | 2018-07-19 10:34:38 +0200 | [diff] [blame] | 117 | RtpVideoSenderTestFixture( |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 118 | const std::vector<uint32_t>& ssrcs, |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 119 | const std::vector<uint32_t>& rtx_ssrcs, |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 120 | int payload_type, |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 121 | const std::map<uint32_t, RtpPayloadState>& suspended_payload_states, |
Marina Ciocea | dc69fd2 | 2020-04-10 20:19:14 +0200 | [diff] [blame] | 122 | FrameCountObserver* frame_count_observer, |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 123 | rtc::scoped_refptr<FrameTransformerInterface> frame_transformer, |
Jonas Oreland | e62c2f2 | 2022-03-29 11:04:48 +0200 | [diff] [blame] | 124 | const FieldTrialsView* field_trials = nullptr) |
Danil Chapovalov | 0c626af | 2020-02-10 11:16:00 +0100 | [diff] [blame] | 125 | : time_controller_(Timestamp::Millis(1000000)), |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 126 | config_(CreateVideoSendStreamConfig(&transport_, |
| 127 | ssrcs, |
| 128 | rtx_ssrcs, |
| 129 | payload_type)), |
Erik Språng | 00cc836 | 2019-11-25 12:21:46 +0100 | [diff] [blame] | 130 | send_delay_stats_(time_controller_.GetClock()), |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 131 | bitrate_config_(GetBitrateConfig()), |
Henrik Boström | da4c102 | 2022-11-15 15:45:41 +0100 | [diff] [blame] | 132 | transport_controller_( |
| 133 | time_controller_.GetClock(), |
| 134 | RtpTransportConfig{ |
| 135 | .bitrate_config = bitrate_config_, |
| 136 | .event_log = &event_log_, |
| 137 | .task_queue_factory = time_controller_.GetTaskQueueFactory(), |
| 138 | .trials = field_trials ? field_trials : &field_trials_, |
| 139 | }), |
Erik Språng | 00cc836 | 2019-11-25 12:21:46 +0100 | [diff] [blame] | 140 | stats_proxy_(time_controller_.GetClock(), |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 141 | config_, |
Jonas Oreland | 8ca0613 | 2022-03-14 12:52:48 +0100 | [diff] [blame] | 142 | VideoEncoderConfig::ContentType::kRealtimeVideo, |
| 143 | field_trials ? *field_trials : field_trials_), |
Erik Språng | 00cc836 | 2019-11-25 12:21:46 +0100 | [diff] [blame] | 144 | retransmission_rate_limiter_(time_controller_.GetClock(), |
| 145 | kRetransmitWindowSizeMs) { |
Erik Språng | 7703f23 | 2020-09-14 11:03:13 +0200 | [diff] [blame] | 146 | transport_controller_.EnsureStarted(); |
Mirko Bonadei | f18f920 | 2019-12-10 13:24:56 +0000 | [diff] [blame] | 147 | std::map<uint32_t, RtpState> suspended_ssrcs; |
| 148 | router_ = std::make_unique<RtpVideoSender>( |
| 149 | time_controller_.GetClock(), suspended_ssrcs, suspended_payload_states, |
| 150 | config_.rtp, config_.rtcp_report_interval_ms, &transport_, |
Niels Möller | 98d9c30 | 2020-12-16 11:53:09 +0100 | [diff] [blame] | 151 | CreateObservers(nullptr, &encoder_feedback_, &stats_proxy_, |
Danil Chapovalov | f01c2c9 | 2021-05-14 15:39:23 +0200 | [diff] [blame] | 152 | &stats_proxy_, &stats_proxy_, frame_count_observer, |
| 153 | &stats_proxy_, &stats_proxy_, &send_delay_stats_), |
Mirko Bonadei | f18f920 | 2019-12-10 13:24:56 +0000 | [diff] [blame] | 154 | &transport_controller_, &event_log_, &retransmission_rate_limiter_, |
| 155 | std::make_unique<FecControllerDefault>(time_controller_.GetClock()), |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 156 | nullptr, CryptoOptions{}, frame_transformer, |
Evan Shrubsole | 9b643d4 | 2022-10-07 14:22:44 +0000 | [diff] [blame] | 157 | field_trials ? *field_trials : field_trials_, |
| 158 | time_controller_.GetTaskQueueFactory()); |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 159 | } |
Marina Ciocea | dc69fd2 | 2020-04-10 20:19:14 +0200 | [diff] [blame] | 160 | |
| 161 | RtpVideoSenderTestFixture( |
| 162 | const std::vector<uint32_t>& ssrcs, |
| 163 | const std::vector<uint32_t>& rtx_ssrcs, |
| 164 | int payload_type, |
| 165 | const std::map<uint32_t, RtpPayloadState>& suspended_payload_states, |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 166 | FrameCountObserver* frame_count_observer, |
Jonas Oreland | e62c2f2 | 2022-03-29 11:04:48 +0200 | [diff] [blame] | 167 | const FieldTrialsView* field_trials = nullptr) |
Marina Ciocea | dc69fd2 | 2020-04-10 20:19:14 +0200 | [diff] [blame] | 168 | : RtpVideoSenderTestFixture(ssrcs, |
| 169 | rtx_ssrcs, |
| 170 | payload_type, |
| 171 | suspended_payload_states, |
| 172 | frame_count_observer, |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 173 | /*frame_transformer=*/nullptr, |
| 174 | field_trials) {} |
Marina Ciocea | dc69fd2 | 2020-04-10 20:19:14 +0200 | [diff] [blame] | 175 | |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 176 | RtpVideoSenderTestFixture( |
| 177 | const std::vector<uint32_t>& ssrcs, |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 178 | const std::vector<uint32_t>& rtx_ssrcs, |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 179 | int payload_type, |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 180 | const std::map<uint32_t, RtpPayloadState>& suspended_payload_states, |
Jonas Oreland | e62c2f2 | 2022-03-29 11:04:48 +0200 | [diff] [blame] | 181 | const FieldTrialsView* field_trials = nullptr) |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 182 | : RtpVideoSenderTestFixture(ssrcs, |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 183 | rtx_ssrcs, |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 184 | payload_type, |
| 185 | suspended_payload_states, |
Marina Ciocea | dc69fd2 | 2020-04-10 20:19:14 +0200 | [diff] [blame] | 186 | /*frame_count_observer=*/nullptr, |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 187 | /*frame_transformer=*/nullptr, |
| 188 | field_trials) {} |
Mirko Bonadei | f18f920 | 2019-12-10 13:24:56 +0000 | [diff] [blame] | 189 | |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 190 | ~RtpVideoSenderTestFixture() { Stop(); } |
Tommi | 6fba6b7 | 2022-01-28 09:00:01 +0100 | [diff] [blame] | 191 | |
Stefan Holmer | 9416ef8 | 2018-07-19 10:34:38 +0200 | [diff] [blame] | 192 | RtpVideoSender* router() { return router_.get(); } |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 193 | MockTransport& transport() { return transport_; } |
Markus Handell | 486cc55 | 2019-12-03 14:37:28 +0100 | [diff] [blame] | 194 | void AdvanceTime(TimeDelta delta) { time_controller_.AdvanceTime(delta); } |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 195 | |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 196 | void Stop() { |
| 197 | RunOnTransportQueue([&]() { router_->Stop(); }); |
Tommi | 6fba6b7 | 2022-01-28 09:00:01 +0100 | [diff] [blame] | 198 | } |
| 199 | |
| 200 | void SetActiveModules(const std::vector<bool>& active_modules) { |
| 201 | RunOnTransportQueue([&]() { router_->SetActiveModules(active_modules); }); |
| 202 | } |
| 203 | |
| 204 | // Several RtpVideoSender methods expect to be called on the task queue as |
| 205 | // owned by the send transport. While the SequenceChecker may pick up the |
| 206 | // default thread as the transport queue, explicit checks for the transport |
| 207 | // queue (not just using a SequenceChecker) aren't possible unless such a |
| 208 | // queue is actually active. So RunOnTransportQueue is a convenience function |
Danil Chapovalov | b7128ed | 2022-07-06 18:35:01 +0200 | [diff] [blame] | 209 | // that allow for running a `task` on the transport queue, similar to |
Tommi | 6fba6b7 | 2022-01-28 09:00:01 +0100 | [diff] [blame] | 210 | // SendTask(). |
Danil Chapovalov | b7128ed | 2022-07-06 18:35:01 +0200 | [diff] [blame] | 211 | void RunOnTransportQueue(absl::AnyInvocable<void() &&> task) { |
Per Kjellander | 828ef91 | 2022-10-10 12:53:41 +0200 | [diff] [blame] | 212 | transport_controller_.GetWorkerQueue()->RunOrPost(std::move(task)); |
Philipp Hancke | a204ad2 | 2022-07-08 18:43:25 +0200 | [diff] [blame] | 213 | AdvanceTime(TimeDelta::Zero()); |
Tommi | 6fba6b7 | 2022-01-28 09:00:01 +0100 | [diff] [blame] | 214 | } |
| 215 | |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 216 | private: |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 217 | test::ScopedKeyValueConfig field_trials_; |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 218 | NiceMock<MockTransport> transport_; |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 219 | NiceMock<MockRtcpIntraFrameObserver> encoder_feedback_; |
Erik Språng | 00cc836 | 2019-11-25 12:21:46 +0100 | [diff] [blame] | 220 | GlobalSimulatedTimeController time_controller_; |
Danil Chapovalov | 83bbe91 | 2019-08-07 12:24:53 +0200 | [diff] [blame] | 221 | RtcEventLogNull event_log_; |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 222 | VideoSendStream::Config config_; |
| 223 | SendDelayStats send_delay_stats_; |
| 224 | BitrateConstraints bitrate_config_; |
| 225 | RtpTransportControllerSend transport_controller_; |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 226 | SendStatisticsProxy stats_proxy_; |
| 227 | RateLimiter retransmission_rate_limiter_; |
Stefan Holmer | 9416ef8 | 2018-07-19 10:34:38 +0200 | [diff] [blame] | 228 | std::unique_ptr<RtpVideoSender> router_; |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 229 | }; |
Jakob Ivarsson | 9a12ee5 | 2020-11-26 16:04:18 +0100 | [diff] [blame] | 230 | |
| 231 | BitrateAllocationUpdate CreateBitrateAllocationUpdate(int target_bitrate_bps) { |
| 232 | BitrateAllocationUpdate update; |
| 233 | update.target_bitrate = DataRate::BitsPerSec(target_bitrate_bps); |
| 234 | update.round_trip_time = TimeDelta::Zero(); |
| 235 | return update; |
| 236 | } |
| 237 | |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 238 | } // namespace |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 239 | |
Sebastian Jansson | 6347029 | 2019-02-01 10:13:43 +0100 | [diff] [blame] | 240 | TEST(RtpVideoSenderTest, SendOnOneModule) { |
Niels Möller | 663844d | 2019-02-14 16:15:54 +0100 | [diff] [blame] | 241 | constexpr uint8_t kPayload = 'a'; |
kjellander | 02b3d27 | 2016-04-20 05:05:54 -0700 | [diff] [blame] | 242 | EncodedImage encoded_image; |
Niels Möller | 2377588 | 2018-08-16 10:24:12 +0200 | [diff] [blame] | 243 | encoded_image.SetTimestamp(1); |
kjellander | 02b3d27 | 2016-04-20 05:05:54 -0700 | [diff] [blame] | 244 | encoded_image.capture_time_ms_ = 2; |
Niels Möller | 8f7ce22 | 2019-03-21 15:43:58 +0100 | [diff] [blame] | 245 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
Niels Möller | 4d504c7 | 2019-06-18 15:56:56 +0200 | [diff] [blame] | 246 | encoded_image.SetEncodedData(EncodedImageBuffer::Create(&kPayload, 1)); |
kjellander | 02b3d27 | 2016-04-20 05:05:54 -0700 | [diff] [blame] | 247 | |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 248 | RtpVideoSenderTestFixture test({kSsrc1}, {kRtxSsrc1}, kPayloadType, {}); |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 249 | EXPECT_NE(EncodedImageCallback::Result::OK, |
| 250 | test.router()->OnEncodedImage(encoded_image, nullptr).error); |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 251 | |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 252 | test.SetActiveModules({true}); |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 253 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
| 254 | test.router()->OnEncodedImage(encoded_image, nullptr).error); |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 255 | |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 256 | test.SetActiveModules({false}); |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 257 | EXPECT_NE(EncodedImageCallback::Result::OK, |
| 258 | test.router()->OnEncodedImage(encoded_image, nullptr).error); |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 259 | |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 260 | test.SetActiveModules({true}); |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 261 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
| 262 | test.router()->OnEncodedImage(encoded_image, nullptr).error); |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 263 | } |
| 264 | |
Sebastian Jansson | 6347029 | 2019-02-01 10:13:43 +0100 | [diff] [blame] | 265 | TEST(RtpVideoSenderTest, SendSimulcastSetActive) { |
Niels Möller | 663844d | 2019-02-14 16:15:54 +0100 | [diff] [blame] | 266 | constexpr uint8_t kPayload = 'a'; |
Niels Möller | d3b8c63 | 2018-08-27 15:33:42 +0200 | [diff] [blame] | 267 | EncodedImage encoded_image_1; |
| 268 | encoded_image_1.SetTimestamp(1); |
| 269 | encoded_image_1.capture_time_ms_ = 2; |
Niels Möller | 8f7ce22 | 2019-03-21 15:43:58 +0100 | [diff] [blame] | 270 | encoded_image_1._frameType = VideoFrameType::kVideoFrameKey; |
Niels Möller | 4d504c7 | 2019-06-18 15:56:56 +0200 | [diff] [blame] | 271 | encoded_image_1.SetEncodedData(EncodedImageBuffer::Create(&kPayload, 1)); |
kjellander | 02b3d27 | 2016-04-20 05:05:54 -0700 | [diff] [blame] | 272 | |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 273 | RtpVideoSenderTestFixture test({kSsrc1, kSsrc2}, {kRtxSsrc1, kRtxSsrc2}, |
| 274 | kPayloadType, {}); |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 275 | |
Niels Möller | d3b8c63 | 2018-08-27 15:33:42 +0200 | [diff] [blame] | 276 | CodecSpecificInfo codec_info; |
Niels Möller | d3b8c63 | 2018-08-27 15:33:42 +0200 | [diff] [blame] | 277 | codec_info.codecType = kVideoCodecVP8; |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 278 | |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 279 | test.SetActiveModules({true, true}); |
sergeyu | 2cb155a | 2016-11-04 11:39:29 -0700 | [diff] [blame] | 280 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 281 | test.router()->OnEncodedImage(encoded_image_1, &codec_info).error); |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 282 | |
Niels Möller | d3b8c63 | 2018-08-27 15:33:42 +0200 | [diff] [blame] | 283 | EncodedImage encoded_image_2(encoded_image_1); |
| 284 | encoded_image_2.SetSpatialIndex(1); |
sergeyu | 2cb155a | 2016-11-04 11:39:29 -0700 | [diff] [blame] | 285 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 286 | test.router()->OnEncodedImage(encoded_image_2, &codec_info).error); |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 287 | |
mflodman@webrtc.org | 50e2816 | 2015-02-23 07:45:11 +0000 | [diff] [blame] | 288 | // Inactive. |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 289 | test.Stop(); |
sergeyu | 2cb155a | 2016-11-04 11:39:29 -0700 | [diff] [blame] | 290 | EXPECT_NE(EncodedImageCallback::Result::OK, |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 291 | test.router()->OnEncodedImage(encoded_image_1, &codec_info).error); |
sergeyu | 2cb155a | 2016-11-04 11:39:29 -0700 | [diff] [blame] | 292 | EXPECT_NE(EncodedImageCallback::Result::OK, |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 293 | test.router()->OnEncodedImage(encoded_image_2, &codec_info).error); |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 294 | } |
| 295 | |
Seth Hampson | cc7125f | 2018-02-02 08:46:16 -0800 | [diff] [blame] | 296 | // Tests how setting individual rtp modules to active affects the overall |
| 297 | // behavior of the payload router. First sets one module to active and checks |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 298 | // that outgoing data can be sent on this module, and checks that no data can |
| 299 | // be sent if both modules are inactive. |
Sebastian Jansson | 6347029 | 2019-02-01 10:13:43 +0100 | [diff] [blame] | 300 | TEST(RtpVideoSenderTest, SendSimulcastSetActiveModules) { |
Niels Möller | 663844d | 2019-02-14 16:15:54 +0100 | [diff] [blame] | 301 | constexpr uint8_t kPayload = 'a'; |
Niels Möller | d3b8c63 | 2018-08-27 15:33:42 +0200 | [diff] [blame] | 302 | EncodedImage encoded_image_1; |
| 303 | encoded_image_1.SetTimestamp(1); |
| 304 | encoded_image_1.capture_time_ms_ = 2; |
Niels Möller | 8f7ce22 | 2019-03-21 15:43:58 +0100 | [diff] [blame] | 305 | encoded_image_1._frameType = VideoFrameType::kVideoFrameKey; |
Niels Möller | 4d504c7 | 2019-06-18 15:56:56 +0200 | [diff] [blame] | 306 | encoded_image_1.SetEncodedData(EncodedImageBuffer::Create(&kPayload, 1)); |
Niels Möller | 77536a2 | 2019-01-15 08:50:01 +0100 | [diff] [blame] | 307 | |
Niels Möller | d3b8c63 | 2018-08-27 15:33:42 +0200 | [diff] [blame] | 308 | EncodedImage encoded_image_2(encoded_image_1); |
| 309 | encoded_image_2.SetSpatialIndex(1); |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 310 | |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 311 | RtpVideoSenderTestFixture test({kSsrc1, kSsrc2}, {kRtxSsrc1, kRtxSsrc2}, |
| 312 | kPayloadType, {}); |
Niels Möller | d3b8c63 | 2018-08-27 15:33:42 +0200 | [diff] [blame] | 313 | CodecSpecificInfo codec_info; |
Niels Möller | d3b8c63 | 2018-08-27 15:33:42 +0200 | [diff] [blame] | 314 | codec_info.codecType = kVideoCodecVP8; |
Seth Hampson | cc7125f | 2018-02-02 08:46:16 -0800 | [diff] [blame] | 315 | |
| 316 | // Only setting one stream to active will still set the payload router to |
| 317 | // active and allow sending data on the active stream. |
| 318 | std::vector<bool> active_modules({true, false}); |
Tommi | 6fba6b7 | 2022-01-28 09:00:01 +0100 | [diff] [blame] | 319 | test.SetActiveModules(active_modules); |
Seth Hampson | cc7125f | 2018-02-02 08:46:16 -0800 | [diff] [blame] | 320 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 321 | test.router()->OnEncodedImage(encoded_image_1, &codec_info).error); |
Seth Hampson | cc7125f | 2018-02-02 08:46:16 -0800 | [diff] [blame] | 322 | |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 323 | // Setting both streams to inactive will turn the payload router to |
| 324 | // inactive. |
Seth Hampson | cc7125f | 2018-02-02 08:46:16 -0800 | [diff] [blame] | 325 | active_modules = {false, false}; |
Tommi | 6fba6b7 | 2022-01-28 09:00:01 +0100 | [diff] [blame] | 326 | test.SetActiveModules(active_modules); |
Seth Hampson | cc7125f | 2018-02-02 08:46:16 -0800 | [diff] [blame] | 327 | // An incoming encoded image will not ask the module to send outgoing data |
| 328 | // because the payload router is inactive. |
Seth Hampson | cc7125f | 2018-02-02 08:46:16 -0800 | [diff] [blame] | 329 | EXPECT_NE(EncodedImageCallback::Result::OK, |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 330 | test.router()->OnEncodedImage(encoded_image_1, &codec_info).error); |
Seth Hampson | cc7125f | 2018-02-02 08:46:16 -0800 | [diff] [blame] | 331 | EXPECT_NE(EncodedImageCallback::Result::OK, |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 332 | test.router()->OnEncodedImage(encoded_image_1, &codec_info).error); |
Seth Hampson | cc7125f | 2018-02-02 08:46:16 -0800 | [diff] [blame] | 333 | } |
| 334 | |
Per Kjellander | fb8641c | 2022-08-11 09:25:00 +0200 | [diff] [blame] | 335 | TEST( |
| 336 | RtpVideoSenderTest, |
| 337 | DiscardsHigherSpatialVideoFramesAfterLayerDisabledInVideoLayersAllocation) { |
| 338 | constexpr uint8_t kPayload = 'a'; |
| 339 | EncodedImage encoded_image_1; |
| 340 | encoded_image_1.SetTimestamp(1); |
| 341 | encoded_image_1.capture_time_ms_ = 2; |
| 342 | encoded_image_1._frameType = VideoFrameType::kVideoFrameKey; |
| 343 | encoded_image_1.SetEncodedData(EncodedImageBuffer::Create(&kPayload, 1)); |
| 344 | EncodedImage encoded_image_2(encoded_image_1); |
| 345 | encoded_image_2.SetSpatialIndex(1); |
| 346 | CodecSpecificInfo codec_info; |
| 347 | codec_info.codecType = kVideoCodecVP8; |
| 348 | RtpVideoSenderTestFixture test({kSsrc1, kSsrc2}, {kRtxSsrc1, kRtxSsrc2}, |
| 349 | kPayloadType, {}); |
| 350 | test.SetActiveModules({true, true}); |
| 351 | // A layer is sent on both rtp streams. |
| 352 | test.router()->OnVideoLayersAllocationUpdated( |
| 353 | {.active_spatial_layers = {{.rtp_stream_index = 0}, |
| 354 | {.rtp_stream_index = 1}}}); |
| 355 | |
| 356 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
| 357 | test.router()->OnEncodedImage(encoded_image_1, &codec_info).error); |
| 358 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
| 359 | test.router()->OnEncodedImage(encoded_image_2, &codec_info).error); |
| 360 | |
| 361 | // Only rtp stream index 0 is configured to send a stream. |
| 362 | test.router()->OnVideoLayersAllocationUpdated( |
| 363 | {.active_spatial_layers = {{.rtp_stream_index = 0}}}); |
| 364 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
| 365 | test.router()->OnEncodedImage(encoded_image_1, &codec_info).error); |
| 366 | EXPECT_NE(EncodedImageCallback::Result::OK, |
| 367 | test.router()->OnEncodedImage(encoded_image_2, &codec_info).error); |
| 368 | } |
| 369 | |
Sebastian Jansson | 6347029 | 2019-02-01 10:13:43 +0100 | [diff] [blame] | 370 | TEST(RtpVideoSenderTest, CreateWithNoPreviousStates) { |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 371 | RtpVideoSenderTestFixture test({kSsrc1, kSsrc2}, {kRtxSsrc1, kRtxSsrc2}, |
| 372 | kPayloadType, {}); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 373 | test.SetActiveModules({true, true}); |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 374 | |
| 375 | std::map<uint32_t, RtpPayloadState> initial_states = |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 376 | test.router()->GetRtpPayloadStates(); |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 377 | EXPECT_EQ(2u, initial_states.size()); |
| 378 | EXPECT_NE(initial_states.find(kSsrc1), initial_states.end()); |
| 379 | EXPECT_NE(initial_states.find(kSsrc2), initial_states.end()); |
| 380 | } |
| 381 | |
Sebastian Jansson | 6347029 | 2019-02-01 10:13:43 +0100 | [diff] [blame] | 382 | TEST(RtpVideoSenderTest, CreateWithPreviousStates) { |
philipel | 25d31ec | 2018-08-08 16:33:01 +0200 | [diff] [blame] | 383 | const int64_t kState1SharedFrameId = 123; |
| 384 | const int64_t kState2SharedFrameId = 234; |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 385 | RtpPayloadState state1; |
| 386 | state1.picture_id = kInitialPictureId1; |
Niels Möller | bb894ff | 2018-03-15 12:28:53 +0100 | [diff] [blame] | 387 | state1.tl0_pic_idx = kInitialTl0PicIdx1; |
philipel | 25d31ec | 2018-08-08 16:33:01 +0200 | [diff] [blame] | 388 | state1.shared_frame_id = kState1SharedFrameId; |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 389 | RtpPayloadState state2; |
| 390 | state2.picture_id = kInitialPictureId2; |
Niels Möller | bb894ff | 2018-03-15 12:28:53 +0100 | [diff] [blame] | 391 | state2.tl0_pic_idx = kInitialTl0PicIdx2; |
philipel | 25d31ec | 2018-08-08 16:33:01 +0200 | [diff] [blame] | 392 | state2.shared_frame_id = kState2SharedFrameId; |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 393 | std::map<uint32_t, RtpPayloadState> states = {{kSsrc1, state1}, |
| 394 | {kSsrc2, state2}}; |
| 395 | |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 396 | RtpVideoSenderTestFixture test({kSsrc1, kSsrc2}, {kRtxSsrc1, kRtxSsrc2}, |
| 397 | kPayloadType, states); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 398 | test.SetActiveModules({true, true}); |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 399 | |
| 400 | std::map<uint32_t, RtpPayloadState> initial_states = |
Stefan Holmer | dbdb3a0 | 2018-07-17 16:03:46 +0200 | [diff] [blame] | 401 | test.router()->GetRtpPayloadStates(); |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 402 | EXPECT_EQ(2u, initial_states.size()); |
| 403 | EXPECT_EQ(kInitialPictureId1, initial_states[kSsrc1].picture_id); |
Niels Möller | bb894ff | 2018-03-15 12:28:53 +0100 | [diff] [blame] | 404 | EXPECT_EQ(kInitialTl0PicIdx1, initial_states[kSsrc1].tl0_pic_idx); |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 405 | EXPECT_EQ(kInitialPictureId2, initial_states[kSsrc2].picture_id); |
Niels Möller | bb894ff | 2018-03-15 12:28:53 +0100 | [diff] [blame] | 406 | EXPECT_EQ(kInitialTl0PicIdx2, initial_states[kSsrc2].tl0_pic_idx); |
philipel | 25d31ec | 2018-08-08 16:33:01 +0200 | [diff] [blame] | 407 | EXPECT_EQ(kState2SharedFrameId, initial_states[kSsrc1].shared_frame_id); |
| 408 | EXPECT_EQ(kState2SharedFrameId, initial_states[kSsrc2].shared_frame_id); |
Åsa Persson | 4bece9a | 2017-10-06 10:04:04 +0200 | [diff] [blame] | 409 | } |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 410 | |
Sebastian Jansson | 6347029 | 2019-02-01 10:13:43 +0100 | [diff] [blame] | 411 | TEST(RtpVideoSenderTest, FrameCountCallbacks) { |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 412 | class MockFrameCountObserver : public FrameCountObserver { |
| 413 | public: |
Danil Chapovalov | 9a5efe9 | 2020-05-14 22:06:18 +0200 | [diff] [blame] | 414 | MOCK_METHOD(void, |
| 415 | FrameCountUpdated, |
| 416 | (const FrameCounts& frame_counts, uint32_t ssrc), |
| 417 | (override)); |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 418 | } callback; |
| 419 | |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 420 | RtpVideoSenderTestFixture test({kSsrc1}, {kRtxSsrc1}, kPayloadType, {}, |
| 421 | &callback); |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 422 | |
Niels Möller | 663844d | 2019-02-14 16:15:54 +0100 | [diff] [blame] | 423 | constexpr uint8_t kPayload = 'a'; |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 424 | EncodedImage encoded_image; |
| 425 | encoded_image.SetTimestamp(1); |
| 426 | encoded_image.capture_time_ms_ = 2; |
Niels Möller | 8f7ce22 | 2019-03-21 15:43:58 +0100 | [diff] [blame] | 427 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
Niels Möller | 4d504c7 | 2019-06-18 15:56:56 +0200 | [diff] [blame] | 428 | encoded_image.SetEncodedData(EncodedImageBuffer::Create(&kPayload, 1)); |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 429 | |
Niels Möller | 8f7ce22 | 2019-03-21 15:43:58 +0100 | [diff] [blame] | 430 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 431 | |
| 432 | // No callbacks when not active. |
| 433 | EXPECT_CALL(callback, FrameCountUpdated).Times(0); |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 434 | EXPECT_NE(EncodedImageCallback::Result::OK, |
| 435 | test.router()->OnEncodedImage(encoded_image, nullptr).error); |
Mirko Bonadei | 6a489f2 | 2019-04-09 15:11:12 +0200 | [diff] [blame] | 436 | ::testing::Mock::VerifyAndClearExpectations(&callback); |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 437 | |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 438 | test.SetActiveModules({true}); |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 439 | |
| 440 | FrameCounts frame_counts; |
| 441 | EXPECT_CALL(callback, FrameCountUpdated(_, kSsrc1)) |
| 442 | .WillOnce(SaveArg<0>(&frame_counts)); |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 443 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
| 444 | test.router()->OnEncodedImage(encoded_image, nullptr).error); |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 445 | |
| 446 | EXPECT_EQ(1, frame_counts.key_frames); |
| 447 | EXPECT_EQ(0, frame_counts.delta_frames); |
| 448 | |
Mirko Bonadei | 6a489f2 | 2019-04-09 15:11:12 +0200 | [diff] [blame] | 449 | ::testing::Mock::VerifyAndClearExpectations(&callback); |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 450 | |
Niels Möller | 8f7ce22 | 2019-03-21 15:43:58 +0100 | [diff] [blame] | 451 | encoded_image._frameType = VideoFrameType::kVideoFrameDelta; |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 452 | EXPECT_CALL(callback, FrameCountUpdated(_, kSsrc1)) |
| 453 | .WillOnce(SaveArg<0>(&frame_counts)); |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 454 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
| 455 | test.router()->OnEncodedImage(encoded_image, nullptr).error); |
Niels Möller | 949f0fd | 2019-01-29 09:44:24 +0100 | [diff] [blame] | 456 | |
| 457 | EXPECT_EQ(1, frame_counts.key_frames); |
| 458 | EXPECT_EQ(1, frame_counts.delta_frames); |
| 459 | } |
| 460 | |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 461 | // Integration test verifying that ack of packet via TransportFeedback means |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 462 | // that the packet is removed from RtpPacketHistory and won't be retransmitted |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 463 | // again. |
| 464 | TEST(RtpVideoSenderTest, DoesNotRetrasmitAckedPackets) { |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 465 | RtpVideoSenderTestFixture test({kSsrc1, kSsrc2}, {kRtxSsrc1, kRtxSsrc2}, |
| 466 | kPayloadType, {}); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 467 | test.SetActiveModules({true, true}); |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 468 | |
| 469 | constexpr uint8_t kPayload = 'a'; |
| 470 | EncodedImage encoded_image; |
| 471 | encoded_image.SetTimestamp(1); |
| 472 | encoded_image.capture_time_ms_ = 2; |
| 473 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
Niels Möller | 4d504c7 | 2019-06-18 15:56:56 +0200 | [diff] [blame] | 474 | encoded_image.SetEncodedData(EncodedImageBuffer::Create(&kPayload, 1)); |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 475 | |
| 476 | // Send two tiny images, mapping to two RTP packets. Capture sequence numbers. |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 477 | std::vector<uint16_t> rtp_sequence_numbers; |
| 478 | std::vector<uint16_t> transport_sequence_numbers; |
| 479 | EXPECT_CALL(test.transport(), SendRtp) |
| 480 | .Times(2) |
Danil Chapovalov | a68eb8c | 2020-02-17 16:13:14 +0100 | [diff] [blame] | 481 | .WillRepeatedly([&rtp_sequence_numbers, &transport_sequence_numbers]( |
| 482 | const uint8_t* packet, size_t length, |
| 483 | const PacketOptions& options) { |
| 484 | RtpPacket rtp_packet; |
| 485 | EXPECT_TRUE(rtp_packet.Parse(packet, length)); |
| 486 | rtp_sequence_numbers.push_back(rtp_packet.SequenceNumber()); |
| 487 | transport_sequence_numbers.push_back(options.packet_id); |
| 488 | return true; |
| 489 | }); |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 490 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
| 491 | test.router()->OnEncodedImage(encoded_image, nullptr).error); |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 492 | encoded_image.SetTimestamp(2); |
| 493 | encoded_image.capture_time_ms_ = 3; |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 494 | EXPECT_EQ(EncodedImageCallback::Result::OK, |
| 495 | test.router()->OnEncodedImage(encoded_image, nullptr).error); |
Erik Språng | bd7046c | 2019-05-07 14:54:29 -0700 | [diff] [blame] | 496 | |
Danil Chapovalov | 0c626af | 2020-02-10 11:16:00 +0100 | [diff] [blame] | 497 | test.AdvanceTime(TimeDelta::Millis(33)); |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 498 | |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 499 | // Construct a NACK message for requesting retransmission of both packet. |
| 500 | rtcp::Nack nack; |
| 501 | nack.SetMediaSsrc(kSsrc1); |
| 502 | nack.SetPacketIds(rtp_sequence_numbers); |
| 503 | rtc::Buffer nack_buffer = nack.Build(); |
| 504 | |
| 505 | std::vector<uint16_t> retransmitted_rtp_sequence_numbers; |
| 506 | EXPECT_CALL(test.transport(), SendRtp) |
| 507 | .Times(2) |
Danil Chapovalov | a68eb8c | 2020-02-17 16:13:14 +0100 | [diff] [blame] | 508 | .WillRepeatedly([&retransmitted_rtp_sequence_numbers]( |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 509 | const uint8_t* packet, size_t length, |
| 510 | const PacketOptions& options) { |
| 511 | RtpPacket rtp_packet; |
| 512 | EXPECT_TRUE(rtp_packet.Parse(packet, length)); |
| 513 | EXPECT_EQ(rtp_packet.Ssrc(), kRtxSsrc1); |
| 514 | // Capture the retransmitted sequence number from the RTX header. |
| 515 | rtc::ArrayView<const uint8_t> payload = rtp_packet.payload(); |
| 516 | retransmitted_rtp_sequence_numbers.push_back( |
| 517 | ByteReader<uint16_t>::ReadBigEndian(payload.data())); |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 518 | return true; |
| 519 | }); |
| 520 | test.router()->DeliverRtcp(nack_buffer.data(), nack_buffer.size()); |
Danil Chapovalov | 0c626af | 2020-02-10 11:16:00 +0100 | [diff] [blame] | 521 | test.AdvanceTime(TimeDelta::Millis(33)); |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 522 | |
| 523 | // Verify that both packets were retransmitted. |
| 524 | EXPECT_EQ(retransmitted_rtp_sequence_numbers, rtp_sequence_numbers); |
| 525 | |
| 526 | // Simulate transport feedback indicating fist packet received, next packet |
Erik Språng | 00cc836 | 2019-11-25 12:21:46 +0100 | [diff] [blame] | 527 | // lost (not other way around as that would trigger early retransmit). |
Sebastian Jansson | f298855 | 2019-10-29 17:18:51 +0100 | [diff] [blame] | 528 | StreamFeedbackObserver::StreamPacketInfo lost_packet_feedback; |
Erik Språng | 00cc836 | 2019-11-25 12:21:46 +0100 | [diff] [blame] | 529 | lost_packet_feedback.rtp_sequence_number = rtp_sequence_numbers[0]; |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 530 | lost_packet_feedback.ssrc = kSsrc1; |
Sebastian Jansson | f298855 | 2019-10-29 17:18:51 +0100 | [diff] [blame] | 531 | lost_packet_feedback.received = false; |
Erik Språng | 6a0a559 | 2021-06-15 19:04:24 +0200 | [diff] [blame] | 532 | lost_packet_feedback.is_retransmission = false; |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 533 | |
Erik Språng | 00cc836 | 2019-11-25 12:21:46 +0100 | [diff] [blame] | 534 | StreamFeedbackObserver::StreamPacketInfo received_packet_feedback; |
| 535 | received_packet_feedback.rtp_sequence_number = rtp_sequence_numbers[1]; |
| 536 | received_packet_feedback.ssrc = kSsrc1; |
| 537 | received_packet_feedback.received = true; |
Erik Språng | 6a0a559 | 2021-06-15 19:04:24 +0200 | [diff] [blame] | 538 | lost_packet_feedback.is_retransmission = false; |
Erik Språng | 00cc836 | 2019-11-25 12:21:46 +0100 | [diff] [blame] | 539 | |
Sebastian Jansson | f298855 | 2019-10-29 17:18:51 +0100 | [diff] [blame] | 540 | test.router()->OnPacketFeedbackVector( |
Erik Språng | 00cc836 | 2019-11-25 12:21:46 +0100 | [diff] [blame] | 541 | {lost_packet_feedback, received_packet_feedback}); |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 542 | |
| 543 | // Advance time to make sure retransmission would be allowed and try again. |
| 544 | // This time the retransmission should not happen for the first packet since |
| 545 | // the history has been notified of the ack and removed the packet. The |
| 546 | // second packet, included in the feedback but not marked as received, should |
| 547 | // still be retransmitted. |
Danil Chapovalov | 0c626af | 2020-02-10 11:16:00 +0100 | [diff] [blame] | 548 | test.AdvanceTime(TimeDelta::Millis(33)); |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 549 | EXPECT_CALL(test.transport(), SendRtp) |
Danil Chapovalov | a68eb8c | 2020-02-17 16:13:14 +0100 | [diff] [blame] | 550 | .WillOnce([&lost_packet_feedback](const uint8_t* packet, size_t length, |
| 551 | const PacketOptions& options) { |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 552 | RtpPacket rtp_packet; |
| 553 | EXPECT_TRUE(rtp_packet.Parse(packet, length)); |
| 554 | EXPECT_EQ(rtp_packet.Ssrc(), kRtxSsrc1); |
| 555 | // Capture the retransmitted sequence number from the RTX header. |
| 556 | rtc::ArrayView<const uint8_t> payload = rtp_packet.payload(); |
| 557 | EXPECT_EQ(lost_packet_feedback.rtp_sequence_number, |
| 558 | ByteReader<uint16_t>::ReadBigEndian(payload.data())); |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 559 | return true; |
| 560 | }); |
| 561 | test.router()->DeliverRtcp(nack_buffer.data(), nack_buffer.size()); |
Danil Chapovalov | 0c626af | 2020-02-10 11:16:00 +0100 | [diff] [blame] | 562 | test.AdvanceTime(TimeDelta::Millis(33)); |
Erik Språng | 490d76c | 2019-05-07 09:29:15 -0700 | [diff] [blame] | 563 | } |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 564 | |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 565 | // This tests that we utilize transport wide feedback to retransmit lost |
Tommi | 6fba6b7 | 2022-01-28 09:00:01 +0100 | [diff] [blame] | 566 | // packets. This is tested by dropping all ordinary packets from a "lossy" |
| 567 | // stream sent along with a secondary untouched stream. The transport wide |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 568 | // feedback packets from the secondary stream allows the sending side to |
| 569 | // detect and retreansmit the lost packets from the lossy stream. |
| 570 | TEST(RtpVideoSenderTest, RetransmitsOnTransportWideLossInfo) { |
| 571 | int rtx_packets; |
| 572 | test::Scenario s(test_info_); |
| 573 | test::CallClientConfig call_conf; |
| 574 | // Keeping the bitrate fixed to avoid RTX due to probing. |
Danil Chapovalov | cad3e0e | 2020-02-17 18:46:07 +0100 | [diff] [blame] | 575 | call_conf.transport.rates.max_rate = DataRate::KilobitsPerSec(300); |
| 576 | call_conf.transport.rates.start_rate = DataRate::KilobitsPerSec(300); |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 577 | test::NetworkSimulationConfig net_conf; |
Danil Chapovalov | cad3e0e | 2020-02-17 18:46:07 +0100 | [diff] [blame] | 578 | net_conf.bandwidth = DataRate::KilobitsPerSec(300); |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 579 | auto send_node = s.CreateSimulationNode(net_conf); |
Tommi | 3c9bcc1 | 2020-04-15 16:45:47 +0200 | [diff] [blame] | 580 | auto* callee = s.CreateClient("return", call_conf); |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 581 | auto* route = s.CreateRoutes(s.CreateClient("send", call_conf), {send_node}, |
Tommi | 3c9bcc1 | 2020-04-15 16:45:47 +0200 | [diff] [blame] | 582 | callee, {s.CreateSimulationNode(net_conf)}); |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 583 | |
| 584 | test::VideoStreamConfig lossy_config; |
| 585 | lossy_config.source.framerate = 5; |
| 586 | auto* lossy = s.CreateVideoStream(route->forward(), lossy_config); |
| 587 | // The secondary stream acts a driver for transport feedback messages, |
| 588 | // ensuring that lost packets on the lossy stream are retransmitted. |
| 589 | s.CreateVideoStream(route->forward(), test::VideoStreamConfig()); |
| 590 | |
| 591 | send_node->router()->SetFilter([&](const EmulatedIpPacket& packet) { |
| 592 | RtpPacket rtp; |
| 593 | if (rtp.Parse(packet.data)) { |
| 594 | // Drops all regular packets for the lossy stream and counts all RTX |
| 595 | // packets. Since no packets are let trough, NACKs can't be triggered |
| 596 | // by the receiving side. |
| 597 | if (lossy->send()->UsingSsrc(rtp.Ssrc())) { |
| 598 | return false; |
| 599 | } else if (lossy->send()->UsingRtxSsrc(rtp.Ssrc())) { |
| 600 | ++rtx_packets; |
| 601 | } |
| 602 | } |
| 603 | return true; |
| 604 | }); |
| 605 | |
| 606 | // Run for a short duration and reset counters to avoid counting RTX packets |
| 607 | // from initial probing. |
Danil Chapovalov | 0c626af | 2020-02-10 11:16:00 +0100 | [diff] [blame] | 608 | s.RunFor(TimeDelta::Seconds(1)); |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 609 | rtx_packets = 0; |
Tommi | 3c9bcc1 | 2020-04-15 16:45:47 +0200 | [diff] [blame] | 610 | int decoded_baseline = 0; |
| 611 | callee->SendTask([&decoded_baseline, &lossy]() { |
| 612 | decoded_baseline = lossy->receive()->GetStats().frames_decoded; |
| 613 | }); |
Danil Chapovalov | 0c626af | 2020-02-10 11:16:00 +0100 | [diff] [blame] | 614 | s.RunFor(TimeDelta::Seconds(1)); |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 615 | // We expect both that RTX packets were sent and that an appropriate number of |
| 616 | // frames were received. This is somewhat redundant but reduces the risk of |
| 617 | // false positives in future regressions (e.g. RTX is send due to probing). |
| 618 | EXPECT_GE(rtx_packets, 1); |
Tommi | 3c9bcc1 | 2020-04-15 16:45:47 +0200 | [diff] [blame] | 619 | int frames_decoded = 0; |
| 620 | callee->SendTask([&decoded_baseline, &frames_decoded, &lossy]() { |
| 621 | frames_decoded = |
| 622 | lossy->receive()->GetStats().frames_decoded - decoded_baseline; |
| 623 | }); |
Sebastian Jansson | 3d4d94a | 2020-01-14 14:25:41 +0100 | [diff] [blame] | 624 | EXPECT_EQ(frames_decoded, 5); |
| 625 | } |
| 626 | |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 627 | // Integration test verifying that retransmissions are sent for packets which |
| 628 | // can be detected as lost early, using transport wide feedback. |
| 629 | TEST(RtpVideoSenderTest, EarlyRetransmits) { |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 630 | RtpVideoSenderTestFixture test({kSsrc1, kSsrc2}, {kRtxSsrc1, kRtxSsrc2}, |
| 631 | kPayloadType, {}); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 632 | test.SetActiveModules({true, true}); |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 633 | |
Niels Möller | b9bfe65 | 2019-10-03 08:43:53 +0200 | [diff] [blame] | 634 | const uint8_t kPayload[1] = {'a'}; |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 635 | EncodedImage encoded_image; |
| 636 | encoded_image.SetTimestamp(1); |
| 637 | encoded_image.capture_time_ms_ = 2; |
| 638 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
Niels Möller | b9bfe65 | 2019-10-03 08:43:53 +0200 | [diff] [blame] | 639 | encoded_image.SetEncodedData( |
| 640 | EncodedImageBuffer::Create(kPayload, sizeof(kPayload))); |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 641 | encoded_image.SetSpatialIndex(0); |
| 642 | |
| 643 | CodecSpecificInfo codec_specific; |
| 644 | codec_specific.codecType = VideoCodecType::kVideoCodecGeneric; |
| 645 | |
| 646 | // Send two tiny images, mapping to single RTP packets. Capture sequence |
| 647 | // numbers. |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 648 | uint16_t frame1_rtp_sequence_number = 0; |
| 649 | uint16_t frame1_transport_sequence_number = 0; |
| 650 | EXPECT_CALL(test.transport(), SendRtp) |
Danil Chapovalov | a68eb8c | 2020-02-17 16:13:14 +0100 | [diff] [blame] | 651 | .WillOnce( |
| 652 | [&frame1_rtp_sequence_number, &frame1_transport_sequence_number]( |
| 653 | const uint8_t* packet, size_t length, |
| 654 | const PacketOptions& options) { |
| 655 | RtpPacket rtp_packet; |
| 656 | EXPECT_TRUE(rtp_packet.Parse(packet, length)); |
| 657 | frame1_rtp_sequence_number = rtp_packet.SequenceNumber(); |
| 658 | frame1_transport_sequence_number = options.packet_id; |
| 659 | EXPECT_EQ(rtp_packet.Ssrc(), kSsrc1); |
| 660 | return true; |
| 661 | }); |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 662 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 663 | EncodedImageCallback::Result::OK); |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 664 | |
Danil Chapovalov | 0c626af | 2020-02-10 11:16:00 +0100 | [diff] [blame] | 665 | test.AdvanceTime(TimeDelta::Millis(33)); |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 666 | |
| 667 | uint16_t frame2_rtp_sequence_number = 0; |
| 668 | uint16_t frame2_transport_sequence_number = 0; |
| 669 | encoded_image.SetSpatialIndex(1); |
| 670 | EXPECT_CALL(test.transport(), SendRtp) |
Danil Chapovalov | a68eb8c | 2020-02-17 16:13:14 +0100 | [diff] [blame] | 671 | .WillOnce( |
| 672 | [&frame2_rtp_sequence_number, &frame2_transport_sequence_number]( |
| 673 | const uint8_t* packet, size_t length, |
| 674 | const PacketOptions& options) { |
| 675 | RtpPacket rtp_packet; |
| 676 | EXPECT_TRUE(rtp_packet.Parse(packet, length)); |
| 677 | frame2_rtp_sequence_number = rtp_packet.SequenceNumber(); |
| 678 | frame2_transport_sequence_number = options.packet_id; |
| 679 | EXPECT_EQ(rtp_packet.Ssrc(), kSsrc2); |
| 680 | return true; |
| 681 | }); |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 682 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 683 | EncodedImageCallback::Result::OK); |
Danil Chapovalov | 0c626af | 2020-02-10 11:16:00 +0100 | [diff] [blame] | 684 | test.AdvanceTime(TimeDelta::Millis(33)); |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 685 | |
| 686 | EXPECT_NE(frame1_transport_sequence_number, frame2_transport_sequence_number); |
| 687 | |
| 688 | // Inject a transport feedback where the packet for the first frame is lost, |
| 689 | // expect a retransmission for it. |
| 690 | EXPECT_CALL(test.transport(), SendRtp) |
Danil Chapovalov | a68eb8c | 2020-02-17 16:13:14 +0100 | [diff] [blame] | 691 | .WillOnce([&frame1_rtp_sequence_number](const uint8_t* packet, |
| 692 | size_t length, |
| 693 | const PacketOptions& options) { |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 694 | RtpPacket rtp_packet; |
| 695 | EXPECT_TRUE(rtp_packet.Parse(packet, length)); |
Sebastian Jansson | f298855 | 2019-10-29 17:18:51 +0100 | [diff] [blame] | 696 | EXPECT_EQ(rtp_packet.Ssrc(), kRtxSsrc1); |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 697 | |
| 698 | // Retransmitted sequence number from the RTX header should match |
| 699 | // the lost packet. |
| 700 | rtc::ArrayView<const uint8_t> payload = rtp_packet.payload(); |
| 701 | EXPECT_EQ(ByteReader<uint16_t>::ReadBigEndian(payload.data()), |
| 702 | frame1_rtp_sequence_number); |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 703 | return true; |
| 704 | }); |
| 705 | |
Sebastian Jansson | f298855 | 2019-10-29 17:18:51 +0100 | [diff] [blame] | 706 | StreamFeedbackObserver::StreamPacketInfo first_packet_feedback; |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 707 | first_packet_feedback.rtp_sequence_number = frame1_rtp_sequence_number; |
| 708 | first_packet_feedback.ssrc = kSsrc1; |
Sebastian Jansson | f298855 | 2019-10-29 17:18:51 +0100 | [diff] [blame] | 709 | first_packet_feedback.received = false; |
Erik Språng | 6a0a559 | 2021-06-15 19:04:24 +0200 | [diff] [blame] | 710 | first_packet_feedback.is_retransmission = false; |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 711 | |
Sebastian Jansson | f298855 | 2019-10-29 17:18:51 +0100 | [diff] [blame] | 712 | StreamFeedbackObserver::StreamPacketInfo second_packet_feedback; |
| 713 | second_packet_feedback.rtp_sequence_number = frame2_rtp_sequence_number; |
| 714 | second_packet_feedback.ssrc = kSsrc2; |
| 715 | second_packet_feedback.received = true; |
Erik Språng | 6a0a559 | 2021-06-15 19:04:24 +0200 | [diff] [blame] | 716 | first_packet_feedback.is_retransmission = false; |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 717 | |
Sebastian Jansson | f298855 | 2019-10-29 17:18:51 +0100 | [diff] [blame] | 718 | test.router()->OnPacketFeedbackVector( |
| 719 | {first_packet_feedback, second_packet_feedback}); |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 720 | |
| 721 | // Wait for pacer to run and send the RTX packet. |
Danil Chapovalov | 0c626af | 2020-02-10 11:16:00 +0100 | [diff] [blame] | 722 | test.AdvanceTime(TimeDelta::Millis(33)); |
Erik Språng | 845c6aa | 2019-05-29 13:02:24 +0200 | [diff] [blame] | 723 | } |
Sebastian Jansson | cf41eb1 | 2019-06-10 11:30:59 +0200 | [diff] [blame] | 724 | |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 725 | TEST(RtpVideoSenderTest, SupportsDependencyDescriptor) { |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 726 | RtpVideoSenderTestFixture test({kSsrc1}, {}, kPayloadType, {}); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 727 | test.SetActiveModules({true}); |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 728 | |
| 729 | RtpHeaderExtensionMap extensions; |
| 730 | extensions.Register<RtpDependencyDescriptorExtension>( |
| 731 | kDependencyDescriptorExtensionId); |
| 732 | std::vector<RtpPacket> sent_packets; |
| 733 | ON_CALL(test.transport(), SendRtp) |
| 734 | .WillByDefault([&](const uint8_t* packet, size_t length, |
| 735 | const PacketOptions& options) { |
| 736 | sent_packets.emplace_back(&extensions); |
| 737 | EXPECT_TRUE(sent_packets.back().Parse(packet, length)); |
| 738 | return true; |
| 739 | }); |
| 740 | |
| 741 | const uint8_t kPayload[1] = {'a'}; |
| 742 | EncodedImage encoded_image; |
| 743 | encoded_image.SetTimestamp(1); |
| 744 | encoded_image.capture_time_ms_ = 2; |
| 745 | encoded_image.SetEncodedData( |
| 746 | EncodedImageBuffer::Create(kPayload, sizeof(kPayload))); |
| 747 | |
| 748 | CodecSpecificInfo codec_specific; |
| 749 | codec_specific.codecType = VideoCodecType::kVideoCodecGeneric; |
| 750 | codec_specific.template_structure.emplace(); |
| 751 | codec_specific.template_structure->num_decode_targets = 1; |
| 752 | codec_specific.template_structure->templates = { |
Danil Chapovalov | 24263f4 | 2020-06-11 13:23:45 +0200 | [diff] [blame] | 753 | FrameDependencyTemplate().T(0).Dtis("S"), |
| 754 | FrameDependencyTemplate().T(0).Dtis("S").FrameDiffs({2}), |
| 755 | FrameDependencyTemplate().T(1).Dtis("D").FrameDiffs({1}), |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 756 | }; |
| 757 | |
| 758 | // Send two tiny images, mapping to single RTP packets. |
| 759 | // Send in key frame. |
| 760 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
| 761 | codec_specific.generic_frame_info = |
| 762 | GenericFrameInfo::Builder().T(0).Dtis("S").Build(); |
| 763 | codec_specific.generic_frame_info->encoder_buffers = {{0, false, true}}; |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 764 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 765 | EncodedImageCallback::Result::OK); |
| 766 | test.AdvanceTime(TimeDelta::Millis(33)); |
| 767 | ASSERT_THAT(sent_packets, SizeIs(1)); |
| 768 | EXPECT_TRUE( |
| 769 | sent_packets.back().HasExtension<RtpDependencyDescriptorExtension>()); |
| 770 | |
| 771 | // Send in delta frame. |
| 772 | encoded_image._frameType = VideoFrameType::kVideoFrameDelta; |
| 773 | codec_specific.template_structure = absl::nullopt; |
| 774 | codec_specific.generic_frame_info = |
| 775 | GenericFrameInfo::Builder().T(1).Dtis("D").Build(); |
| 776 | codec_specific.generic_frame_info->encoder_buffers = {{0, true, false}}; |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 777 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 778 | EncodedImageCallback::Result::OK); |
| 779 | test.AdvanceTime(TimeDelta::Millis(33)); |
| 780 | ASSERT_THAT(sent_packets, SizeIs(2)); |
| 781 | EXPECT_TRUE( |
| 782 | sent_packets.back().HasExtension<RtpDependencyDescriptorExtension>()); |
| 783 | } |
| 784 | |
Emil Lundmark | 6c81a42 | 2022-05-18 17:13:34 +0200 | [diff] [blame] | 785 | TEST(RtpVideoSenderTest, |
| 786 | SupportsDependencyDescriptorForVp8NotProvidedByEncoder) { |
| 787 | constexpr uint8_t kPayload[1] = {'a'}; |
| 788 | RtpVideoSenderTestFixture test({kSsrc1}, {}, kPayloadType, {}); |
| 789 | RtpHeaderExtensionMap extensions; |
| 790 | extensions.Register<RtpDependencyDescriptorExtension>( |
| 791 | kDependencyDescriptorExtensionId); |
| 792 | std::vector<RtpPacket> sent_packets; |
| 793 | ON_CALL(test.transport(), SendRtp) |
| 794 | .WillByDefault( |
| 795 | [&](const uint8_t* packet, size_t length, const PacketOptions&) { |
| 796 | EXPECT_TRUE( |
| 797 | sent_packets.emplace_back(&extensions).Parse(packet, length)); |
| 798 | return true; |
| 799 | }); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 800 | test.SetActiveModules({true}); |
Emil Lundmark | 6c81a42 | 2022-05-18 17:13:34 +0200 | [diff] [blame] | 801 | |
| 802 | EncodedImage key_frame_image; |
| 803 | key_frame_image._frameType = VideoFrameType::kVideoFrameKey; |
| 804 | key_frame_image.SetEncodedData( |
| 805 | EncodedImageBuffer::Create(kPayload, sizeof(kPayload))); |
| 806 | CodecSpecificInfo key_frame_info; |
| 807 | key_frame_info.codecType = VideoCodecType::kVideoCodecVP8; |
| 808 | ASSERT_EQ( |
| 809 | test.router()->OnEncodedImage(key_frame_image, &key_frame_info).error, |
| 810 | EncodedImageCallback::Result::OK); |
| 811 | |
| 812 | EncodedImage delta_image; |
| 813 | delta_image._frameType = VideoFrameType::kVideoFrameDelta; |
| 814 | delta_image.SetEncodedData( |
| 815 | EncodedImageBuffer::Create(kPayload, sizeof(kPayload))); |
| 816 | CodecSpecificInfo delta_info; |
| 817 | delta_info.codecType = VideoCodecType::kVideoCodecVP8; |
| 818 | ASSERT_EQ(test.router()->OnEncodedImage(delta_image, &delta_info).error, |
| 819 | EncodedImageCallback::Result::OK); |
| 820 | |
| 821 | test.AdvanceTime(TimeDelta::Millis(123)); |
| 822 | |
| 823 | DependencyDescriptor key_frame_dd; |
| 824 | DependencyDescriptor delta_dd; |
| 825 | ASSERT_THAT(sent_packets, SizeIs(2)); |
| 826 | EXPECT_TRUE(sent_packets[0].GetExtension<RtpDependencyDescriptorExtension>( |
| 827 | /*structure=*/nullptr, &key_frame_dd)); |
| 828 | EXPECT_TRUE(sent_packets[1].GetExtension<RtpDependencyDescriptorExtension>( |
| 829 | key_frame_dd.attached_structure.get(), &delta_dd)); |
| 830 | } |
| 831 | |
Danil Chapovalov | 0be1846 | 2021-01-20 12:11:29 +0100 | [diff] [blame] | 832 | TEST(RtpVideoSenderTest, SupportsDependencyDescriptorForVp9) { |
| 833 | RtpVideoSenderTestFixture test({kSsrc1}, {}, kPayloadType, {}); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 834 | test.SetActiveModules({true}); |
Danil Chapovalov | 0be1846 | 2021-01-20 12:11:29 +0100 | [diff] [blame] | 835 | |
| 836 | RtpHeaderExtensionMap extensions; |
| 837 | extensions.Register<RtpDependencyDescriptorExtension>( |
| 838 | kDependencyDescriptorExtensionId); |
| 839 | std::vector<RtpPacket> sent_packets; |
| 840 | ON_CALL(test.transport(), SendRtp) |
| 841 | .WillByDefault([&](const uint8_t* packet, size_t length, |
| 842 | const PacketOptions& options) { |
| 843 | sent_packets.emplace_back(&extensions); |
| 844 | EXPECT_TRUE(sent_packets.back().Parse(packet, length)); |
| 845 | return true; |
| 846 | }); |
| 847 | |
| 848 | const uint8_t kPayload[1] = {'a'}; |
| 849 | EncodedImage encoded_image; |
| 850 | encoded_image.SetTimestamp(1); |
| 851 | encoded_image.capture_time_ms_ = 2; |
| 852 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
| 853 | encoded_image.SetEncodedData( |
| 854 | EncodedImageBuffer::Create(kPayload, sizeof(kPayload))); |
| 855 | |
| 856 | CodecSpecificInfo codec_specific; |
| 857 | codec_specific.codecType = VideoCodecType::kVideoCodecVP9; |
| 858 | codec_specific.template_structure.emplace(); |
| 859 | codec_specific.template_structure->num_decode_targets = 2; |
| 860 | codec_specific.template_structure->templates = { |
| 861 | FrameDependencyTemplate().S(0).Dtis("SS"), |
| 862 | FrameDependencyTemplate().S(1).Dtis("-S").FrameDiffs({1}), |
| 863 | }; |
| 864 | |
| 865 | // Send two tiny images, each mapping to single RTP packet. |
| 866 | // Send in key frame for the base spatial layer. |
| 867 | codec_specific.generic_frame_info = |
| 868 | GenericFrameInfo::Builder().S(0).Dtis("SS").Build(); |
| 869 | codec_specific.generic_frame_info->encoder_buffers = {{0, false, true}}; |
| 870 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
| 871 | EncodedImageCallback::Result::OK); |
| 872 | // Send in 2nd spatial layer. |
| 873 | codec_specific.template_structure = absl::nullopt; |
| 874 | codec_specific.generic_frame_info = |
| 875 | GenericFrameInfo::Builder().S(1).Dtis("-S").Build(); |
| 876 | codec_specific.generic_frame_info->encoder_buffers = {{0, true, false}, |
| 877 | {1, false, true}}; |
| 878 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
| 879 | EncodedImageCallback::Result::OK); |
| 880 | |
| 881 | test.AdvanceTime(TimeDelta::Millis(33)); |
| 882 | ASSERT_THAT(sent_packets, SizeIs(2)); |
| 883 | EXPECT_TRUE(sent_packets[0].HasExtension<RtpDependencyDescriptorExtension>()); |
| 884 | EXPECT_TRUE(sent_packets[1].HasExtension<RtpDependencyDescriptorExtension>()); |
| 885 | } |
| 886 | |
Danil Chapovalov | 748550d | 2021-04-29 11:42:54 +0200 | [diff] [blame] | 887 | TEST(RtpVideoSenderTest, |
| 888 | SupportsDependencyDescriptorForVp9NotProvidedByEncoder) { |
Danil Chapovalov | 748550d | 2021-04-29 11:42:54 +0200 | [diff] [blame] | 889 | RtpVideoSenderTestFixture test({kSsrc1}, {}, kPayloadType, {}); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 890 | test.SetActiveModules({true}); |
Danil Chapovalov | 748550d | 2021-04-29 11:42:54 +0200 | [diff] [blame] | 891 | |
| 892 | RtpHeaderExtensionMap extensions; |
| 893 | extensions.Register<RtpDependencyDescriptorExtension>( |
| 894 | kDependencyDescriptorExtensionId); |
| 895 | std::vector<RtpPacket> sent_packets; |
| 896 | ON_CALL(test.transport(), SendRtp) |
| 897 | .WillByDefault([&](const uint8_t* packet, size_t length, |
| 898 | const PacketOptions& options) { |
| 899 | sent_packets.emplace_back(&extensions); |
| 900 | EXPECT_TRUE(sent_packets.back().Parse(packet, length)); |
| 901 | return true; |
| 902 | }); |
| 903 | |
| 904 | const uint8_t kPayload[1] = {'a'}; |
| 905 | EncodedImage encoded_image; |
| 906 | encoded_image.SetTimestamp(1); |
| 907 | encoded_image.capture_time_ms_ = 2; |
| 908 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
| 909 | encoded_image._encodedWidth = 320; |
| 910 | encoded_image._encodedHeight = 180; |
| 911 | encoded_image.SetEncodedData( |
| 912 | EncodedImageBuffer::Create(kPayload, sizeof(kPayload))); |
| 913 | |
| 914 | CodecSpecificInfo codec_specific; |
| 915 | codec_specific.codecType = VideoCodecType::kVideoCodecVP9; |
| 916 | codec_specific.codecSpecific.VP9.num_spatial_layers = 1; |
| 917 | codec_specific.codecSpecific.VP9.temporal_idx = kNoTemporalIdx; |
| 918 | codec_specific.codecSpecific.VP9.first_frame_in_picture = true; |
| 919 | codec_specific.end_of_picture = true; |
| 920 | codec_specific.codecSpecific.VP9.inter_pic_predicted = false; |
| 921 | |
| 922 | // Send two tiny images, each mapping to single RTP packet. |
| 923 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
| 924 | EncodedImageCallback::Result::OK); |
| 925 | |
| 926 | // Send in 2nd picture. |
| 927 | encoded_image._frameType = VideoFrameType::kVideoFrameDelta; |
| 928 | encoded_image.SetTimestamp(3000); |
| 929 | codec_specific.codecSpecific.VP9.inter_pic_predicted = true; |
| 930 | codec_specific.codecSpecific.VP9.num_ref_pics = 1; |
| 931 | codec_specific.codecSpecific.VP9.p_diff[0] = 1; |
| 932 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
| 933 | EncodedImageCallback::Result::OK); |
| 934 | |
| 935 | test.AdvanceTime(TimeDelta::Millis(33)); |
| 936 | ASSERT_THAT(sent_packets, SizeIs(2)); |
| 937 | EXPECT_TRUE(sent_packets[0].HasExtension<RtpDependencyDescriptorExtension>()); |
| 938 | EXPECT_TRUE(sent_packets[1].HasExtension<RtpDependencyDescriptorExtension>()); |
| 939 | } |
| 940 | |
philipel | 5b231de | 2021-09-01 15:21:16 +0200 | [diff] [blame] | 941 | TEST(RtpVideoSenderTest, GenerateDependecyDescriptorForGenericCodecs) { |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 942 | test::ScopedKeyValueConfig field_trials( |
philipel | 5b231de | 2021-09-01 15:21:16 +0200 | [diff] [blame] | 943 | "WebRTC-GenericCodecDependencyDescriptor/Enabled/"); |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 944 | RtpVideoSenderTestFixture test({kSsrc1}, {}, kPayloadType, {}, &field_trials); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 945 | test.SetActiveModules({true}); |
philipel | 5b231de | 2021-09-01 15:21:16 +0200 | [diff] [blame] | 946 | |
| 947 | RtpHeaderExtensionMap extensions; |
| 948 | extensions.Register<RtpDependencyDescriptorExtension>( |
| 949 | kDependencyDescriptorExtensionId); |
| 950 | std::vector<RtpPacket> sent_packets; |
| 951 | ON_CALL(test.transport(), SendRtp) |
| 952 | .WillByDefault([&](const uint8_t* packet, size_t length, |
| 953 | const PacketOptions& options) { |
| 954 | sent_packets.emplace_back(&extensions); |
| 955 | EXPECT_TRUE(sent_packets.back().Parse(packet, length)); |
| 956 | return true; |
| 957 | }); |
| 958 | |
| 959 | const uint8_t kPayload[1] = {'a'}; |
| 960 | EncodedImage encoded_image; |
| 961 | encoded_image.SetTimestamp(1); |
| 962 | encoded_image.capture_time_ms_ = 2; |
| 963 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
| 964 | encoded_image._encodedWidth = 320; |
| 965 | encoded_image._encodedHeight = 180; |
| 966 | encoded_image.SetEncodedData( |
| 967 | EncodedImageBuffer::Create(kPayload, sizeof(kPayload))); |
| 968 | |
| 969 | CodecSpecificInfo codec_specific; |
| 970 | codec_specific.codecType = VideoCodecType::kVideoCodecGeneric; |
| 971 | codec_specific.end_of_picture = true; |
| 972 | |
| 973 | // Send two tiny images, each mapping to single RTP packet. |
| 974 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
| 975 | EncodedImageCallback::Result::OK); |
| 976 | |
| 977 | // Send in 2nd picture. |
| 978 | encoded_image._frameType = VideoFrameType::kVideoFrameDelta; |
| 979 | encoded_image.SetTimestamp(3000); |
| 980 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
| 981 | EncodedImageCallback::Result::OK); |
| 982 | |
| 983 | test.AdvanceTime(TimeDelta::Millis(33)); |
| 984 | ASSERT_THAT(sent_packets, SizeIs(2)); |
| 985 | EXPECT_TRUE(sent_packets[0].HasExtension<RtpDependencyDescriptorExtension>()); |
| 986 | EXPECT_TRUE(sent_packets[1].HasExtension<RtpDependencyDescriptorExtension>()); |
| 987 | } |
| 988 | |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 989 | TEST(RtpVideoSenderTest, SupportsStoppingUsingDependencyDescriptor) { |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 990 | RtpVideoSenderTestFixture test({kSsrc1}, {}, kPayloadType, {}); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 991 | test.SetActiveModules({true}); |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 992 | |
| 993 | RtpHeaderExtensionMap extensions; |
| 994 | extensions.Register<RtpDependencyDescriptorExtension>( |
| 995 | kDependencyDescriptorExtensionId); |
| 996 | std::vector<RtpPacket> sent_packets; |
| 997 | ON_CALL(test.transport(), SendRtp) |
| 998 | .WillByDefault([&](const uint8_t* packet, size_t length, |
| 999 | const PacketOptions& options) { |
| 1000 | sent_packets.emplace_back(&extensions); |
| 1001 | EXPECT_TRUE(sent_packets.back().Parse(packet, length)); |
| 1002 | return true; |
| 1003 | }); |
| 1004 | |
| 1005 | const uint8_t kPayload[1] = {'a'}; |
| 1006 | EncodedImage encoded_image; |
| 1007 | encoded_image.SetTimestamp(1); |
| 1008 | encoded_image.capture_time_ms_ = 2; |
| 1009 | encoded_image.SetEncodedData( |
| 1010 | EncodedImageBuffer::Create(kPayload, sizeof(kPayload))); |
| 1011 | |
| 1012 | CodecSpecificInfo codec_specific; |
| 1013 | codec_specific.codecType = VideoCodecType::kVideoCodecGeneric; |
| 1014 | codec_specific.template_structure.emplace(); |
| 1015 | codec_specific.template_structure->num_decode_targets = 1; |
| 1016 | codec_specific.template_structure->templates = { |
Danil Chapovalov | 24263f4 | 2020-06-11 13:23:45 +0200 | [diff] [blame] | 1017 | FrameDependencyTemplate().T(0).Dtis("S"), |
| 1018 | FrameDependencyTemplate().T(0).Dtis("S").FrameDiffs({2}), |
| 1019 | FrameDependencyTemplate().T(1).Dtis("D").FrameDiffs({1}), |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 1020 | }; |
| 1021 | |
| 1022 | // Send two tiny images, mapping to single RTP packets. |
| 1023 | // Send in a key frame. |
| 1024 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
| 1025 | codec_specific.generic_frame_info = |
| 1026 | GenericFrameInfo::Builder().T(0).Dtis("S").Build(); |
| 1027 | codec_specific.generic_frame_info->encoder_buffers = {{0, false, true}}; |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 1028 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 1029 | EncodedImageCallback::Result::OK); |
| 1030 | test.AdvanceTime(TimeDelta::Millis(33)); |
| 1031 | ASSERT_THAT(sent_packets, SizeIs(1)); |
| 1032 | EXPECT_TRUE( |
| 1033 | sent_packets.back().HasExtension<RtpDependencyDescriptorExtension>()); |
| 1034 | |
| 1035 | // Send in a new key frame without the support for the dependency descriptor. |
| 1036 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
| 1037 | codec_specific.template_structure = absl::nullopt; |
Danil Chapovalov | 383f2cf | 2020-08-12 11:06:53 +0200 | [diff] [blame] | 1038 | EXPECT_EQ(test.router()->OnEncodedImage(encoded_image, &codec_specific).error, |
Danil Chapovalov | 2272f20 | 2020-02-18 12:09:43 +0100 | [diff] [blame] | 1039 | EncodedImageCallback::Result::OK); |
| 1040 | test.AdvanceTime(TimeDelta::Millis(33)); |
| 1041 | ASSERT_THAT(sent_packets, SizeIs(2)); |
| 1042 | EXPECT_FALSE( |
| 1043 | sent_packets.back().HasExtension<RtpDependencyDescriptorExtension>()); |
| 1044 | } |
| 1045 | |
Jakob Ivarsson | 9a12ee5 | 2020-11-26 16:04:18 +0100 | [diff] [blame] | 1046 | TEST(RtpVideoSenderTest, CanSetZeroBitrate) { |
Sebastian Jansson | cf41eb1 | 2019-06-10 11:30:59 +0200 | [diff] [blame] | 1047 | RtpVideoSenderTestFixture test({kSsrc1}, {kRtxSsrc1}, kPayloadType, {}); |
Jakob Ivarsson | 9a12ee5 | 2020-11-26 16:04:18 +0100 | [diff] [blame] | 1048 | test.router()->OnBitrateUpdated(CreateBitrateAllocationUpdate(0), |
| 1049 | /*framerate*/ 0); |
Sebastian Jansson | cf41eb1 | 2019-06-10 11:30:59 +0200 | [diff] [blame] | 1050 | } |
Marina Ciocea | dc69fd2 | 2020-04-10 20:19:14 +0200 | [diff] [blame] | 1051 | |
| 1052 | TEST(RtpVideoSenderTest, SimulcastSenderRegistersFrameTransformers) { |
Marina Ciocea | dc69fd2 | 2020-04-10 20:19:14 +0200 | [diff] [blame] | 1053 | rtc::scoped_refptr<MockFrameTransformer> transformer = |
Tomas Gunnarsson | c1d5891 | 2021-04-22 19:21:43 +0200 | [diff] [blame] | 1054 | rtc::make_ref_counted<MockFrameTransformer>(); |
Marina Ciocea | dc69fd2 | 2020-04-10 20:19:14 +0200 | [diff] [blame] | 1055 | |
| 1056 | EXPECT_CALL(*transformer, RegisterTransformedFrameSinkCallback(_, kSsrc1)); |
| 1057 | EXPECT_CALL(*transformer, RegisterTransformedFrameSinkCallback(_, kSsrc2)); |
| 1058 | RtpVideoSenderTestFixture test({kSsrc1, kSsrc2}, {kRtxSsrc1, kRtxSsrc2}, |
| 1059 | kPayloadType, {}, nullptr, transformer); |
| 1060 | |
| 1061 | EXPECT_CALL(*transformer, UnregisterTransformedFrameSinkCallback(kSsrc1)); |
| 1062 | EXPECT_CALL(*transformer, UnregisterTransformedFrameSinkCallback(kSsrc2)); |
| 1063 | } |
Jakob Ivarsson | 9a12ee5 | 2020-11-26 16:04:18 +0100 | [diff] [blame] | 1064 | |
| 1065 | TEST(RtpVideoSenderTest, OverheadIsSubtractedFromTargetBitrate) { |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 1066 | test::ScopedKeyValueConfig field_trials( |
Jakob Ivarsson | 9a12ee5 | 2020-11-26 16:04:18 +0100 | [diff] [blame] | 1067 | "WebRTC-Video-UseFrameRateForOverhead/Enabled/"); |
| 1068 | |
| 1069 | // TODO(jakobi): RTP header size should not be hard coded. |
| 1070 | constexpr uint32_t kRtpHeaderSizeBytes = 20; |
| 1071 | constexpr uint32_t kTransportPacketOverheadBytes = 40; |
| 1072 | constexpr uint32_t kOverheadPerPacketBytes = |
| 1073 | kRtpHeaderSizeBytes + kTransportPacketOverheadBytes; |
Jonas Oreland | c7f691a | 2022-03-09 15:12:07 +0100 | [diff] [blame] | 1074 | RtpVideoSenderTestFixture test({kSsrc1}, {}, kPayloadType, {}, &field_trials); |
Jakob Ivarsson | 9a12ee5 | 2020-11-26 16:04:18 +0100 | [diff] [blame] | 1075 | test.router()->OnTransportOverheadChanged(kTransportPacketOverheadBytes); |
Per Kjellander | 59ade01 | 2022-12-02 08:09:37 +0000 | [diff] [blame] | 1076 | test.SetActiveModules({true}); |
Jakob Ivarsson | 9a12ee5 | 2020-11-26 16:04:18 +0100 | [diff] [blame] | 1077 | |
| 1078 | { |
| 1079 | test.router()->OnBitrateUpdated(CreateBitrateAllocationUpdate(300000), |
| 1080 | /*framerate*/ 15); |
| 1081 | // 1 packet per frame. |
| 1082 | EXPECT_EQ(test.router()->GetPayloadBitrateBps(), |
| 1083 | 300000 - kOverheadPerPacketBytes * 8 * 30); |
| 1084 | } |
| 1085 | { |
| 1086 | test.router()->OnBitrateUpdated(CreateBitrateAllocationUpdate(150000), |
| 1087 | /*framerate*/ 15); |
| 1088 | // 1 packet per frame. |
| 1089 | EXPECT_EQ(test.router()->GetPayloadBitrateBps(), |
| 1090 | 150000 - kOverheadPerPacketBytes * 8 * 15); |
| 1091 | } |
| 1092 | { |
| 1093 | test.router()->OnBitrateUpdated(CreateBitrateAllocationUpdate(1000000), |
| 1094 | /*framerate*/ 30); |
| 1095 | // 3 packets per frame. |
| 1096 | EXPECT_EQ(test.router()->GetPayloadBitrateBps(), |
| 1097 | 1000000 - kOverheadPerPacketBytes * 8 * 30 * 3); |
| 1098 | } |
| 1099 | } |
| 1100 | |
Erik Språng | 1b11b58 | 2022-12-09 21:38:44 +0100 | [diff] [blame] | 1101 | TEST(RtpVideoSenderTest, ClearsPendingPacketsOnInactivation) { |
| 1102 | RtpVideoSenderTestFixture test({kSsrc1}, {kRtxSsrc1}, kPayloadType, {}); |
| 1103 | test.SetActiveModules({true}); |
| 1104 | |
| 1105 | RtpHeaderExtensionMap extensions; |
| 1106 | extensions.Register<RtpDependencyDescriptorExtension>( |
| 1107 | kDependencyDescriptorExtensionId); |
| 1108 | std::vector<RtpPacket> sent_packets; |
| 1109 | ON_CALL(test.transport(), SendRtp) |
| 1110 | .WillByDefault([&](const uint8_t* packet, size_t length, |
| 1111 | const PacketOptions& options) { |
| 1112 | sent_packets.emplace_back(&extensions); |
| 1113 | EXPECT_TRUE(sent_packets.back().Parse(packet, length)); |
| 1114 | return true; |
| 1115 | }); |
| 1116 | |
| 1117 | // Set a very low bitrate. |
| 1118 | test.router()->OnBitrateUpdated( |
| 1119 | CreateBitrateAllocationUpdate(/*rate_bps=*/30'000), |
| 1120 | /*framerate=*/30); |
| 1121 | |
| 1122 | // Create and send a large keyframe. |
| 1123 | const size_t kImageSizeBytes = 10000; |
| 1124 | constexpr uint8_t kPayload[kImageSizeBytes] = {'a'}; |
| 1125 | EncodedImage encoded_image; |
| 1126 | encoded_image.SetTimestamp(1); |
| 1127 | encoded_image.capture_time_ms_ = 2; |
| 1128 | encoded_image._frameType = VideoFrameType::kVideoFrameKey; |
| 1129 | encoded_image.SetEncodedData( |
| 1130 | EncodedImageBuffer::Create(kPayload, sizeof(kPayload))); |
| 1131 | EXPECT_EQ(test.router() |
| 1132 | ->OnEncodedImage(encoded_image, /*codec_specific=*/nullptr) |
| 1133 | .error, |
| 1134 | EncodedImageCallback::Result::OK); |
| 1135 | |
| 1136 | // Advance time a small amount, check that sent data is only part of the |
| 1137 | // image. |
| 1138 | test.AdvanceTime(TimeDelta::Millis(5)); |
| 1139 | DataSize transmittedPayload = DataSize::Zero(); |
| 1140 | for (const RtpPacket& packet : sent_packets) { |
| 1141 | transmittedPayload += DataSize::Bytes(packet.payload_size()); |
| 1142 | // Make sure we don't see the end of the frame. |
| 1143 | EXPECT_FALSE(packet.Marker()); |
| 1144 | } |
| 1145 | EXPECT_GT(transmittedPayload, DataSize::Zero()); |
| 1146 | EXPECT_LT(transmittedPayload, DataSize::Bytes(kImageSizeBytes / 4)); |
| 1147 | |
| 1148 | // Record the RTP timestamp of the first frame. |
| 1149 | const uint32_t first_frame_timestamp = sent_packets[0].Timestamp(); |
| 1150 | sent_packets.clear(); |
| 1151 | |
| 1152 | // Disable the sending module and advance time slightly. No packets should be |
| 1153 | // sent. |
| 1154 | test.SetActiveModules({false}); |
| 1155 | test.AdvanceTime(TimeDelta::Millis(20)); |
| 1156 | EXPECT_TRUE(sent_packets.empty()); |
| 1157 | |
| 1158 | // Reactive the send module - any packets should have been removed, so nothing |
| 1159 | // should be transmitted. |
| 1160 | test.SetActiveModules({true}); |
| 1161 | test.AdvanceTime(TimeDelta::Millis(33)); |
| 1162 | EXPECT_TRUE(sent_packets.empty()); |
| 1163 | |
| 1164 | // Send a new frame. |
| 1165 | encoded_image.SetTimestamp(3); |
| 1166 | encoded_image.capture_time_ms_ = 4; |
| 1167 | EXPECT_EQ(test.router() |
| 1168 | ->OnEncodedImage(encoded_image, /*codec_specific=*/nullptr) |
| 1169 | .error, |
| 1170 | EncodedImageCallback::Result::OK); |
| 1171 | test.AdvanceTime(TimeDelta::Millis(33)); |
| 1172 | |
| 1173 | // Advance time, check we get new packets - but only for the second frame. |
| 1174 | EXPECT_FALSE(sent_packets.empty()); |
| 1175 | EXPECT_NE(sent_packets[0].Timestamp(), first_frame_timestamp); |
| 1176 | } |
| 1177 | |
mflodman@webrtc.org | 02270cd | 2015-02-06 13:10:19 +0000 | [diff] [blame] | 1178 | } // namespace webrtc |