blob: e5861ea81536a4c911ee8e4135935f7322604923 [file] [log] [blame]
Henrik Boström933d8b02017-10-10 10:05:16 -07001/*
2 * Copyright 2017 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
Harald Alvestrandc24a2182022-02-23 13:44:59 +000011#include <stddef.h>
Jonas Olssona4d87372019-07-05 19:08:33 +020012
Harald Alvestrandc24a2182022-02-23 13:44:59 +000013#include <cstdint>
Henrik Boström933d8b02017-10-10 10:05:16 -070014#include <memory>
Yves Gerey3e707812018-11-28 16:47:49 +010015#include <string>
16#include <utility>
Henrik Boström933d8b02017-10-10 10:05:16 -070017#include <vector>
18
Yves Gerey3e707812018-11-28 16:47:49 +010019#include "absl/types/optional.h"
20#include "api/audio/audio_mixer.h"
Karl Wiberg1b0eae32017-10-17 14:48:54 +020021#include "api/audio_codecs/builtin_audio_decoder_factory.h"
22#include "api/audio_codecs/builtin_audio_encoder_factory.h"
Mirko Bonadei2ff3f492018-11-22 09:00:13 +010023#include "api/create_peerconnection_factory.h"
Henrik Boström933d8b02017-10-10 10:05:16 -070024#include "api/jsep.h"
Steve Anton10542f22019-01-11 09:11:00 -080025#include "api/media_stream_interface.h"
26#include "api/media_types.h"
27#include "api/peer_connection_interface.h"
28#include "api/rtc_error.h"
29#include "api/rtp_parameters.h"
30#include "api/rtp_receiver_interface.h"
31#include "api/rtp_sender_interface.h"
Harald Alvestrandc24a2182022-02-23 13:44:59 +000032#include "api/rtp_transceiver_direction.h"
Steve Anton10542f22019-01-11 09:11:00 -080033#include "api/rtp_transceiver_interface.h"
Mirko Bonadeid9708072019-01-25 20:26:48 +010034#include "api/scoped_refptr.h"
Steve Anton10542f22019-01-11 09:11:00 -080035#include "api/set_remote_description_observer_interface.h"
36#include "api/uma_metrics.h"
Anders Carlsson67537952018-05-03 11:28:29 +020037#include "api/video_codecs/builtin_video_decoder_factory.h"
38#include "api/video_codecs/builtin_video_encoder_factory.h"
Steve Anton10542f22019-01-11 09:11:00 -080039#include "media/base/stream_params.h"
Yves Gerey3e707812018-11-28 16:47:49 +010040#include "modules/audio_device/include/audio_device.h"
41#include "modules/audio_processing/include/audio_processing.h"
Steve Anton10542f22019-01-11 09:11:00 -080042#include "p2p/base/port_allocator.h"
43#include "pc/media_session.h"
44#include "pc/peer_connection_wrapper.h"
45#include "pc/sdp_utils.h"
46#include "pc/session_description.h"
47#include "pc/test/fake_audio_capture_module.h"
48#include "pc/test/mock_peer_connection_observers.h"
Henrik Boström933d8b02017-10-10 10:05:16 -070049#include "rtc_base/checks.h"
50#include "rtc_base/gunit.h"
Steve Anton10542f22019-01-11 09:11:00 -080051#include "rtc_base/rtc_certificate_generator.h"
Henrik Boström933d8b02017-10-10 10:05:16 -070052#include "rtc_base/thread.h"
Mirko Bonadei17f48782018-09-28 08:51:10 +020053#include "system_wrappers/include/metrics.h"
Steve Anton9158ef62017-11-27 13:01:52 -080054#include "test/gmock.h"
Yves Gerey3e707812018-11-28 16:47:49 +010055#include "test/gtest.h"
Henrik Boström933d8b02017-10-10 10:05:16 -070056
57// This file contains tests for RTP Media API-related behavior of
Artem Titovcfea2182021-08-10 01:22:31 +020058// `webrtc::PeerConnection`, see https://w3c.github.io/webrtc-pc/#rtp-media-api.
Henrik Boström933d8b02017-10-10 10:05:16 -070059
Steve Anton9158ef62017-11-27 13:01:52 -080060namespace webrtc {
61
62using RTCConfiguration = PeerConnectionInterface::RTCConfiguration;
63using ::testing::ElementsAre;
Steve Antonc1e6e862019-03-04 14:43:44 -080064using ::testing::Pair;
Steve Anton9158ef62017-11-27 13:01:52 -080065using ::testing::UnorderedElementsAre;
Steve Anton3172c032018-05-03 15:30:18 -070066using ::testing::Values;
Henrik Boström933d8b02017-10-10 10:05:16 -070067
Henrik Boström31638672017-11-23 17:48:32 +010068const uint32_t kDefaultTimeout = 10000u;
69
70template <typename MethodFunctor>
Niels Möller9dde1202022-03-21 10:36:32 +010071class OnSuccessObserver : public webrtc::SetRemoteDescriptionObserverInterface {
Henrik Boström31638672017-11-23 17:48:32 +010072 public:
73 explicit OnSuccessObserver(MethodFunctor on_success)
74 : on_success_(std::move(on_success)) {}
75
76 // webrtc::SetRemoteDescriptionObserverInterface implementation.
77 void OnSetRemoteDescriptionComplete(webrtc::RTCError error) override {
78 RTC_CHECK(error.ok());
79 on_success_();
80 }
81
82 private:
83 MethodFunctor on_success_;
84};
85
Mirko Bonadei6a489f22019-04-09 15:11:12 +020086class PeerConnectionRtpBaseTest : public ::testing::Test {
Henrik Boström933d8b02017-10-10 10:05:16 -070087 public:
Steve Anton3172c032018-05-03 15:30:18 -070088 explicit PeerConnectionRtpBaseTest(SdpSemantics sdp_semantics)
89 : sdp_semantics_(sdp_semantics),
90 pc_factory_(
Steve Anton9158ef62017-11-27 13:01:52 -080091 CreatePeerConnectionFactory(rtc::Thread::Current(),
92 rtc::Thread::Current(),
93 rtc::Thread::Current(),
94 FakeAudioCaptureModule::Create(),
95 CreateBuiltinAudioEncoderFactory(),
96 CreateBuiltinAudioDecoderFactory(),
Anders Carlsson67537952018-05-03 11:28:29 +020097 CreateBuiltinVideoEncoderFactory(),
98 CreateBuiltinVideoDecoderFactory(),
99 nullptr /* audio_mixer */,
Qingsi Wang7fc821d2018-07-12 12:54:53 -0700100 nullptr /* audio_processing */)) {
101 webrtc::metrics::Reset();
102 }
Henrik Boström933d8b02017-10-10 10:05:16 -0700103
Steve Anton9158ef62017-11-27 13:01:52 -0800104 std::unique_ptr<PeerConnectionWrapper> CreatePeerConnection() {
105 return CreatePeerConnection(RTCConfiguration());
106 }
107
Steve Antone831b8c2018-02-01 12:22:16 -0800108 std::unique_ptr<PeerConnectionWrapper> CreatePeerConnectionWithPlanB() {
109 RTCConfiguration config;
Florent Castelli15a38de2022-04-06 00:38:21 +0200110 config.sdp_semantics = SdpSemantics::kPlanB_DEPRECATED;
Steve Anton3172c032018-05-03 15:30:18 -0700111 return CreatePeerConnectionInternal(config);
Steve Antone831b8c2018-02-01 12:22:16 -0800112 }
113
Steve Anton9158ef62017-11-27 13:01:52 -0800114 std::unique_ptr<PeerConnectionWrapper> CreatePeerConnectionWithUnifiedPlan() {
115 RTCConfiguration config;
116 config.sdp_semantics = SdpSemantics::kUnifiedPlan;
Steve Anton3172c032018-05-03 15:30:18 -0700117 return CreatePeerConnectionInternal(config);
Steve Anton9158ef62017-11-27 13:01:52 -0800118 }
119
120 std::unique_ptr<PeerConnectionWrapper> CreatePeerConnection(
121 const RTCConfiguration& config) {
Steve Anton3172c032018-05-03 15:30:18 -0700122 RTCConfiguration modified_config = config;
123 modified_config.sdp_semantics = sdp_semantics_;
124 return CreatePeerConnectionInternal(modified_config);
125 }
126
127 protected:
128 const SdpSemantics sdp_semantics_;
129 rtc::scoped_refptr<PeerConnectionFactoryInterface> pc_factory_;
130
131 private:
132 // Private so that tests don't accidentally bypass the SdpSemantics
133 // adjustment.
134 std::unique_ptr<PeerConnectionWrapper> CreatePeerConnectionInternal(
135 const RTCConfiguration& config) {
Mirko Bonadei317a1f02019-09-17 17:06:18 +0200136 auto observer = std::make_unique<MockPeerConnectionObserver>();
Florent Castelli72424402022-04-06 03:45:10 +0200137 auto result = pc_factory_->CreatePeerConnectionOrError(
138 config, PeerConnectionDependencies(observer.get()));
139 EXPECT_TRUE(result.ok());
Niels Möllerafb246b2022-04-20 14:26:50 +0200140 observer->SetPeerConnectionInterface(result.value().get());
Florent Castelli72424402022-04-06 03:45:10 +0200141 return std::make_unique<PeerConnectionWrapper>(
142 pc_factory_, result.MoveValue(), std::move(observer));
Henrik Boström933d8b02017-10-10 10:05:16 -0700143 }
Niels Möller83830f32022-05-20 09:12:57 +0200144
145 rtc::AutoThread main_thread_;
Steve Anton3172c032018-05-03 15:30:18 -0700146};
Henrik Boström933d8b02017-10-10 10:05:16 -0700147
Steve Anton3172c032018-05-03 15:30:18 -0700148class PeerConnectionRtpTest
149 : public PeerConnectionRtpBaseTest,
150 public ::testing::WithParamInterface<SdpSemantics> {
Henrik Boström933d8b02017-10-10 10:05:16 -0700151 protected:
Steve Anton3172c032018-05-03 15:30:18 -0700152 PeerConnectionRtpTest() : PeerConnectionRtpBaseTest(GetParam()) {}
153};
154
155class PeerConnectionRtpTestPlanB : public PeerConnectionRtpBaseTest {
156 protected:
157 PeerConnectionRtpTestPlanB()
Florent Castelli15a38de2022-04-06 00:38:21 +0200158 : PeerConnectionRtpBaseTest(SdpSemantics::kPlanB_DEPRECATED) {}
Steve Anton3172c032018-05-03 15:30:18 -0700159};
160
161class PeerConnectionRtpTestUnifiedPlan : public PeerConnectionRtpBaseTest {
162 protected:
163 PeerConnectionRtpTestUnifiedPlan()
164 : PeerConnectionRtpBaseTest(SdpSemantics::kUnifiedPlan) {}
Harald Alvestrand09bd9ba2020-10-09 08:13:30 +0000165
166 // Helper to emulate an SFU that rejects an offered media section
167 // in answer.
168 bool ExchangeOfferAnswerWhereRemoteStopsTransceiver(
169 PeerConnectionWrapper* caller,
170 PeerConnectionWrapper* callee,
171 size_t mid_to_stop) {
172 auto offer = caller->CreateOffer();
173 caller->SetLocalDescription(CloneSessionDescription(offer.get()));
174 callee->SetRemoteDescription(std::move(offer));
175 EXPECT_LT(mid_to_stop, callee->pc()->GetTransceivers().size());
176 // Must use StopInternal in order to do instant reject.
177 callee->pc()->GetTransceivers()[mid_to_stop]->StopInternal();
178 auto answer = callee->CreateAnswer();
179 EXPECT_TRUE(answer);
180 bool set_local_answer =
181 callee->SetLocalDescription(CloneSessionDescription(answer.get()));
182 EXPECT_TRUE(set_local_answer);
183 bool set_remote_answer = caller->SetRemoteDescription(std::move(answer));
184 EXPECT_TRUE(set_remote_answer);
185 return set_remote_answer;
186 }
Henrik Boström933d8b02017-10-10 10:05:16 -0700187};
188
Artem Titovcfea2182021-08-10 01:22:31 +0200189// These tests cover `webrtc::PeerConnectionObserver` callbacks firing upon
Henrik Boström31638672017-11-23 17:48:32 +0100190// setting the remote description.
Henrik Boström31638672017-11-23 17:48:32 +0100191
Steve Anton3172c032018-05-03 15:30:18 -0700192TEST_P(PeerConnectionRtpTest, AddTrackWithoutStreamFiresOnAddTrack) {
Henrik Boström933d8b02017-10-10 10:05:16 -0700193 auto caller = CreatePeerConnection();
194 auto callee = CreatePeerConnection();
195
Steve Anton3172c032018-05-03 15:30:18 -0700196 ASSERT_TRUE(caller->AddAudioTrack("audio_track"));
Seth Hampson5897a6e2018-04-03 11:16:33 -0700197 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström933d8b02017-10-10 10:05:16 -0700198
Henrik Boström31638672017-11-23 17:48:32 +0100199 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Steve Anton3172c032018-05-03 15:30:18 -0700200 const auto& add_track_event = callee->observer()->add_track_events_[0];
Henrik Boström9e6fd2b2017-11-21 13:41:51 +0100201 EXPECT_EQ(add_track_event.streams, add_track_event.receiver->streams());
Steve Anton3172c032018-05-03 15:30:18 -0700202
Florent Castelli15a38de2022-04-06 00:38:21 +0200203 if (sdp_semantics_ == SdpSemantics::kPlanB_DEPRECATED) {
Steve Anton3172c032018-05-03 15:30:18 -0700204 // Since we are not supporting the no stream case with Plan B, there should
205 // be a generated stream, even though we didn't set one with AddTrack.
206 ASSERT_EQ(1u, add_track_event.streams.size());
207 EXPECT_TRUE(add_track_event.streams[0]->FindAudioTrack("audio_track"));
208 } else {
209 EXPECT_EQ(0u, add_track_event.streams.size());
210 }
Henrik Boström933d8b02017-10-10 10:05:16 -0700211}
212
Steve Anton3172c032018-05-03 15:30:18 -0700213TEST_P(PeerConnectionRtpTest, AddTrackWithStreamFiresOnAddTrack) {
Henrik Boström933d8b02017-10-10 10:05:16 -0700214 auto caller = CreatePeerConnection();
215 auto callee = CreatePeerConnection();
216
Steve Anton3172c032018-05-03 15:30:18 -0700217 ASSERT_TRUE(caller->AddAudioTrack("audio_track", {"audio_stream"}));
Seth Hampson5897a6e2018-04-03 11:16:33 -0700218 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström933d8b02017-10-10 10:05:16 -0700219
Henrik Boström31638672017-11-23 17:48:32 +0100220 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Henrik Boström9e6fd2b2017-11-21 13:41:51 +0100221 auto& add_track_event = callee->observer()->add_track_events_[0];
Henrik Boström31638672017-11-23 17:48:32 +0100222 ASSERT_EQ(add_track_event.streams.size(), 1u);
Seth Hampson13b8bad2018-03-13 16:05:28 -0700223 EXPECT_EQ("audio_stream", add_track_event.streams[0]->id());
Henrik Boström9e6fd2b2017-11-21 13:41:51 +0100224 EXPECT_TRUE(add_track_event.streams[0]->FindAudioTrack("audio_track"));
225 EXPECT_EQ(add_track_event.streams, add_track_event.receiver->streams());
Henrik Boström933d8b02017-10-10 10:05:16 -0700226}
227
Steve Anton3172c032018-05-03 15:30:18 -0700228TEST_P(PeerConnectionRtpTest, RemoveTrackWithoutStreamFiresOnRemoveTrack) {
Henrik Boström933d8b02017-10-10 10:05:16 -0700229 auto caller = CreatePeerConnection();
230 auto callee = CreatePeerConnection();
231
Steve Anton3172c032018-05-03 15:30:18 -0700232 auto sender = caller->AddAudioTrack("audio_track", {});
Seth Hampson5897a6e2018-04-03 11:16:33 -0700233 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström31638672017-11-23 17:48:32 +0100234 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Steve Anton3172c032018-05-03 15:30:18 -0700235 ASSERT_TRUE(
236 caller->SetRemoteDescription(callee->CreateAnswerAndSetAsLocal()));
237
Harald Alvestrand93dd7632022-01-19 12:28:45 +0000238 EXPECT_TRUE(caller->pc()->RemoveTrackOrError(sender).ok());
Seth Hampson5897a6e2018-04-03 11:16:33 -0700239 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström933d8b02017-10-10 10:05:16 -0700240
Henrik Boström31638672017-11-23 17:48:32 +0100241 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700242 EXPECT_EQ(callee->observer()->GetAddTrackReceivers(),
243 callee->observer()->remove_track_events_);
244}
245
Steve Anton3172c032018-05-03 15:30:18 -0700246TEST_P(PeerConnectionRtpTest, RemoveTrackWithStreamFiresOnRemoveTrack) {
Henrik Boström933d8b02017-10-10 10:05:16 -0700247 auto caller = CreatePeerConnection();
248 auto callee = CreatePeerConnection();
249
Steve Anton3172c032018-05-03 15:30:18 -0700250 auto sender = caller->AddAudioTrack("audio_track", {"audio_stream"});
Seth Hampson5897a6e2018-04-03 11:16:33 -0700251 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström31638672017-11-23 17:48:32 +0100252 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Steve Anton3172c032018-05-03 15:30:18 -0700253 ASSERT_TRUE(
254 caller->SetRemoteDescription(callee->CreateAnswerAndSetAsLocal()));
255
Harald Alvestrand93dd7632022-01-19 12:28:45 +0000256 EXPECT_TRUE(caller->pc()->RemoveTrackOrError(sender).ok());
Seth Hampson5897a6e2018-04-03 11:16:33 -0700257 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström933d8b02017-10-10 10:05:16 -0700258
Henrik Boström31638672017-11-23 17:48:32 +0100259 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700260 EXPECT_EQ(callee->observer()->GetAddTrackReceivers(),
261 callee->observer()->remove_track_events_);
Steve Anton3172c032018-05-03 15:30:18 -0700262 EXPECT_EQ(0u, callee->observer()->remote_streams()->count());
Henrik Boström933d8b02017-10-10 10:05:16 -0700263}
264
Steve Anton3172c032018-05-03 15:30:18 -0700265TEST_P(PeerConnectionRtpTest, RemoveTrackWithSharedStreamFiresOnRemoveTrack) {
Henrik Boström933d8b02017-10-10 10:05:16 -0700266 auto caller = CreatePeerConnection();
267 auto callee = CreatePeerConnection();
268
Seth Hampson845e8782018-03-02 11:34:10 -0800269 const char kSharedStreamId[] = "shared_audio_stream";
Steve Anton3172c032018-05-03 15:30:18 -0700270 auto sender1 = caller->AddAudioTrack("audio_track1", {kSharedStreamId});
271 auto sender2 = caller->AddAudioTrack("audio_track2", {kSharedStreamId});
Seth Hampson5897a6e2018-04-03 11:16:33 -0700272 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström31638672017-11-23 17:48:32 +0100273 ASSERT_EQ(callee->observer()->add_track_events_.size(), 2u);
Steve Anton3172c032018-05-03 15:30:18 -0700274 ASSERT_TRUE(
275 caller->SetRemoteDescription(callee->CreateAnswerAndSetAsLocal()));
Henrik Boström933d8b02017-10-10 10:05:16 -0700276
277 // Remove "audio_track1".
Harald Alvestrand93dd7632022-01-19 12:28:45 +0000278 EXPECT_TRUE(caller->pc()->RemoveTrackOrError(sender1).ok());
Seth Hampson5897a6e2018-04-03 11:16:33 -0700279 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström31638672017-11-23 17:48:32 +0100280 ASSERT_EQ(callee->observer()->add_track_events_.size(), 2u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700281 EXPECT_EQ(
Steve Anton9158ef62017-11-27 13:01:52 -0800282 std::vector<rtc::scoped_refptr<RtpReceiverInterface>>{
Henrik Boström933d8b02017-10-10 10:05:16 -0700283 callee->observer()->add_track_events_[0].receiver},
284 callee->observer()->remove_track_events_);
Steve Anton3172c032018-05-03 15:30:18 -0700285 ASSERT_EQ(1u, callee->observer()->remote_streams()->count());
286 ASSERT_TRUE(
287 caller->SetRemoteDescription(callee->CreateAnswerAndSetAsLocal()));
Henrik Boström933d8b02017-10-10 10:05:16 -0700288
289 // Remove "audio_track2".
Harald Alvestrand93dd7632022-01-19 12:28:45 +0000290 EXPECT_TRUE(caller->pc()->RemoveTrackOrError(sender2).ok());
Seth Hampson5897a6e2018-04-03 11:16:33 -0700291 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström31638672017-11-23 17:48:32 +0100292 ASSERT_EQ(callee->observer()->add_track_events_.size(), 2u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700293 EXPECT_EQ(callee->observer()->GetAddTrackReceivers(),
294 callee->observer()->remove_track_events_);
Steve Anton3172c032018-05-03 15:30:18 -0700295 EXPECT_EQ(0u, callee->observer()->remote_streams()->count());
Henrik Boström933d8b02017-10-10 10:05:16 -0700296}
297
Seth Hampson5b4f0752018-04-02 16:31:36 -0700298// Tests the edge case that if a stream ID changes for a given track that both
299// OnRemoveTrack and OnAddTrack is fired.
Steve Anton3172c032018-05-03 15:30:18 -0700300TEST_F(PeerConnectionRtpTestPlanB,
Seth Hampson5b4f0752018-04-02 16:31:36 -0700301 RemoteStreamIdChangesFiresOnRemoveAndOnAddTrack) {
302 auto caller = CreatePeerConnection();
303 auto callee = CreatePeerConnection();
304
305 const char kStreamId1[] = "stream1";
306 const char kStreamId2[] = "stream2";
Steve Anton3172c032018-05-03 15:30:18 -0700307 caller->AddAudioTrack("audio_track1", {kStreamId1});
Seth Hampson5897a6e2018-04-03 11:16:33 -0700308 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Seth Hampson5b4f0752018-04-02 16:31:36 -0700309 EXPECT_EQ(callee->observer()->add_track_events_.size(), 1u);
310
311 // Change the stream ID of the sender in the session description.
312 auto offer = caller->CreateOfferAndSetAsLocal();
Steve Anton3172c032018-05-03 15:30:18 -0700313 auto* audio_desc =
314 cricket::GetFirstAudioContentDescription(offer->description());
Seth Hampson5b4f0752018-04-02 16:31:36 -0700315 ASSERT_EQ(audio_desc->mutable_streams().size(), 1u);
316 audio_desc->mutable_streams()[0].set_stream_ids({kStreamId2});
Steve Anton3172c032018-05-03 15:30:18 -0700317 ASSERT_TRUE(callee->SetRemoteDescription(std::move(offer)));
Seth Hampson5b4f0752018-04-02 16:31:36 -0700318
319 ASSERT_EQ(callee->observer()->add_track_events_.size(), 2u);
320 EXPECT_EQ(callee->observer()->add_track_events_[1].streams[0]->id(),
321 kStreamId2);
322 ASSERT_EQ(callee->observer()->remove_track_events_.size(), 1u);
323 EXPECT_EQ(callee->observer()->remove_track_events_[0]->streams()[0]->id(),
324 kStreamId1);
325}
326
Steve Anton8b815cd2018-02-16 16:14:42 -0800327// Tests that setting a remote description with sending transceivers will fire
328// the OnTrack callback for each transceiver and setting a remote description
Seth Hampson5b4f0752018-04-02 16:31:36 -0700329// with receive only transceivers will not call OnTrack. One transceiver is
330// created without any stream_ids, while the other is created with multiple
331// stream_ids.
Steve Anton3172c032018-05-03 15:30:18 -0700332TEST_F(PeerConnectionRtpTestUnifiedPlan, AddTransceiverCallsOnTrack) {
Seth Hampson5b4f0752018-04-02 16:31:36 -0700333 const std::string kStreamId1 = "video_stream1";
334 const std::string kStreamId2 = "video_stream2";
Steve Anton3172c032018-05-03 15:30:18 -0700335 auto caller = CreatePeerConnection();
336 auto callee = CreatePeerConnection();
Steve Anton8b815cd2018-02-16 16:14:42 -0800337
338 auto audio_transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
Seth Hampson5b4f0752018-04-02 16:31:36 -0700339 RtpTransceiverInit video_transceiver_init;
340 video_transceiver_init.stream_ids = {kStreamId1, kStreamId2};
341 auto video_transceiver =
342 caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, video_transceiver_init);
Steve Anton8b815cd2018-02-16 16:14:42 -0800343
344 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
345
346 ASSERT_EQ(0u, caller->observer()->on_track_transceivers_.size());
347 ASSERT_EQ(2u, callee->observer()->on_track_transceivers_.size());
348 EXPECT_EQ(audio_transceiver->mid(),
349 callee->pc()->GetTransceivers()[0]->mid());
350 EXPECT_EQ(video_transceiver->mid(),
351 callee->pc()->GetTransceivers()[1]->mid());
Seth Hampson5b4f0752018-04-02 16:31:36 -0700352 std::vector<rtc::scoped_refptr<MediaStreamInterface>> audio_streams =
353 callee->pc()->GetTransceivers()[0]->receiver()->streams();
354 std::vector<rtc::scoped_refptr<MediaStreamInterface>> video_streams =
355 callee->pc()->GetTransceivers()[1]->receiver()->streams();
356 ASSERT_EQ(0u, audio_streams.size());
357 ASSERT_EQ(2u, video_streams.size());
358 EXPECT_EQ(kStreamId1, video_streams[0]->id());
359 EXPECT_EQ(kStreamId2, video_streams[1]->id());
Steve Anton8b815cd2018-02-16 16:14:42 -0800360}
361
362// Test that doing additional offer/answer exchanges with no changes to tracks
363// will cause no additional OnTrack calls after the tracks have been negotiated.
Steve Anton3172c032018-05-03 15:30:18 -0700364TEST_F(PeerConnectionRtpTestUnifiedPlan, ReofferDoesNotCallOnTrack) {
365 auto caller = CreatePeerConnection();
366 auto callee = CreatePeerConnection();
Steve Anton8b815cd2018-02-16 16:14:42 -0800367
368 caller->AddAudioTrack("audio");
369 callee->AddAudioTrack("audio");
370
371 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
372 EXPECT_EQ(1u, caller->observer()->on_track_transceivers_.size());
373 EXPECT_EQ(1u, callee->observer()->on_track_transceivers_.size());
374
375 // If caller reoffers with no changes expect no additional OnTrack calls.
376 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
377 EXPECT_EQ(1u, caller->observer()->on_track_transceivers_.size());
378 EXPECT_EQ(1u, callee->observer()->on_track_transceivers_.size());
379
380 // Also if callee reoffers with no changes expect no additional OnTrack calls.
381 ASSERT_TRUE(callee->ExchangeOfferAnswerWith(caller.get()));
382 EXPECT_EQ(1u, caller->observer()->on_track_transceivers_.size());
383 EXPECT_EQ(1u, callee->observer()->on_track_transceivers_.size());
384}
385
386// Test that OnTrack is called when the transceiver direction changes to send
387// the track.
Steve Anton3172c032018-05-03 15:30:18 -0700388TEST_F(PeerConnectionRtpTestUnifiedPlan, SetDirectionCallsOnTrack) {
389 auto caller = CreatePeerConnection();
390 auto callee = CreatePeerConnection();
Steve Anton8b815cd2018-02-16 16:14:42 -0800391
392 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
Harald Alvestrand6060df52020-08-11 09:54:02 +0200393 EXPECT_TRUE(
394 transceiver->SetDirectionWithError(RtpTransceiverDirection::kInactive)
395 .ok());
Steve Anton8b815cd2018-02-16 16:14:42 -0800396 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
397 EXPECT_EQ(0u, caller->observer()->on_track_transceivers_.size());
398 EXPECT_EQ(0u, callee->observer()->on_track_transceivers_.size());
399
Harald Alvestrand6060df52020-08-11 09:54:02 +0200400 EXPECT_TRUE(
401 transceiver->SetDirectionWithError(RtpTransceiverDirection::kSendOnly)
402 .ok());
Steve Anton8b815cd2018-02-16 16:14:42 -0800403 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
404 EXPECT_EQ(0u, caller->observer()->on_track_transceivers_.size());
405 EXPECT_EQ(1u, callee->observer()->on_track_transceivers_.size());
406
407 // If the direction changes but it is still receiving on the remote side, then
408 // OnTrack should not be fired again.
Harald Alvestrand6060df52020-08-11 09:54:02 +0200409 EXPECT_TRUE(
410 transceiver->SetDirectionWithError(RtpTransceiverDirection::kSendRecv)
411 .ok());
Steve Anton8b815cd2018-02-16 16:14:42 -0800412 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
413 EXPECT_EQ(0u, caller->observer()->on_track_transceivers_.size());
414 EXPECT_EQ(1u, callee->observer()->on_track_transceivers_.size());
415}
416
417// Test that OnTrack is called twice when a sendrecv call is started, the callee
418// changes the direction to inactive, then changes it back to sendrecv.
Steve Anton3172c032018-05-03 15:30:18 -0700419TEST_F(PeerConnectionRtpTestUnifiedPlan, SetDirectionHoldCallsOnTrackTwice) {
420 auto caller = CreatePeerConnection();
421 auto callee = CreatePeerConnection();
Steve Anton8b815cd2018-02-16 16:14:42 -0800422
423 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
424
425 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
426 EXPECT_EQ(0u, caller->observer()->on_track_transceivers_.size());
427 EXPECT_EQ(1u, callee->observer()->on_track_transceivers_.size());
428
429 // Put the call on hold by no longer receiving the track.
Harald Alvestrand6060df52020-08-11 09:54:02 +0200430 EXPECT_TRUE(callee->pc()
431 ->GetTransceivers()[0]
432 ->SetDirectionWithError(RtpTransceiverDirection::kInactive)
433 .ok());
Steve Anton8b815cd2018-02-16 16:14:42 -0800434
435 ASSERT_TRUE(callee->ExchangeOfferAnswerWith(caller.get()));
436 EXPECT_EQ(0u, caller->observer()->on_track_transceivers_.size());
437 EXPECT_EQ(1u, callee->observer()->on_track_transceivers_.size());
438
439 // Resume the call by changing the direction to recvonly. This should call
440 // OnTrack again on the callee side.
Harald Alvestrand6060df52020-08-11 09:54:02 +0200441 EXPECT_TRUE(callee->pc()
442 ->GetTransceivers()[0]
443 ->SetDirectionWithError(RtpTransceiverDirection::kRecvOnly)
444 .ok());
Steve Anton8b815cd2018-02-16 16:14:42 -0800445
446 ASSERT_TRUE(callee->ExchangeOfferAnswerWith(caller.get()));
447 EXPECT_EQ(0u, caller->observer()->on_track_transceivers_.size());
448 EXPECT_EQ(2u, callee->observer()->on_track_transceivers_.size());
449}
450
Steve Anton3172c032018-05-03 15:30:18 -0700451// Test that setting a remote offer twice with no answer in the middle results
Steve Anton0f5400a2018-07-17 14:25:36 -0700452// in OnAddTrack being fired only once.
Steve Anton3172c032018-05-03 15:30:18 -0700453TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Anton0f5400a2018-07-17 14:25:36 -0700454 ApplyTwoRemoteOffersWithNoAnswerResultsInOneAddTrackEvent) {
Henrik Boström31638672017-11-23 17:48:32 +0100455 auto caller = CreatePeerConnection();
456 auto callee = CreatePeerConnection();
457
Steve Anton3172c032018-05-03 15:30:18 -0700458 caller->AddAudioTrack("audio_track", {});
459
Amit Hilbuchae3df542019-01-07 12:13:08 -0800460 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Steve Anton3172c032018-05-03 15:30:18 -0700461 ASSERT_EQ(1u, callee->observer()->add_track_events_.size());
462
Amit Hilbuchae3df542019-01-07 12:13:08 -0800463 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Steve Anton0f5400a2018-07-17 14:25:36 -0700464 EXPECT_EQ(1u, callee->observer()->add_track_events_.size());
Steve Anton3172c032018-05-03 15:30:18 -0700465}
466
467// Test that setting a remote offer twice with no answer in the middle and the
468// track being removed between the two offers results in OnAddTrack being called
Steve Anton0f5400a2018-07-17 14:25:36 -0700469// once the first time and OnRemoveTrack being called once the second time.
Steve Anton3172c032018-05-03 15:30:18 -0700470TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Anton0f5400a2018-07-17 14:25:36 -0700471 ApplyRemoteOfferAddThenRemoteOfferRemoveResultsInOneRemoveTrackEvent) {
Steve Anton3172c032018-05-03 15:30:18 -0700472 auto caller = CreatePeerConnection();
473 auto callee = CreatePeerConnection();
474
475 auto sender = caller->AddAudioTrack("audio_track", {});
476
Amit Hilbuchae3df542019-01-07 12:13:08 -0800477 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Steve Anton3172c032018-05-03 15:30:18 -0700478 ASSERT_EQ(1u, callee->observer()->add_track_events_.size());
Steve Anton0f5400a2018-07-17 14:25:36 -0700479 EXPECT_EQ(0u, callee->observer()->remove_track_events_.size());
Steve Anton3172c032018-05-03 15:30:18 -0700480
Harald Alvestrand93dd7632022-01-19 12:28:45 +0000481 caller->pc()->RemoveTrackOrError(sender);
Steve Anton3172c032018-05-03 15:30:18 -0700482
Amit Hilbuchae3df542019-01-07 12:13:08 -0800483 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Steve Anton3172c032018-05-03 15:30:18 -0700484 EXPECT_EQ(1u, callee->observer()->add_track_events_.size());
Steve Anton0f5400a2018-07-17 14:25:36 -0700485 EXPECT_EQ(1u, callee->observer()->remove_track_events_.size());
486}
487
488// Test that changing the direction from receiving to not receiving between
489// setting the remote offer and creating / setting the local answer results in
490// a remove track event when SetLocalDescription is called.
491TEST_F(PeerConnectionRtpTestUnifiedPlan,
492 ChangeDirectionInAnswerResultsInRemoveTrackEvent) {
493 auto caller = CreatePeerConnection();
494 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
495 auto callee = CreatePeerConnection();
496 callee->AddAudioTrack("audio_track", {});
497
498 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOffer()));
499 EXPECT_EQ(1u, callee->observer()->add_track_events_.size());
Steve Anton3172c032018-05-03 15:30:18 -0700500 EXPECT_EQ(0u, callee->observer()->remove_track_events_.size());
Steve Anton0f5400a2018-07-17 14:25:36 -0700501
502 auto callee_transceiver = callee->pc()->GetTransceivers()[0];
Harald Alvestrand6060df52020-08-11 09:54:02 +0200503 EXPECT_TRUE(callee_transceiver
504 ->SetDirectionWithError(RtpTransceiverDirection::kSendOnly)
505 .ok());
Steve Anton0f5400a2018-07-17 14:25:36 -0700506
507 ASSERT_TRUE(callee->SetLocalDescription(callee->CreateAnswer()));
508 EXPECT_EQ(1u, callee->observer()->add_track_events_.size());
509 EXPECT_EQ(1u, callee->observer()->remove_track_events_.size());
Steve Anton3172c032018-05-03 15:30:18 -0700510}
511
Henrik Boströmafa07dd2018-12-20 11:06:02 +0100512TEST_F(PeerConnectionRtpTestUnifiedPlan, ChangeMsidWhileReceiving) {
513 auto caller = CreatePeerConnection();
514 caller->AddAudioTrack("audio_track", {"stream1"});
515 auto callee = CreatePeerConnection();
Amit Hilbuchae3df542019-01-07 12:13:08 -0800516 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boströmafa07dd2018-12-20 11:06:02 +0100517
518 ASSERT_EQ(1u, callee->observer()->on_track_transceivers_.size());
519 auto transceiver = callee->observer()->on_track_transceivers_[0];
520 ASSERT_EQ(1u, transceiver->receiver()->streams().size());
521 EXPECT_EQ("stream1", transceiver->receiver()->streams()[0]->id());
522
Amit Hilbuchae3df542019-01-07 12:13:08 -0800523 ASSERT_TRUE(callee->CreateAnswerAndSetAsLocal());
Henrik Boströmafa07dd2018-12-20 11:06:02 +0100524
525 // Change the stream ID in the offer.
Guido Urdaneta1ff16c82019-05-20 19:31:53 +0200526 caller->pc()->GetSenders()[0]->SetStreams({"stream2"});
527 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boströmafa07dd2018-12-20 11:06:02 +0100528 ASSERT_EQ(1u, transceiver->receiver()->streams().size());
529 EXPECT_EQ("stream2", transceiver->receiver()->streams()[0]->id());
530}
531
Steve Anton3172c032018-05-03 15:30:18 -0700532// These tests examine the state of the peer connection as a result of
533// performing SetRemoteDescription().
534
535TEST_P(PeerConnectionRtpTest, AddTrackWithoutStreamAddsReceiver) {
536 auto caller = CreatePeerConnection();
537 auto callee = CreatePeerConnection();
538
539 ASSERT_TRUE(caller->AddAudioTrack("audio_track", {}));
Seth Hampson5897a6e2018-04-03 11:16:33 -0700540 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström31638672017-11-23 17:48:32 +0100541
542 EXPECT_EQ(callee->pc()->GetReceivers().size(), 1u);
543 auto receiver_added = callee->pc()->GetReceivers()[0];
544 EXPECT_EQ("audio_track", receiver_added->track()->id());
Steve Anton3172c032018-05-03 15:30:18 -0700545
Florent Castelli15a38de2022-04-06 00:38:21 +0200546 if (sdp_semantics_ == SdpSemantics::kPlanB_DEPRECATED) {
Steve Anton3172c032018-05-03 15:30:18 -0700547 // Since we are not supporting the no stream case with Plan B, there should
548 // be a generated stream, even though we didn't set one with AddTrack.
549 ASSERT_EQ(1u, receiver_added->streams().size());
550 EXPECT_TRUE(receiver_added->streams()[0]->FindAudioTrack("audio_track"));
551 } else {
552 EXPECT_EQ(0u, receiver_added->streams().size());
553 }
Henrik Boström31638672017-11-23 17:48:32 +0100554}
555
Steve Anton3172c032018-05-03 15:30:18 -0700556TEST_P(PeerConnectionRtpTest, AddTrackWithStreamAddsReceiver) {
Henrik Boström31638672017-11-23 17:48:32 +0100557 auto caller = CreatePeerConnection();
558 auto callee = CreatePeerConnection();
559
Steve Anton3172c032018-05-03 15:30:18 -0700560 ASSERT_TRUE(caller->AddAudioTrack("audio_track", {"audio_stream"}));
Seth Hampson5897a6e2018-04-03 11:16:33 -0700561 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Henrik Boström31638672017-11-23 17:48:32 +0100562
563 EXPECT_EQ(callee->pc()->GetReceivers().size(), 1u);
564 auto receiver_added = callee->pc()->GetReceivers()[0];
565 EXPECT_EQ("audio_track", receiver_added->track()->id());
566 EXPECT_EQ(receiver_added->streams().size(), 1u);
Seth Hampson13b8bad2018-03-13 16:05:28 -0700567 EXPECT_EQ("audio_stream", receiver_added->streams()[0]->id());
Henrik Boström31638672017-11-23 17:48:32 +0100568 EXPECT_TRUE(receiver_added->streams()[0]->FindAudioTrack("audio_track"));
569}
570
Steve Anton3172c032018-05-03 15:30:18 -0700571TEST_P(PeerConnectionRtpTest, RemoveTrackWithoutStreamRemovesReceiver) {
Henrik Boström31638672017-11-23 17:48:32 +0100572 auto caller = CreatePeerConnection();
573 auto callee = CreatePeerConnection();
574
Steve Anton3172c032018-05-03 15:30:18 -0700575 auto sender = caller->AddAudioTrack("audio_track", {});
Henrik Boström31638672017-11-23 17:48:32 +0100576 ASSERT_TRUE(sender);
Steve Anton3172c032018-05-03 15:30:18 -0700577 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
578
Henrik Boström31638672017-11-23 17:48:32 +0100579 ASSERT_EQ(callee->pc()->GetReceivers().size(), 1u);
580 auto receiver = callee->pc()->GetReceivers()[0];
Harald Alvestrand93dd7632022-01-19 12:28:45 +0000581 ASSERT_TRUE(caller->pc()->RemoveTrackOrError(sender).ok());
Steve Anton3172c032018-05-03 15:30:18 -0700582 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Henrik Boström31638672017-11-23 17:48:32 +0100583
Steve Anton3172c032018-05-03 15:30:18 -0700584 if (sdp_semantics_ == SdpSemantics::kUnifiedPlan) {
585 // With Unified Plan the receiver stays but the transceiver transitions to
586 // inactive.
587 ASSERT_EQ(1u, callee->pc()->GetReceivers().size());
588 EXPECT_EQ(RtpTransceiverDirection::kInactive,
589 callee->pc()->GetTransceivers()[0]->current_direction());
590 } else {
591 // With Plan B the receiver is removed.
592 ASSERT_EQ(0u, callee->pc()->GetReceivers().size());
593 }
Henrik Boström31638672017-11-23 17:48:32 +0100594}
595
Steve Anton3172c032018-05-03 15:30:18 -0700596TEST_P(PeerConnectionRtpTest, RemoveTrackWithStreamRemovesReceiver) {
Henrik Boström31638672017-11-23 17:48:32 +0100597 auto caller = CreatePeerConnection();
598 auto callee = CreatePeerConnection();
599
Steve Anton3172c032018-05-03 15:30:18 -0700600 auto sender = caller->AddAudioTrack("audio_track", {"audio_stream"});
Henrik Boström31638672017-11-23 17:48:32 +0100601 ASSERT_TRUE(sender);
Steve Anton3172c032018-05-03 15:30:18 -0700602 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Henrik Boström31638672017-11-23 17:48:32 +0100603 ASSERT_EQ(callee->pc()->GetReceivers().size(), 1u);
604 auto receiver = callee->pc()->GetReceivers()[0];
Harald Alvestrand93dd7632022-01-19 12:28:45 +0000605 ASSERT_TRUE(caller->pc()->RemoveTrackOrError(sender).ok());
Steve Anton3172c032018-05-03 15:30:18 -0700606 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Henrik Boström31638672017-11-23 17:48:32 +0100607
Steve Anton3172c032018-05-03 15:30:18 -0700608 if (sdp_semantics_ == SdpSemantics::kUnifiedPlan) {
609 // With Unified Plan the receiver stays but the transceiver transitions to
610 // inactive.
611 EXPECT_EQ(1u, callee->pc()->GetReceivers().size());
612 EXPECT_EQ(RtpTransceiverDirection::kInactive,
613 callee->pc()->GetTransceivers()[0]->current_direction());
614 } else {
615 // With Plan B the receiver is removed.
616 EXPECT_EQ(0u, callee->pc()->GetReceivers().size());
617 }
Henrik Boström31638672017-11-23 17:48:32 +0100618}
619
Steve Anton3172c032018-05-03 15:30:18 -0700620TEST_P(PeerConnectionRtpTest, RemoveTrackWithSharedStreamRemovesReceiver) {
Henrik Boström31638672017-11-23 17:48:32 +0100621 auto caller = CreatePeerConnection();
622 auto callee = CreatePeerConnection();
623
Seth Hampson845e8782018-03-02 11:34:10 -0800624 const char kSharedStreamId[] = "shared_audio_stream";
Steve Anton3172c032018-05-03 15:30:18 -0700625 auto sender1 = caller->AddAudioTrack("audio_track1", {kSharedStreamId});
626 auto sender2 = caller->AddAudioTrack("audio_track2", {kSharedStreamId});
627 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
628 ASSERT_EQ(2u, callee->pc()->GetReceivers().size());
Henrik Boström31638672017-11-23 17:48:32 +0100629
630 // Remove "audio_track1".
Harald Alvestrand93dd7632022-01-19 12:28:45 +0000631 EXPECT_TRUE(caller->pc()->RemoveTrackOrError(sender1).ok());
Steve Anton3172c032018-05-03 15:30:18 -0700632 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
633
634 if (sdp_semantics_ == SdpSemantics::kUnifiedPlan) {
635 // With Unified Plan the receiver stays but the transceiver transitions to
636 // inactive.
637 ASSERT_EQ(2u, callee->pc()->GetReceivers().size());
638 auto transceiver = callee->pc()->GetTransceivers()[0];
639 EXPECT_EQ("audio_track1", transceiver->receiver()->track()->id());
640 EXPECT_EQ(RtpTransceiverDirection::kInactive,
641 transceiver->current_direction());
642 } else {
643 // With Plan B the receiver is removed.
644 ASSERT_EQ(1u, callee->pc()->GetReceivers().size());
645 EXPECT_EQ("audio_track2", callee->pc()->GetReceivers()[0]->track()->id());
646 }
Henrik Boström31638672017-11-23 17:48:32 +0100647
648 // Remove "audio_track2".
Harald Alvestrand93dd7632022-01-19 12:28:45 +0000649 EXPECT_TRUE(caller->pc()->RemoveTrackOrError(sender2).ok());
Steve Anton3172c032018-05-03 15:30:18 -0700650 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
651
652 if (sdp_semantics_ == SdpSemantics::kUnifiedPlan) {
653 // With Unified Plan the receiver stays but the transceiver transitions to
654 // inactive.
655 ASSERT_EQ(2u, callee->pc()->GetReceivers().size());
656 auto transceiver = callee->pc()->GetTransceivers()[1];
657 EXPECT_EQ("audio_track2", transceiver->receiver()->track()->id());
658 EXPECT_EQ(RtpTransceiverDirection::kInactive,
659 transceiver->current_direction());
660 } else {
661 // With Plan B the receiver is removed.
662 ASSERT_EQ(0u, callee->pc()->GetReceivers().size());
663 }
Henrik Boström31638672017-11-23 17:48:32 +0100664}
665
Florent Castelliabe301f2018-06-12 18:33:49 +0200666TEST_P(PeerConnectionRtpTest, AudioGetParametersHasHeaderExtensions) {
667 auto caller = CreatePeerConnection();
668 auto callee = CreatePeerConnection();
669 auto sender = caller->AddAudioTrack("audio_track");
670 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
671
672 ASSERT_GT(caller->pc()->GetSenders().size(), 0u);
673 EXPECT_GT(sender->GetParameters().header_extensions.size(), 0u);
674
675 ASSERT_GT(callee->pc()->GetReceivers().size(), 0u);
676 auto receiver = callee->pc()->GetReceivers()[0];
677 EXPECT_GT(receiver->GetParameters().header_extensions.size(), 0u);
678}
679
680TEST_P(PeerConnectionRtpTest, VideoGetParametersHasHeaderExtensions) {
681 auto caller = CreatePeerConnection();
682 auto callee = CreatePeerConnection();
683 auto sender = caller->AddVideoTrack("video_track");
684 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
685
686 ASSERT_GT(caller->pc()->GetSenders().size(), 0u);
687 EXPECT_GT(sender->GetParameters().header_extensions.size(), 0u);
688
689 ASSERT_GT(callee->pc()->GetReceivers().size(), 0u);
690 auto receiver = callee->pc()->GetReceivers()[0];
691 EXPECT_GT(receiver->GetParameters().header_extensions.size(), 0u);
692}
693
Henrik Boström31638672017-11-23 17:48:32 +0100694// Invokes SetRemoteDescription() twice in a row without synchronizing the two
695// calls and examine the state of the peer connection inside the callbacks to
696// ensure that the second call does not occur prematurely, contaminating the
697// state of the peer connection of the first callback.
Steve Anton3172c032018-05-03 15:30:18 -0700698TEST_F(PeerConnectionRtpTestPlanB,
Henrik Boström31638672017-11-23 17:48:32 +0100699 StatesCorrelateWithSetRemoteDescriptionCall) {
700 auto caller = CreatePeerConnection();
701 auto callee = CreatePeerConnection();
702
Henrik Boström31638672017-11-23 17:48:32 +0100703 // Create SDP for adding a track and for removing it. This will be used in the
704 // first and second SetRemoteDescription() calls.
Steve Anton3172c032018-05-03 15:30:18 -0700705 auto sender = caller->AddAudioTrack("audio_track", {});
Henrik Boström31638672017-11-23 17:48:32 +0100706 auto srd1_sdp = caller->CreateOfferAndSetAsLocal();
Harald Alvestrand93dd7632022-01-19 12:28:45 +0000707 EXPECT_TRUE(caller->pc()->RemoveTrackOrError(sender).ok());
Henrik Boström31638672017-11-23 17:48:32 +0100708 auto srd2_sdp = caller->CreateOfferAndSetAsLocal();
709
710 // In the first SetRemoteDescription() callback, check that we have a
711 // receiver for the track.
712 auto pc = callee->pc();
713 bool srd1_callback_called = false;
714 auto srd1_callback = [&srd1_callback_called, &pc]() {
715 EXPECT_EQ(pc->GetReceivers().size(), 1u);
716 srd1_callback_called = true;
717 };
718
719 // In the second SetRemoteDescription() callback, check that the receiver has
720 // been removed.
721 // TODO(hbos): When we implement Unified Plan, receivers will not be removed.
722 // Instead, the transceiver owning the receiver will become inactive.
723 // https://crbug.com/webrtc/7600
724 bool srd2_callback_called = false;
725 auto srd2_callback = [&srd2_callback_called, &pc]() {
726 EXPECT_TRUE(pc->GetReceivers().empty());
727 srd2_callback_called = true;
728 };
729
730 // Invoke SetRemoteDescription() twice in a row without synchronizing the two
731 // calls. The callbacks verify that the two calls are synchronized, as in, the
732 // effects of the second SetRemoteDescription() call must not have happened by
733 // the time the first callback is invoked. If it has then the receiver that is
734 // added as a result of the first SetRemoteDescription() call will already
735 // have been removed as a result of the second SetRemoteDescription() call
736 // when the first callback is invoked.
737 callee->pc()->SetRemoteDescription(
738 std::move(srd1_sdp),
Niels Möllere7cc8832022-01-04 15:20:03 +0100739 rtc::make_ref_counted<OnSuccessObserver<decltype(srd1_callback)>>(
740 srd1_callback));
Henrik Boström31638672017-11-23 17:48:32 +0100741 callee->pc()->SetRemoteDescription(
742 std::move(srd2_sdp),
Niels Möllere7cc8832022-01-04 15:20:03 +0100743 rtc::make_ref_counted<OnSuccessObserver<decltype(srd2_callback)>>(
744 srd2_callback));
Henrik Boström31638672017-11-23 17:48:32 +0100745 EXPECT_TRUE_WAIT(srd1_callback_called, kDefaultTimeout);
746 EXPECT_TRUE_WAIT(srd2_callback_called, kDefaultTimeout);
747}
748
Seth Hampson5897a6e2018-04-03 11:16:33 -0700749// Tests that a remote track is created with the signaled MSIDs when they are
750// communicated with a=msid and no SSRCs are signaled at all (i.e., no a=ssrc
751// lines).
Steve Anton3172c032018-05-03 15:30:18 -0700752TEST_F(PeerConnectionRtpTestUnifiedPlan, UnsignaledSsrcCreatesReceiverStreams) {
753 auto caller = CreatePeerConnection();
754 auto callee = CreatePeerConnection();
Seth Hampson5897a6e2018-04-03 11:16:33 -0700755 const char kStreamId1[] = "stream1";
756 const char kStreamId2[] = "stream2";
757 caller->AddTrack(caller->CreateAudioTrack("audio_track1"),
758 {kStreamId1, kStreamId2});
759
760 auto offer = caller->CreateOfferAndSetAsLocal();
761 // Munge the offer to take out everything but the stream_ids.
762 auto contents = offer->description()->contents();
763 ASSERT_TRUE(!contents.empty());
764 ASSERT_TRUE(!contents[0].media_description()->streams().empty());
765 std::vector<std::string> stream_ids =
766 contents[0].media_description()->streams()[0].stream_ids();
767 contents[0].media_description()->mutable_streams().clear();
768 cricket::StreamParams new_stream;
769 new_stream.set_stream_ids(stream_ids);
770 contents[0].media_description()->AddStream(new_stream);
771
772 // Set the remote description and verify that the streams were added to the
773 // receiver correctly.
774 ASSERT_TRUE(
775 callee->SetRemoteDescription(CloneSessionDescription(offer.get())));
776 auto receivers = callee->pc()->GetReceivers();
777 ASSERT_EQ(receivers.size(), 1u);
778 ASSERT_EQ(receivers[0]->streams().size(), 2u);
779 EXPECT_EQ(receivers[0]->streams()[0]->id(), kStreamId1);
780 EXPECT_EQ(receivers[0]->streams()[1]->id(), kStreamId2);
781}
Henrik Boströmc335b0e2021-04-08 07:25:38 +0200782TEST_F(PeerConnectionRtpTestUnifiedPlan, TracksDoNotEndWhenSsrcChanges) {
783 constexpr uint32_t kFirstMungedSsrc = 1337u;
784
785 auto caller = CreatePeerConnection();
786 auto callee = CreatePeerConnection();
787
788 // Caller offers to receive audio and video.
789 RtpTransceiverInit init;
790 init.direction = RtpTransceiverDirection::kRecvOnly;
791 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, init);
792 caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, init);
793
794 // Callee wants to send audio and video tracks.
795 callee->AddTrack(callee->CreateAudioTrack("audio_track"), {});
796 callee->AddTrack(callee->CreateVideoTrack("video_track"), {});
797
798 // Do inittial offer/answer exchange.
799 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
800 ASSERT_TRUE(
801 caller->SetRemoteDescription(callee->CreateAnswerAndSetAsLocal()));
802 ASSERT_EQ(caller->observer()->add_track_events_.size(), 2u);
803 ASSERT_EQ(caller->pc()->GetReceivers().size(), 2u);
804
805 // Do a follow-up offer/answer exchange where the SSRCs are modified.
806 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
807 auto answer = callee->CreateAnswer();
808 auto& contents = answer->description()->contents();
809 ASSERT_TRUE(!contents.empty());
810 for (size_t i = 0; i < contents.size(); ++i) {
811 auto& mutable_streams = contents[i].media_description()->mutable_streams();
812 ASSERT_EQ(mutable_streams.size(), 1u);
813 mutable_streams[0].ssrcs = {kFirstMungedSsrc + static_cast<uint32_t>(i)};
814 }
815 ASSERT_TRUE(
816 callee->SetLocalDescription(CloneSessionDescription(answer.get())));
817 ASSERT_TRUE(
818 caller->SetRemoteDescription(CloneSessionDescription(answer.get())));
819
820 // No furher track events should fire because we never changed direction, only
821 // SSRCs.
822 ASSERT_EQ(caller->observer()->add_track_events_.size(), 2u);
823 // We should have the same number of receivers as before.
824 auto receivers = caller->pc()->GetReceivers();
825 ASSERT_EQ(receivers.size(), 2u);
826 // The tracks are still alive.
827 EXPECT_EQ(receivers[0]->track()->state(),
828 MediaStreamTrackInterface::TrackState::kLive);
829 EXPECT_EQ(receivers[1]->track()->state(),
830 MediaStreamTrackInterface::TrackState::kLive);
831}
Seth Hampson5897a6e2018-04-03 11:16:33 -0700832
Seth Hampson5b4f0752018-04-02 16:31:36 -0700833// Tests that with Unified Plan if the the stream id changes for a track when
834// when setting a new remote description, that the media stream is updated
835// appropriately for the receiver.
Steve Anton0f5400a2018-07-17 14:25:36 -0700836// TODO(https://github.com/w3c/webrtc-pc/issues/1937): Resolve spec issue or fix
837// test.
838TEST_F(PeerConnectionRtpTestUnifiedPlan,
839 DISABLED_RemoteStreamIdChangesUpdatesReceiver) {
Steve Anton3172c032018-05-03 15:30:18 -0700840 auto caller = CreatePeerConnection();
841 auto callee = CreatePeerConnection();
Seth Hampson5b4f0752018-04-02 16:31:36 -0700842
843 const char kStreamId1[] = "stream1";
844 const char kStreamId2[] = "stream2";
Steve Anton0f5400a2018-07-17 14:25:36 -0700845 caller->AddAudioTrack("audio_track1", {kStreamId1});
Seth Hampson5897a6e2018-04-03 11:16:33 -0700846 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
Seth Hampson5b4f0752018-04-02 16:31:36 -0700847 EXPECT_EQ(callee->observer()->add_track_events_.size(), 1u);
848
849 // Change the stream id of the sender in the session description.
850 auto offer = caller->CreateOfferAndSetAsLocal();
851 auto contents = offer->description()->contents();
852 ASSERT_EQ(contents.size(), 1u);
853 ASSERT_EQ(contents[0].media_description()->mutable_streams().size(), 1u);
854 contents[0].media_description()->mutable_streams()[0].set_stream_ids(
855 {kStreamId2});
856
Steve Anton0f5400a2018-07-17 14:25:36 -0700857 // Set the remote description and verify that the stream was updated
858 // properly.
859 ASSERT_TRUE(callee->SetRemoteDescription(std::move(offer)));
Seth Hampson5b4f0752018-04-02 16:31:36 -0700860 auto receivers = callee->pc()->GetReceivers();
861 ASSERT_EQ(receivers.size(), 1u);
862 ASSERT_EQ(receivers[0]->streams().size(), 1u);
863 EXPECT_EQ(receivers[0]->streams()[0]->id(), kStreamId2);
864}
865
866// This tests a regression caught by a downstream client, that occured when
867// applying a remote description with a SessionDescription object that
868// contained StreamParams that didn't have ids. Although there were multiple
869// remote audio senders, FindSenderInfo didn't find them as unique, because
870// it looked up by StreamParam.id, which none had. This meant only one
871// AudioRtpReceiver was created, as opposed to one for each remote sender.
Steve Anton3172c032018-05-03 15:30:18 -0700872TEST_F(PeerConnectionRtpTestPlanB,
Seth Hampson5b4f0752018-04-02 16:31:36 -0700873 MultipleRemoteSendersWithoutStreamParamIdAddsMultipleReceivers) {
874 auto caller = CreatePeerConnection();
875 auto callee = CreatePeerConnection();
876
877 const char kStreamId1[] = "stream1";
878 const char kStreamId2[] = "stream2";
879 caller->AddAudioTrack("audio_track1", {kStreamId1});
880 caller->AddAudioTrack("audio_track2", {kStreamId2});
881
882 auto offer = caller->CreateOfferAndSetAsLocal();
883 auto mutable_streams =
884 cricket::GetFirstAudioContentDescription(offer->description())
885 ->mutable_streams();
886 ASSERT_EQ(mutable_streams.size(), 2u);
887 // Clear the IDs in the StreamParams.
888 mutable_streams[0].id.clear();
889 mutable_streams[1].id.clear();
890 ASSERT_TRUE(
891 callee->SetRemoteDescription(CloneSessionDescription(offer.get())));
892
893 auto receivers = callee->pc()->GetReceivers();
894 ASSERT_EQ(receivers.size(), 2u);
895 ASSERT_EQ(receivers[0]->streams().size(), 1u);
896 EXPECT_EQ(kStreamId1, receivers[0]->streams()[0]->id());
897 ASSERT_EQ(receivers[1]->streams().size(), 1u);
898 EXPECT_EQ(kStreamId2, receivers[1]->streams()[0]->id());
899}
900
Henrik Boström31638672017-11-23 17:48:32 +0100901// Tests for the legacy SetRemoteDescription() function signature.
Henrik Boström31638672017-11-23 17:48:32 +0100902
903// Sanity test making sure the callback is invoked.
Steve Anton3172c032018-05-03 15:30:18 -0700904TEST_P(PeerConnectionRtpTest, LegacyObserverOnSuccess) {
Henrik Boström31638672017-11-23 17:48:32 +0100905 auto caller = CreatePeerConnection();
906 auto callee = CreatePeerConnection();
907
908 std::string error;
909 ASSERT_TRUE(
910 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(), &error));
911}
912
913// Verifies legacy behavior: The observer is not called if if the peer
914// connection is destroyed because the asynchronous callback is executed in the
915// peer connection's message handler.
Steve Anton3172c032018-05-03 15:30:18 -0700916TEST_P(PeerConnectionRtpTest,
917 LegacyObserverNotCalledIfPeerConnectionDereferenced) {
Henrik Boström31638672017-11-23 17:48:32 +0100918 auto caller = CreatePeerConnection();
919 auto callee = CreatePeerConnection();
920
921 rtc::scoped_refptr<webrtc::MockSetSessionDescriptionObserver> observer =
Tommi87f70902021-04-27 14:43:08 +0200922 rtc::make_ref_counted<webrtc::MockSetSessionDescriptionObserver>();
Henrik Boström31638672017-11-23 17:48:32 +0100923
924 auto offer = caller->CreateOfferAndSetAsLocal();
Niels Möllerafb246b2022-04-20 14:26:50 +0200925 callee->pc()->SetRemoteDescription(observer.get(), offer.release());
Henrik Boström31638672017-11-23 17:48:32 +0100926 callee = nullptr;
927 rtc::Thread::Current()->ProcessMessages(0);
928 EXPECT_FALSE(observer->called());
929}
930
Steve Antonf9381f02017-12-14 10:23:57 -0800931// RtpTransceiver Tests.
Steve Anton9158ef62017-11-27 13:01:52 -0800932
933// Test that by default there are no transceivers with Unified Plan.
Steve Anton3172c032018-05-03 15:30:18 -0700934TEST_F(PeerConnectionRtpTestUnifiedPlan, PeerConnectionHasNoTransceivers) {
935 auto caller = CreatePeerConnection();
Steve Anton9158ef62017-11-27 13:01:52 -0800936 EXPECT_THAT(caller->pc()->GetTransceivers(), ElementsAre());
937}
938
939// Test that a transceiver created with the audio kind has the correct initial
940// properties.
Steve Anton3172c032018-05-03 15:30:18 -0700941TEST_F(PeerConnectionRtpTestUnifiedPlan,
942 AddTransceiverHasCorrectInitProperties) {
943 auto caller = CreatePeerConnection();
Steve Anton9158ef62017-11-27 13:01:52 -0800944
945 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
Danil Chapovalov66cadcc2018-06-19 16:47:43 +0200946 EXPECT_EQ(absl::nullopt, transceiver->mid());
Steve Anton9158ef62017-11-27 13:01:52 -0800947 EXPECT_FALSE(transceiver->stopped());
948 EXPECT_EQ(RtpTransceiverDirection::kSendRecv, transceiver->direction());
Danil Chapovalov66cadcc2018-06-19 16:47:43 +0200949 EXPECT_EQ(absl::nullopt, transceiver->current_direction());
Steve Anton9158ef62017-11-27 13:01:52 -0800950}
951
952// Test that adding a transceiver with the audio kind creates an audio sender
953// and audio receiver with the receiver having a live audio track.
Steve Anton3172c032018-05-03 15:30:18 -0700954TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Anton9158ef62017-11-27 13:01:52 -0800955 AddAudioTransceiverCreatesAudioSenderAndReceiver) {
Steve Anton3172c032018-05-03 15:30:18 -0700956 auto caller = CreatePeerConnection();
Steve Anton9158ef62017-11-27 13:01:52 -0800957
958 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
Steve Anton69470252018-02-09 11:43:08 -0800959 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, transceiver->media_type());
Steve Anton9158ef62017-11-27 13:01:52 -0800960
961 ASSERT_TRUE(transceiver->sender());
962 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, transceiver->sender()->media_type());
963
964 ASSERT_TRUE(transceiver->receiver());
965 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, transceiver->receiver()->media_type());
966
967 auto track = transceiver->receiver()->track();
968 ASSERT_TRUE(track);
969 EXPECT_EQ(MediaStreamTrackInterface::kAudioKind, track->kind());
970 EXPECT_EQ(MediaStreamTrackInterface::TrackState::kLive, track->state());
971}
972
973// Test that adding a transceiver with the video kind creates an video sender
974// and video receiver with the receiver having a live video track.
Steve Anton3172c032018-05-03 15:30:18 -0700975TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Anton9158ef62017-11-27 13:01:52 -0800976 AddAudioTransceiverCreatesVideoSenderAndReceiver) {
Steve Anton3172c032018-05-03 15:30:18 -0700977 auto caller = CreatePeerConnection();
Steve Anton9158ef62017-11-27 13:01:52 -0800978
979 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO);
Steve Anton69470252018-02-09 11:43:08 -0800980 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, transceiver->media_type());
Steve Anton9158ef62017-11-27 13:01:52 -0800981
982 ASSERT_TRUE(transceiver->sender());
983 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, transceiver->sender()->media_type());
984
985 ASSERT_TRUE(transceiver->receiver());
986 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, transceiver->receiver()->media_type());
987
988 auto track = transceiver->receiver()->track();
989 ASSERT_TRUE(track);
990 EXPECT_EQ(MediaStreamTrackInterface::kVideoKind, track->kind());
991 EXPECT_EQ(MediaStreamTrackInterface::TrackState::kLive, track->state());
992}
993
994// Test that after a call to AddTransceiver, the transceiver shows in
995// GetTransceivers(), the transceiver's sender shows in GetSenders(), and the
996// transceiver's receiver shows in GetReceivers().
Steve Anton3172c032018-05-03 15:30:18 -0700997TEST_F(PeerConnectionRtpTestUnifiedPlan, AddTransceiverShowsInLists) {
998 auto caller = CreatePeerConnection();
Steve Anton9158ef62017-11-27 13:01:52 -0800999
1000 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1001 EXPECT_EQ(
1002 std::vector<rtc::scoped_refptr<RtpTransceiverInterface>>{transceiver},
1003 caller->pc()->GetTransceivers());
1004 EXPECT_EQ(
1005 std::vector<rtc::scoped_refptr<RtpSenderInterface>>{
1006 transceiver->sender()},
1007 caller->pc()->GetSenders());
1008 EXPECT_EQ(
1009 std::vector<rtc::scoped_refptr<RtpReceiverInterface>>{
1010 transceiver->receiver()},
1011 caller->pc()->GetReceivers());
1012}
1013
1014// Test that the direction passed in through the AddTransceiver init parameter
1015// is set in the returned transceiver.
Steve Anton3172c032018-05-03 15:30:18 -07001016TEST_F(PeerConnectionRtpTestUnifiedPlan,
1017 AddTransceiverWithDirectionIsReflected) {
1018 auto caller = CreatePeerConnection();
Steve Anton9158ef62017-11-27 13:01:52 -08001019
1020 RtpTransceiverInit init;
1021 init.direction = RtpTransceiverDirection::kSendOnly;
1022 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, init);
1023 EXPECT_EQ(RtpTransceiverDirection::kSendOnly, transceiver->direction());
1024}
1025
Steve Anton9158ef62017-11-27 13:01:52 -08001026// Test that calling AddTransceiver with a track creates a transceiver which has
1027// its sender's track set to the passed-in track.
Steve Anton3172c032018-05-03 15:30:18 -07001028TEST_F(PeerConnectionRtpTestUnifiedPlan,
1029 AddTransceiverWithTrackCreatesSenderWithTrack) {
1030 auto caller = CreatePeerConnection();
Steve Anton9158ef62017-11-27 13:01:52 -08001031
1032 auto audio_track = caller->CreateAudioTrack("audio track");
1033 auto transceiver = caller->AddTransceiver(audio_track);
1034
1035 auto sender = transceiver->sender();
1036 ASSERT_TRUE(sender->track());
1037 EXPECT_EQ(audio_track, sender->track());
1038
1039 auto receiver = transceiver->receiver();
1040 ASSERT_TRUE(receiver->track());
1041 EXPECT_EQ(MediaStreamTrackInterface::kAudioKind, receiver->track()->kind());
1042 EXPECT_EQ(MediaStreamTrackInterface::TrackState::kLive,
1043 receiver->track()->state());
1044}
1045
1046// Test that calling AddTransceiver twice with the same track creates distinct
1047// transceivers, senders with the same track.
Steve Anton3172c032018-05-03 15:30:18 -07001048TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Anton9158ef62017-11-27 13:01:52 -08001049 AddTransceiverTwiceWithSameTrackCreatesMultipleTransceivers) {
Steve Anton3172c032018-05-03 15:30:18 -07001050 auto caller = CreatePeerConnection();
Steve Anton9158ef62017-11-27 13:01:52 -08001051
1052 auto audio_track = caller->CreateAudioTrack("audio track");
1053
1054 auto transceiver1 = caller->AddTransceiver(audio_track);
1055 auto transceiver2 = caller->AddTransceiver(audio_track);
1056
1057 EXPECT_NE(transceiver1, transceiver2);
1058
1059 auto sender1 = transceiver1->sender();
1060 auto sender2 = transceiver2->sender();
1061 EXPECT_NE(sender1, sender2);
1062 EXPECT_EQ(audio_track, sender1->track());
1063 EXPECT_EQ(audio_track, sender2->track());
1064
1065 EXPECT_THAT(caller->pc()->GetTransceivers(),
1066 UnorderedElementsAre(transceiver1, transceiver2));
1067 EXPECT_THAT(caller->pc()->GetSenders(),
1068 UnorderedElementsAre(sender1, sender2));
1069}
1070
Steve Anton3fe1b152017-12-12 10:20:08 -08001071// RtpTransceiver error handling tests.
1072
Steve Anton3172c032018-05-03 15:30:18 -07001073TEST_F(PeerConnectionRtpTestUnifiedPlan,
1074 AddTransceiverWithInvalidKindReturnsError) {
1075 auto caller = CreatePeerConnection();
Steve Anton3fe1b152017-12-12 10:20:08 -08001076
1077 auto result = caller->pc()->AddTransceiver(cricket::MEDIA_TYPE_DATA);
1078 EXPECT_EQ(RTCErrorType::INVALID_PARAMETER, result.error().type());
1079}
1080
Steve Anton3172c032018-05-03 15:30:18 -07001081TEST_F(PeerConnectionRtpTestUnifiedPlan,
1082 CanClosePeerConnectionWithoutCrashing) {
1083 auto caller = CreatePeerConnection();
Steve Anton3fe1b152017-12-12 10:20:08 -08001084
1085 caller->pc()->Close();
1086}
1087
Steve Antonf9381f02017-12-14 10:23:57 -08001088// Unified Plan AddTrack tests.
1089
Steve Antonf9381f02017-12-14 10:23:57 -08001090// Test that adding an audio track creates a new audio RtpSender with the given
1091// track.
Steve Anton3172c032018-05-03 15:30:18 -07001092TEST_F(PeerConnectionRtpTestUnifiedPlan, AddAudioTrackCreatesAudioSender) {
1093 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001094
1095 auto audio_track = caller->CreateAudioTrack("a");
Steve Anton2d6c76a2018-01-05 17:10:52 -08001096 auto sender = caller->AddTrack(audio_track);
Steve Antonf9381f02017-12-14 10:23:57 -08001097 ASSERT_TRUE(sender);
1098
1099 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, sender->media_type());
1100 EXPECT_EQ(audio_track, sender->track());
1101}
1102
1103// Test that adding a video track creates a new video RtpSender with the given
1104// track.
Steve Anton3172c032018-05-03 15:30:18 -07001105TEST_F(PeerConnectionRtpTestUnifiedPlan, AddVideoTrackCreatesVideoSender) {
1106 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001107
1108 auto video_track = caller->CreateVideoTrack("a");
Steve Anton2d6c76a2018-01-05 17:10:52 -08001109 auto sender = caller->AddTrack(video_track);
Steve Antonf9381f02017-12-14 10:23:57 -08001110 ASSERT_TRUE(sender);
1111
1112 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, sender->media_type());
1113 EXPECT_EQ(video_track, sender->track());
1114}
1115
1116// Test that adding a track to a new PeerConnection creates an RtpTransceiver
1117// with the sender that AddTrack returns and in the sendrecv direction.
Steve Anton3172c032018-05-03 15:30:18 -07001118TEST_F(PeerConnectionRtpTestUnifiedPlan, AddFirstTrackCreatesTransceiver) {
1119 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001120
1121 auto sender = caller->AddAudioTrack("a");
1122 ASSERT_TRUE(sender);
1123
1124 auto transceivers = caller->pc()->GetTransceivers();
1125 ASSERT_EQ(1u, transceivers.size());
1126 EXPECT_EQ(sender, transceivers[0]->sender());
1127 EXPECT_EQ(RtpTransceiverDirection::kSendRecv, transceivers[0]->direction());
1128}
1129
1130// Test that if a transceiver of the same type but no track had been added to
1131// the PeerConnection and later a call to AddTrack is made, the resulting sender
1132// is the transceiver's sender and the sender's track is the newly-added track.
Steve Anton3172c032018-05-03 15:30:18 -07001133TEST_F(PeerConnectionRtpTestUnifiedPlan, AddTrackReusesTransceiver) {
1134 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001135
1136 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1137 auto audio_track = caller->CreateAudioTrack("a");
Steve Anton2d6c76a2018-01-05 17:10:52 -08001138 auto sender = caller->AddTrack(audio_track);
Steve Antonf9381f02017-12-14 10:23:57 -08001139 ASSERT_TRUE(sender);
1140
1141 auto transceivers = caller->pc()->GetTransceivers();
1142 ASSERT_EQ(1u, transceivers.size());
1143 EXPECT_EQ(transceiver, transceivers[0]);
1144 EXPECT_EQ(sender, transceiver->sender());
1145 EXPECT_EQ(audio_track, sender->track());
1146}
1147
Jonas Oreland4b2a1062022-10-19 09:24:42 +02001148TEST_F(PeerConnectionRtpTestUnifiedPlan,
1149 AddTrackWithSendEncodingDoesNotReuseTransceiver) {
1150 auto caller = CreatePeerConnection();
1151
1152 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1153 auto audio_track = caller->CreateAudioTrack("a");
1154 RtpEncodingParameters encoding;
1155 auto sender = caller->AddTrack(audio_track, {}, {encoding});
1156 ASSERT_TRUE(sender);
1157
1158 auto transceivers = caller->pc()->GetTransceivers();
1159 ASSERT_EQ(2u, transceivers.size());
1160 EXPECT_EQ(transceiver, transceivers[0]);
1161 EXPECT_NE(sender, transceiver->sender());
1162 EXPECT_EQ(audio_track, sender->track());
1163}
1164
Steve Antonf9381f02017-12-14 10:23:57 -08001165// Test that adding two tracks to a new PeerConnection creates two
1166// RtpTransceivers in the same order.
Steve Anton3172c032018-05-03 15:30:18 -07001167TEST_F(PeerConnectionRtpTestUnifiedPlan, TwoAddTrackCreatesTwoTransceivers) {
1168 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001169
1170 auto sender1 = caller->AddAudioTrack("a");
1171 auto sender2 = caller->AddVideoTrack("v");
1172 ASSERT_TRUE(sender2);
1173
1174 auto transceivers = caller->pc()->GetTransceivers();
1175 ASSERT_EQ(2u, transceivers.size());
1176 EXPECT_EQ(sender1, transceivers[0]->sender());
1177 EXPECT_EQ(sender2, transceivers[1]->sender());
1178}
1179
1180// Test that if there are multiple transceivers with no sending track then a
1181// later call to AddTrack will use the one of the same type as the newly-added
1182// track.
Steve Anton3172c032018-05-03 15:30:18 -07001183TEST_F(PeerConnectionRtpTestUnifiedPlan, AddTrackReusesTransceiverOfType) {
1184 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001185
1186 auto audio_transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1187 auto video_transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO);
1188 auto sender = caller->AddVideoTrack("v");
1189
1190 ASSERT_EQ(2u, caller->pc()->GetTransceivers().size());
1191 EXPECT_NE(sender, audio_transceiver->sender());
1192 EXPECT_EQ(sender, video_transceiver->sender());
1193}
1194
1195// Test that if the only transceivers that do not have a sending track have a
1196// different type from the added track, then AddTrack will create a new
1197// transceiver for the track.
Steve Anton3172c032018-05-03 15:30:18 -07001198TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Antonf9381f02017-12-14 10:23:57 -08001199 AddTrackDoesNotReuseTransceiverOfWrongType) {
Steve Anton3172c032018-05-03 15:30:18 -07001200 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001201
1202 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1203 auto sender = caller->AddVideoTrack("v");
1204
1205 auto transceivers = caller->pc()->GetTransceivers();
1206 ASSERT_EQ(2u, transceivers.size());
1207 EXPECT_NE(sender, transceivers[0]->sender());
1208 EXPECT_EQ(sender, transceivers[1]->sender());
1209}
1210
1211// Test that the first available transceiver is reused by AddTrack when multiple
1212// are available.
Steve Anton3172c032018-05-03 15:30:18 -07001213TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Antonf9381f02017-12-14 10:23:57 -08001214 AddTrackReusesFirstMatchingTransceiver) {
Steve Anton3172c032018-05-03 15:30:18 -07001215 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001216
1217 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1218 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1219 auto sender = caller->AddAudioTrack("a");
1220
1221 auto transceivers = caller->pc()->GetTransceivers();
1222 ASSERT_EQ(2u, transceivers.size());
1223 EXPECT_EQ(sender, transceivers[0]->sender());
1224 EXPECT_NE(sender, transceivers[1]->sender());
1225}
1226
1227// Test that a call to AddTrack that reuses a transceiver will change the
1228// direction from inactive to sendonly.
Steve Anton3172c032018-05-03 15:30:18 -07001229TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Antonf9381f02017-12-14 10:23:57 -08001230 AddTrackChangesDirectionFromInactiveToSendOnly) {
Steve Anton3172c032018-05-03 15:30:18 -07001231 auto caller = CreatePeerConnection();
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001232 auto callee = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001233
1234 RtpTransceiverInit init;
1235 init.direction = RtpTransceiverDirection::kInactive;
1236 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, init);
Henrik Boströme574a312020-08-25 10:20:11 +02001237 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1238 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001239
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001240 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Henrik Boströme574a312020-08-25 10:20:11 +02001241 caller->observer()->clear_legacy_renegotiation_needed();
1242 caller->observer()->clear_latest_negotiation_needed_event();
Steve Antonf9381f02017-12-14 10:23:57 -08001243 ASSERT_TRUE(caller->AddAudioTrack("a"));
Henrik Boströme574a312020-08-25 10:20:11 +02001244 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1245 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001246
1247 EXPECT_EQ(RtpTransceiverDirection::kSendOnly, transceiver->direction());
1248}
1249
1250// Test that a call to AddTrack that reuses a transceiver will change the
1251// direction from recvonly to sendrecv.
Steve Anton3172c032018-05-03 15:30:18 -07001252TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Antonf9381f02017-12-14 10:23:57 -08001253 AddTrackChangesDirectionFromRecvOnlyToSendRecv) {
Steve Anton3172c032018-05-03 15:30:18 -07001254 auto caller = CreatePeerConnection();
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001255 auto callee = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001256
1257 RtpTransceiverInit init;
1258 init.direction = RtpTransceiverDirection::kRecvOnly;
1259 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, init);
Henrik Boströme574a312020-08-25 10:20:11 +02001260 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1261 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001262
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001263 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Henrik Boströme574a312020-08-25 10:20:11 +02001264 caller->observer()->clear_legacy_renegotiation_needed();
1265 caller->observer()->clear_latest_negotiation_needed_event();
Steve Antonf9381f02017-12-14 10:23:57 -08001266 ASSERT_TRUE(caller->AddAudioTrack("a"));
Henrik Boströme574a312020-08-25 10:20:11 +02001267 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1268 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001269
1270 EXPECT_EQ(RtpTransceiverDirection::kSendRecv, transceiver->direction());
1271}
1272
Steve Anton3172c032018-05-03 15:30:18 -07001273TEST_F(PeerConnectionRtpTestUnifiedPlan, AddTrackCreatesSenderWithTrackId) {
Steve Anton02ee47c2018-01-10 16:26:06 -08001274 const std::string kTrackId = "audio_track";
1275
Steve Anton3172c032018-05-03 15:30:18 -07001276 auto caller = CreatePeerConnection();
Steve Anton02ee47c2018-01-10 16:26:06 -08001277
1278 auto audio_track = caller->CreateAudioTrack(kTrackId);
1279 auto sender = caller->AddTrack(audio_track);
1280
1281 EXPECT_EQ(kTrackId, sender->id());
1282}
1283
Steve Antonf9381f02017-12-14 10:23:57 -08001284// Unified Plan AddTrack error handling.
1285
Steve Anton3172c032018-05-03 15:30:18 -07001286TEST_F(PeerConnectionRtpTestUnifiedPlan, AddTrackErrorIfClosed) {
1287 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001288
1289 auto audio_track = caller->CreateAudioTrack("a");
1290 caller->pc()->Close();
1291
Henrik Boströme574a312020-08-25 10:20:11 +02001292 caller->observer()->clear_legacy_renegotiation_needed();
1293 caller->observer()->clear_latest_negotiation_needed_event();
Yves Gerey665174f2018-06-19 15:03:05 +02001294 auto result = caller->pc()->AddTrack(audio_track, std::vector<std::string>());
Steve Anton2d6c76a2018-01-05 17:10:52 -08001295 EXPECT_EQ(RTCErrorType::INVALID_STATE, result.error().type());
Henrik Boströme574a312020-08-25 10:20:11 +02001296 EXPECT_FALSE(caller->observer()->legacy_renegotiation_needed());
1297 EXPECT_FALSE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001298}
1299
Steve Anton3172c032018-05-03 15:30:18 -07001300TEST_F(PeerConnectionRtpTestUnifiedPlan, AddTrackErrorIfTrackAlreadyHasSender) {
1301 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001302
1303 auto audio_track = caller->CreateAudioTrack("a");
Steve Anton2d6c76a2018-01-05 17:10:52 -08001304 ASSERT_TRUE(caller->AddTrack(audio_track));
Steve Antonf9381f02017-12-14 10:23:57 -08001305
Henrik Boströme574a312020-08-25 10:20:11 +02001306 caller->observer()->clear_legacy_renegotiation_needed();
1307 caller->observer()->clear_latest_negotiation_needed_event();
Yves Gerey665174f2018-06-19 15:03:05 +02001308 auto result = caller->pc()->AddTrack(audio_track, std::vector<std::string>());
Steve Anton2d6c76a2018-01-05 17:10:52 -08001309 EXPECT_EQ(RTCErrorType::INVALID_PARAMETER, result.error().type());
Henrik Boströme574a312020-08-25 10:20:11 +02001310 EXPECT_FALSE(caller->observer()->legacy_renegotiation_needed());
1311 EXPECT_FALSE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001312}
1313
1314// Unified Plan RemoveTrack tests.
1315
1316// Test that calling RemoveTrack on a sender with a previously-added track
1317// clears the sender's track.
Steve Anton3172c032018-05-03 15:30:18 -07001318TEST_F(PeerConnectionRtpTestUnifiedPlan, RemoveTrackClearsSenderTrack) {
1319 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001320
1321 auto sender = caller->AddAudioTrack("a");
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001322 ASSERT_TRUE(caller->pc()->RemoveTrackOrError(sender).ok());
Steve Antonf9381f02017-12-14 10:23:57 -08001323
1324 EXPECT_FALSE(sender->track());
1325}
1326
1327// Test that calling RemoveTrack on a sender where the transceiver is configured
1328// in the sendrecv direction changes the transceiver's direction to recvonly.
Steve Anton3172c032018-05-03 15:30:18 -07001329TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Antonf9381f02017-12-14 10:23:57 -08001330 RemoveTrackChangesDirectionFromSendRecvToRecvOnly) {
Steve Anton3172c032018-05-03 15:30:18 -07001331 auto caller = CreatePeerConnection();
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001332 auto callee = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001333
1334 RtpTransceiverInit init;
1335 init.direction = RtpTransceiverDirection::kSendRecv;
1336 auto transceiver =
1337 caller->AddTransceiver(caller->CreateAudioTrack("a"), init);
Henrik Boströme574a312020-08-25 10:20:11 +02001338 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1339 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001340
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001341 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Henrik Boströme574a312020-08-25 10:20:11 +02001342 caller->observer()->clear_legacy_renegotiation_needed();
1343 caller->observer()->clear_latest_negotiation_needed_event();
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001344
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001345 ASSERT_TRUE(caller->pc()->RemoveTrackOrError(transceiver->sender()).ok());
Henrik Boströme574a312020-08-25 10:20:11 +02001346 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1347 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001348
1349 EXPECT_EQ(RtpTransceiverDirection::kRecvOnly, transceiver->direction());
Steve Antonf9381f02017-12-14 10:23:57 -08001350}
1351
1352// Test that calling RemoveTrack on a sender where the transceiver is configured
1353// in the sendonly direction changes the transceiver's direction to inactive.
Steve Anton3172c032018-05-03 15:30:18 -07001354TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Antonf9381f02017-12-14 10:23:57 -08001355 RemoveTrackChangesDirectionFromSendOnlyToInactive) {
Steve Anton3172c032018-05-03 15:30:18 -07001356 auto caller = CreatePeerConnection();
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001357 auto callee = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001358
1359 RtpTransceiverInit init;
1360 init.direction = RtpTransceiverDirection::kSendOnly;
1361 auto transceiver =
1362 caller->AddTransceiver(caller->CreateAudioTrack("a"), init);
Henrik Boströme574a312020-08-25 10:20:11 +02001363 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1364 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001365
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001366 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Henrik Boströme574a312020-08-25 10:20:11 +02001367 caller->observer()->clear_legacy_renegotiation_needed();
1368 caller->observer()->clear_latest_negotiation_needed_event();
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001369
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001370 ASSERT_TRUE(caller->pc()->RemoveTrackOrError(transceiver->sender()).ok());
Henrik Boströme574a312020-08-25 10:20:11 +02001371 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1372 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001373
1374 EXPECT_EQ(RtpTransceiverDirection::kInactive, transceiver->direction());
1375}
1376
1377// Test that calling RemoveTrack with a sender that has a null track results in
1378// no change in state.
Steve Anton3172c032018-05-03 15:30:18 -07001379TEST_F(PeerConnectionRtpTestUnifiedPlan, RemoveTrackWithNullSenderTrackIsNoOp) {
1380 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001381
1382 auto sender = caller->AddAudioTrack("a");
1383 auto transceiver = caller->pc()->GetTransceivers()[0];
1384 ASSERT_TRUE(sender->SetTrack(nullptr));
1385
Henrik Boströme574a312020-08-25 10:20:11 +02001386 caller->observer()->clear_legacy_renegotiation_needed();
1387 caller->observer()->clear_latest_negotiation_needed_event();
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001388 ASSERT_TRUE(caller->pc()->RemoveTrackOrError(sender).ok());
Henrik Boströme574a312020-08-25 10:20:11 +02001389 EXPECT_FALSE(caller->observer()->legacy_renegotiation_needed());
1390 EXPECT_FALSE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001391
1392 EXPECT_EQ(RtpTransceiverDirection::kSendRecv, transceiver->direction());
1393}
1394
1395// Unified Plan RemoveTrack error handling.
1396
Steve Anton3172c032018-05-03 15:30:18 -07001397TEST_F(PeerConnectionRtpTestUnifiedPlan, RemoveTrackErrorIfClosed) {
1398 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001399
1400 auto sender = caller->AddAudioTrack("a");
1401 caller->pc()->Close();
1402
Henrik Boströme574a312020-08-25 10:20:11 +02001403 caller->observer()->clear_legacy_renegotiation_needed();
1404 caller->observer()->clear_latest_negotiation_needed_event();
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001405 EXPECT_FALSE(caller->pc()->RemoveTrackOrError(sender).ok());
Henrik Boströme574a312020-08-25 10:20:11 +02001406 EXPECT_FALSE(caller->observer()->legacy_renegotiation_needed());
1407 EXPECT_FALSE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001408}
1409
Steve Anton3172c032018-05-03 15:30:18 -07001410TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Antonf9381f02017-12-14 10:23:57 -08001411 RemoveTrackNoErrorIfTrackAlreadyRemoved) {
Steve Anton3172c032018-05-03 15:30:18 -07001412 auto caller = CreatePeerConnection();
Steve Antonf9381f02017-12-14 10:23:57 -08001413
1414 auto sender = caller->AddAudioTrack("a");
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001415 ASSERT_TRUE(caller->pc()->RemoveTrackOrError(sender).ok());
Steve Antonf9381f02017-12-14 10:23:57 -08001416
Henrik Boströme574a312020-08-25 10:20:11 +02001417 caller->observer()->clear_legacy_renegotiation_needed();
1418 caller->observer()->clear_latest_negotiation_needed_event();
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001419 EXPECT_TRUE(caller->pc()->RemoveTrackOrError(sender).ok());
Henrik Boströme574a312020-08-25 10:20:11 +02001420 EXPECT_FALSE(caller->observer()->legacy_renegotiation_needed());
1421 EXPECT_FALSE(caller->observer()->has_negotiation_needed_event());
Steve Antonf9381f02017-12-14 10:23:57 -08001422}
1423
Steve Anton60b6c1d2018-06-13 11:32:27 -07001424// Test that setting offers that add/remove/add a track repeatedly without
1425// setting the appropriate answer in between works.
1426// These are regression tests for bugs.webrtc.org/9401
1427TEST_F(PeerConnectionRtpTestUnifiedPlan, AddRemoveAddTrackOffersWorksAudio) {
1428 auto caller = CreatePeerConnection();
1429
1430 auto sender1 = caller->AddAudioTrack("audio1");
1431 ASSERT_TRUE(caller->SetLocalDescription(caller->CreateOffer()));
1432
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001433 caller->pc()->RemoveTrackOrError(sender1);
Steve Anton60b6c1d2018-06-13 11:32:27 -07001434 ASSERT_TRUE(caller->SetLocalDescription(caller->CreateOffer()));
1435
1436 // This will re-use the transceiver created by the first AddTrack.
1437 auto sender2 = caller->AddAudioTrack("audio2");
1438 ASSERT_TRUE(caller->SetLocalDescription(caller->CreateOffer()));
1439
1440 EXPECT_EQ(1u, caller->pc()->GetTransceivers().size());
1441 EXPECT_EQ(sender1, sender2);
1442}
1443TEST_F(PeerConnectionRtpTestUnifiedPlan, AddRemoveAddTrackOffersWorksVideo) {
1444 auto caller = CreatePeerConnection();
1445
1446 auto sender1 = caller->AddVideoTrack("video1");
1447 ASSERT_TRUE(caller->SetLocalDescription(caller->CreateOffer()));
1448
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001449 caller->pc()->RemoveTrackOrError(sender1);
Steve Anton60b6c1d2018-06-13 11:32:27 -07001450 ASSERT_TRUE(caller->SetLocalDescription(caller->CreateOffer()));
1451
1452 // This will re-use the transceiver created by the first AddTrack.
1453 auto sender2 = caller->AddVideoTrack("video2");
1454 ASSERT_TRUE(caller->SetLocalDescription(caller->CreateOffer()));
1455
1456 EXPECT_EQ(1u, caller->pc()->GetTransceivers().size());
1457 EXPECT_EQ(sender1, sender2);
1458}
1459
Steve Anton07563732018-06-26 11:13:50 -07001460// Test that CreateOffer succeeds if two tracks with the same label are added.
1461TEST_F(PeerConnectionRtpTestUnifiedPlan, CreateOfferSameTrackLabel) {
1462 auto caller = CreatePeerConnection();
1463
1464 auto audio_sender = caller->AddAudioTrack("track", {});
1465 auto video_sender = caller->AddVideoTrack("track", {});
1466
1467 EXPECT_TRUE(caller->CreateOffer());
1468
1469 EXPECT_EQ(audio_sender->track()->id(), video_sender->track()->id());
1470 EXPECT_NE(audio_sender->id(), video_sender->id());
1471}
1472
1473// Test that CreateAnswer succeeds if two tracks with the same label are added.
1474TEST_F(PeerConnectionRtpTestUnifiedPlan, CreateAnswerSameTrackLabel) {
1475 auto caller = CreatePeerConnection();
1476 auto callee = CreatePeerConnection();
1477
1478 RtpTransceiverInit recvonly;
1479 recvonly.direction = RtpTransceiverDirection::kRecvOnly;
1480 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, recvonly);
1481 caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, recvonly);
1482
1483 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOffer()));
1484
1485 auto audio_sender = callee->AddAudioTrack("track", {});
1486 auto video_sender = callee->AddVideoTrack("track", {});
1487
1488 EXPECT_TRUE(callee->CreateAnswer());
1489
1490 EXPECT_EQ(audio_sender->track()->id(), video_sender->track()->id());
1491 EXPECT_NE(audio_sender->id(), video_sender->id());
1492}
1493
1494// Test that calling AddTrack, RemoveTrack and AddTrack again creates a second
1495// m= section with a random sender id (different from the first, now rejected,
1496// m= section).
1497TEST_F(PeerConnectionRtpTestUnifiedPlan,
1498 AddRemoveAddTrackGeneratesNewSenderId) {
1499 auto caller = CreatePeerConnection();
1500 auto callee = CreatePeerConnection();
1501
1502 auto track = caller->CreateVideoTrack("video");
1503 auto sender1 = caller->AddTrack(track);
1504 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1505
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001506 caller->pc()->RemoveTrackOrError(sender1);
Steve Anton07563732018-06-26 11:13:50 -07001507 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1508
1509 auto sender2 = caller->AddTrack(track);
1510
1511 EXPECT_NE(sender1, sender2);
1512 EXPECT_NE(sender1->id(), sender2->id());
1513 std::string sender2_id = sender2->id();
1514
1515 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1516
1517 // The sender's ID should not change after negotiation.
1518 EXPECT_EQ(sender2_id, sender2->id());
1519}
1520
Steve Anton52d86772018-02-20 15:48:12 -08001521// Test that OnRenegotiationNeeded is fired if SetDirection is called on an
1522// active RtpTransceiver with a new direction.
Steve Anton3172c032018-05-03 15:30:18 -07001523TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Anton52d86772018-02-20 15:48:12 -08001524 RenegotiationNeededAfterTransceiverSetDirection) {
Steve Anton3172c032018-05-03 15:30:18 -07001525 auto caller = CreatePeerConnection();
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001526 auto callee = CreatePeerConnection();
Henrik Boströme574a312020-08-25 10:20:11 +02001527 EXPECT_FALSE(caller->observer()->legacy_renegotiation_needed());
1528 EXPECT_FALSE(caller->observer()->has_negotiation_needed_event());
Steve Anton52d86772018-02-20 15:48:12 -08001529
1530 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
Henrik Boströme574a312020-08-25 10:20:11 +02001531 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1532 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Steve Anton52d86772018-02-20 15:48:12 -08001533
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001534 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Henrik Boströme574a312020-08-25 10:20:11 +02001535 caller->observer()->clear_legacy_renegotiation_needed();
1536 caller->observer()->clear_latest_negotiation_needed_event();
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001537
Harald Alvestrand6060df52020-08-11 09:54:02 +02001538 transceiver->SetDirectionWithError(RtpTransceiverDirection::kInactive);
Henrik Boströme574a312020-08-25 10:20:11 +02001539 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1540 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Steve Anton52d86772018-02-20 15:48:12 -08001541}
1542
1543// Test that OnRenegotiationNeeded is not fired if SetDirection is called on an
1544// active RtpTransceiver with current direction.
Steve Anton3172c032018-05-03 15:30:18 -07001545TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Anton52d86772018-02-20 15:48:12 -08001546 NoRenegotiationNeededAfterTransceiverSetSameDirection) {
Steve Anton3172c032018-05-03 15:30:18 -07001547 auto caller = CreatePeerConnection();
Steve Anton52d86772018-02-20 15:48:12 -08001548
1549 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1550
Henrik Boströme574a312020-08-25 10:20:11 +02001551 caller->observer()->clear_legacy_renegotiation_needed();
1552 caller->observer()->clear_latest_negotiation_needed_event();
Harald Alvestrand6060df52020-08-11 09:54:02 +02001553 transceiver->SetDirectionWithError(transceiver->direction());
Henrik Boströme574a312020-08-25 10:20:11 +02001554 EXPECT_FALSE(caller->observer()->legacy_renegotiation_needed());
1555 EXPECT_FALSE(caller->observer()->has_negotiation_needed_event());
Steve Anton52d86772018-02-20 15:48:12 -08001556}
1557
1558// Test that OnRenegotiationNeeded is not fired if SetDirection is called on a
1559// stopped RtpTransceiver.
Steve Anton3172c032018-05-03 15:30:18 -07001560TEST_F(PeerConnectionRtpTestUnifiedPlan,
Steve Anton52d86772018-02-20 15:48:12 -08001561 NoRenegotiationNeededAfterSetDirectionOnStoppedTransceiver) {
Steve Anton3172c032018-05-03 15:30:18 -07001562 auto caller = CreatePeerConnection();
Steve Anton52d86772018-02-20 15:48:12 -08001563
1564 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
Harald Alvestrand6060df52020-08-11 09:54:02 +02001565 transceiver->StopInternal();
Steve Anton52d86772018-02-20 15:48:12 -08001566
Henrik Boströme574a312020-08-25 10:20:11 +02001567 caller->observer()->clear_legacy_renegotiation_needed();
1568 caller->observer()->clear_latest_negotiation_needed_event();
Harald Alvestrand6060df52020-08-11 09:54:02 +02001569 transceiver->SetDirectionWithError(RtpTransceiverDirection::kInactive);
Henrik Boströme574a312020-08-25 10:20:11 +02001570 EXPECT_FALSE(caller->observer()->legacy_renegotiation_needed());
1571 EXPECT_FALSE(caller->observer()->has_negotiation_needed_event());
Steve Anton52d86772018-02-20 15:48:12 -08001572}
1573
Harald Alvestrand6060df52020-08-11 09:54:02 +02001574// Test that currentDirection returnes "stopped" if the transceiver was stopped.
1575TEST_F(PeerConnectionRtpTestUnifiedPlan,
1576 CheckStoppedCurrentDirectionOnStoppedTransceiver) {
1577 auto caller = CreatePeerConnection();
1578
1579 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1580 transceiver->StopInternal();
1581
1582 EXPECT_TRUE(transceiver->stopping());
1583 EXPECT_TRUE(transceiver->stopped());
1584 EXPECT_EQ(RtpTransceiverDirection::kStopped,
1585 transceiver->current_direction());
1586}
1587
1588// Test that InvalidState is thrown on a stopping transceiver.
1589TEST_F(PeerConnectionRtpTestUnifiedPlan,
1590 CheckForInvalidStateOnStoppingTransceiver) {
1591 auto caller = CreatePeerConnection();
1592
1593 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1594 transceiver->StopStandard();
1595
1596 EXPECT_TRUE(transceiver->stopping());
1597 EXPECT_FALSE(transceiver->stopped());
1598 EXPECT_EQ(
1599 RTCErrorType::INVALID_STATE,
1600 transceiver->SetDirectionWithError(RtpTransceiverDirection::kInactive)
1601 .type());
1602}
1603
1604// Test that InvalidState is thrown on a stopped transceiver.
1605TEST_F(PeerConnectionRtpTestUnifiedPlan,
1606 CheckForInvalidStateOnStoppedTransceiver) {
1607 auto caller = CreatePeerConnection();
1608
1609 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1610 transceiver->StopInternal();
1611
1612 EXPECT_TRUE(transceiver->stopping());
1613 EXPECT_TRUE(transceiver->stopped());
1614 EXPECT_EQ(
1615 RTCErrorType::INVALID_STATE,
1616 transceiver->SetDirectionWithError(RtpTransceiverDirection::kInactive)
1617 .type());
1618}
1619
1620// Test that TypeError is thrown if the direction is set to "stopped".
1621TEST_F(PeerConnectionRtpTestUnifiedPlan,
1622 CheckForTypeErrorForStoppedOnTransceiver) {
1623 auto caller = CreatePeerConnection();
1624
1625 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1626 EXPECT_EQ(
1627 RTCErrorType::INVALID_PARAMETER,
1628 transceiver->SetDirectionWithError(RtpTransceiverDirection::kStopped)
1629 .type());
1630}
1631
Harald Alvestrand198cd732020-09-16 12:41:23 +00001632// Test that you can do createOffer/setLocalDescription with a stopped
1633// media section.
1634TEST_F(PeerConnectionRtpTestUnifiedPlan,
1635 SetLocalDescriptionWithStoppedMediaSection) {
1636 auto caller = CreatePeerConnection();
1637 auto callee = CreatePeerConnection();
1638 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1639 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1640 callee->pc()->GetTransceivers()[0]->StopStandard();
1641 ASSERT_TRUE(callee->ExchangeOfferAnswerWith(caller.get()));
1642 EXPECT_EQ(RtpTransceiverDirection::kStopped,
1643 transceiver->current_direction());
1644 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1645}
1646
Harald Alvestrand936f1af2020-09-22 07:41:50 +00001647TEST_F(PeerConnectionRtpTestUnifiedPlan,
1648 StopAndNegotiateCausesTransceiverToDisappear) {
1649 auto caller = CreatePeerConnection();
1650 auto callee = CreatePeerConnection();
1651 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
1652 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1653 callee->pc()->GetTransceivers()[0]->StopStandard();
1654 ASSERT_TRUE(callee->ExchangeOfferAnswerWith(caller.get()));
1655 EXPECT_EQ(RtpTransceiverDirection::kStopped,
1656 transceiver->current_direction());
1657 EXPECT_EQ(0U, caller->pc()->GetTransceivers().size());
1658 EXPECT_EQ(0U, callee->pc()->GetTransceivers().size());
1659 EXPECT_EQ(0U, caller->pc()->GetSenders().size());
1660 EXPECT_EQ(0U, callee->pc()->GetSenders().size());
1661 EXPECT_EQ(0U, caller->pc()->GetReceivers().size());
1662 EXPECT_EQ(0U, callee->pc()->GetReceivers().size());
1663}
1664
Harald Alvestrand09bd9ba2020-10-09 08:13:30 +00001665TEST_F(PeerConnectionRtpTestUnifiedPlan,
1666 SetLocalDescriptionWorksAfterRepeatedAddRemove) {
1667 auto caller = CreatePeerConnection();
1668 auto callee = CreatePeerConnection();
1669 auto video_track = caller->CreateVideoTrack("v");
1670 auto track = caller->CreateAudioTrack("a");
1671 caller->AddTransceiver(video_track);
1672 auto transceiver = caller->AddTransceiver(track);
1673 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001674 caller->pc()->RemoveTrackOrError(transceiver->sender());
Harald Alvestrand09bd9ba2020-10-09 08:13:30 +00001675 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1676 caller->AddTrack(track);
1677 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001678 caller->pc()->RemoveTrackOrError(transceiver->sender());
Harald Alvestrand09bd9ba2020-10-09 08:13:30 +00001679 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1680}
1681
1682// This is a repro of Chromium bug https://crbug.com/1134686
1683TEST_F(PeerConnectionRtpTestUnifiedPlan,
1684 SetLocalDescriptionWorksAfterRepeatedAddRemoveWithRemoteReject) {
1685 auto caller = CreatePeerConnection();
1686 auto callee = CreatePeerConnection();
1687 auto video_track = caller->CreateVideoTrack("v");
1688 auto track = caller->CreateAudioTrack("a");
1689 caller->AddTransceiver(video_track);
1690 auto transceiver = caller->AddTransceiver(track);
1691 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001692 caller->pc()->RemoveTrackOrError(transceiver->sender());
Harald Alvestrand09bd9ba2020-10-09 08:13:30 +00001693 ExchangeOfferAnswerWhereRemoteStopsTransceiver(caller.get(), callee.get(), 1);
1694 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1695 caller->AddTrack(track);
1696 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Harald Alvestrand93dd7632022-01-19 12:28:45 +00001697 caller->pc()->RemoveTrackOrError(transceiver->sender());
Harald Alvestrand09bd9ba2020-10-09 08:13:30 +00001698 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1699}
1700
Florent Castelli892acf02018-10-01 22:47:20 +02001701// Test that AddTransceiver fails if trying to use unimplemented RTP encoding
1702// parameters with the send_encodings parameters.
1703TEST_F(PeerConnectionRtpTestUnifiedPlan,
1704 CheckForUnsupportedEncodingParameters) {
1705 auto caller = CreatePeerConnection();
1706
1707 RtpTransceiverInit init;
1708 init.send_encodings.emplace_back();
1709
1710 auto default_send_encodings = init.send_encodings;
1711
Henrik Grunelle1301a82018-12-13 12:13:22 +00001712 // Unimplemented RtpParameters: ssrc, codec_payload_type, fec, rtx, dtx,
Amit Hilbuchaa584152019-02-06 17:09:52 -08001713 // ptime, scale_framerate_down_by, dependency_rids.
Florent Castelli892acf02018-10-01 22:47:20 +02001714 init.send_encodings[0].ssrc = 1;
1715 EXPECT_EQ(RTCErrorType::UNSUPPORTED_PARAMETER,
1716 caller->pc()
1717 ->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, init)
1718 .error()
1719 .type());
1720 init.send_encodings = default_send_encodings;
Florent Castelli892acf02018-10-01 22:47:20 +02001721}
1722
Florent Castellic1a0bcb2019-01-29 14:26:48 +01001723// Test that AddTransceiver fails if trying to use invalid RTP encoding
1724// parameters with the send_encodings parameters.
1725TEST_F(PeerConnectionRtpTestUnifiedPlan, CheckForInvalidEncodingParameters) {
1726 auto caller = CreatePeerConnection();
1727
1728 RtpTransceiverInit init;
1729 init.send_encodings.emplace_back();
1730
1731 auto default_send_encodings = init.send_encodings;
1732
1733 init.send_encodings[0].scale_resolution_down_by = 0.5;
1734 EXPECT_EQ(RTCErrorType::INVALID_RANGE,
1735 caller->pc()
1736 ->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, init)
1737 .error()
1738 .type());
1739 init.send_encodings = default_send_encodings;
1740
1741 init.send_encodings[0].bitrate_priority = 0;
1742 EXPECT_EQ(RTCErrorType::INVALID_RANGE,
1743 caller->pc()
1744 ->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, init)
1745 .error()
1746 .type());
1747 init.send_encodings = default_send_encodings;
1748
1749 init.send_encodings[0].min_bitrate_bps = 200000;
1750 init.send_encodings[0].max_bitrate_bps = 100000;
1751 EXPECT_EQ(RTCErrorType::INVALID_RANGE,
1752 caller->pc()
1753 ->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, init)
1754 .error()
1755 .type());
1756 init.send_encodings = default_send_encodings;
1757
1758 init.send_encodings[0].num_temporal_layers = 0;
1759 EXPECT_EQ(RTCErrorType::INVALID_RANGE,
1760 caller->pc()
1761 ->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, init)
1762 .error()
1763 .type());
1764 init.send_encodings = default_send_encodings;
1765
1766 init.send_encodings[0].num_temporal_layers = 5;
1767 EXPECT_EQ(RTCErrorType::INVALID_RANGE,
1768 caller->pc()
1769 ->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, init)
1770 .error()
1771 .type());
1772 init.send_encodings = default_send_encodings;
1773}
1774
Florent Castelli892acf02018-10-01 22:47:20 +02001775// Test that AddTransceiver transfers the send_encodings to the sender and they
1776// are retained after SetLocalDescription().
1777TEST_F(PeerConnectionRtpTestUnifiedPlan, SendEncodingsPassedToSender) {
1778 auto caller = CreatePeerConnection();
1779
1780 RtpTransceiverInit init;
1781 init.send_encodings.emplace_back();
1782 init.send_encodings[0].active = false;
1783 init.send_encodings[0].max_bitrate_bps = 180000;
1784
1785 auto result = caller->pc()->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, init);
1786 ASSERT_TRUE(result.ok());
1787
1788 auto init_send_encodings = result.value()->sender()->init_send_encodings();
1789 EXPECT_FALSE(init_send_encodings[0].active);
1790 EXPECT_EQ(init_send_encodings[0].max_bitrate_bps, 180000);
1791
1792 auto parameters = result.value()->sender()->GetParameters();
1793 EXPECT_FALSE(parameters.encodings[0].active);
1794 EXPECT_EQ(parameters.encodings[0].max_bitrate_bps, 180000);
1795
1796 ASSERT_TRUE(caller->SetLocalDescription(caller->CreateOffer()));
1797
1798 parameters = result.value()->sender()->GetParameters();
1799 EXPECT_FALSE(parameters.encodings[0].active);
1800 EXPECT_EQ(parameters.encodings[0].max_bitrate_bps, 180000);
1801}
1802
Steve Antone831b8c2018-02-01 12:22:16 -08001803// Test MSID signaling between Unified Plan and Plan B endpoints. There are two
1804// options for this kind of signaling: media section based (a=msid) and ssrc
1805// based (a=ssrc MSID). While JSEP only specifies media section MSID signaling,
1806// we want to ensure compatibility with older Plan B endpoints that might expect
1807// ssrc based MSID signaling. Thus we test here that Unified Plan offers both
1808// types but answers with the same type as the offer.
1809
Steve Anton3172c032018-05-03 15:30:18 -07001810class PeerConnectionMsidSignalingTest
1811 : public PeerConnectionRtpTestUnifiedPlan {};
Steve Antone831b8c2018-02-01 12:22:16 -08001812
1813TEST_F(PeerConnectionMsidSignalingTest, UnifiedPlanTalkingToOurself) {
1814 auto caller = CreatePeerConnectionWithUnifiedPlan();
1815 caller->AddAudioTrack("caller_audio");
1816 auto callee = CreatePeerConnectionWithUnifiedPlan();
1817 callee->AddAudioTrack("callee_audio");
1818
1819 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1820
1821 // Offer should have had both a=msid and a=ssrc MSID lines.
1822 auto* offer = callee->pc()->remote_description();
1823 EXPECT_EQ((cricket::kMsidSignalingMediaSection |
1824 cricket::kMsidSignalingSsrcAttribute),
1825 offer->description()->msid_signaling());
1826
1827 // Answer should have had only a=msid lines.
1828 auto* answer = caller->pc()->remote_description();
1829 EXPECT_EQ(cricket::kMsidSignalingMediaSection,
1830 answer->description()->msid_signaling());
1831}
1832
1833TEST_F(PeerConnectionMsidSignalingTest, PlanBOfferToUnifiedPlanAnswer) {
1834 auto caller = CreatePeerConnectionWithPlanB();
1835 caller->AddAudioTrack("caller_audio");
1836 auto callee = CreatePeerConnectionWithUnifiedPlan();
1837 callee->AddAudioTrack("callee_audio");
1838
1839 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1840
1841 // Offer should have only a=ssrc MSID lines.
1842 auto* offer = callee->pc()->remote_description();
1843 EXPECT_EQ(cricket::kMsidSignalingSsrcAttribute,
1844 offer->description()->msid_signaling());
1845
1846 // Answer should have only a=ssrc MSID lines to match the offer.
1847 auto* answer = caller->pc()->remote_description();
1848 EXPECT_EQ(cricket::kMsidSignalingSsrcAttribute,
1849 answer->description()->msid_signaling());
1850}
1851
Seth Hampson5b4f0752018-04-02 16:31:36 -07001852// This tests that a Plan B endpoint appropriately sets the remote description
1853// from a Unified Plan offer. When the Unified Plan offer contains a=msid lines
1854// that signal no stream ids or multiple stream ids we expect that the Plan B
1855// endpoint always has exactly one media stream per track.
1856TEST_F(PeerConnectionMsidSignalingTest, UnifiedPlanToPlanBAnswer) {
1857 const std::string kStreamId1 = "audio_stream_1";
1858 const std::string kStreamId2 = "audio_stream_2";
1859
1860 auto caller = CreatePeerConnectionWithUnifiedPlan();
1861 caller->AddAudioTrack("caller_audio", {kStreamId1, kStreamId2});
1862 caller->AddVideoTrack("caller_video", {});
1863 auto callee = CreatePeerConnectionWithPlanB();
1864 callee->AddAudioTrack("callee_audio");
1865 caller->AddVideoTrack("callee_video");
1866
1867 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1868
1869 // Offer should have had both a=msid and a=ssrc MSID lines.
1870 auto* offer = callee->pc()->remote_description();
1871 EXPECT_EQ((cricket::kMsidSignalingMediaSection |
1872 cricket::kMsidSignalingSsrcAttribute),
1873 offer->description()->msid_signaling());
1874
1875 // Callee should always have 1 stream for all of it's receivers.
1876 const auto& track_events = callee->observer()->add_track_events_;
1877 ASSERT_EQ(2u, track_events.size());
1878 ASSERT_EQ(1u, track_events[0].streams.size());
1879 EXPECT_EQ(kStreamId1, track_events[0].streams[0]->id());
1880 ASSERT_EQ(1u, track_events[1].streams.size());
1881 // This autogenerated a stream id for the empty one signalled.
1882 EXPECT_FALSE(track_events[1].streams[0]->id().empty());
1883}
1884
Steve Antone831b8c2018-02-01 12:22:16 -08001885TEST_F(PeerConnectionMsidSignalingTest, PureUnifiedPlanToUs) {
1886 auto caller = CreatePeerConnectionWithUnifiedPlan();
1887 caller->AddAudioTrack("caller_audio");
1888 auto callee = CreatePeerConnectionWithUnifiedPlan();
1889 callee->AddAudioTrack("callee_audio");
1890
1891 auto offer = caller->CreateOffer();
1892 // Simulate a pure Unified Plan offerer by setting the MSID signaling to media
1893 // section only.
1894 offer->description()->set_msid_signaling(cricket::kMsidSignalingMediaSection);
1895
1896 ASSERT_TRUE(
1897 caller->SetLocalDescription(CloneSessionDescription(offer.get())));
1898 ASSERT_TRUE(callee->SetRemoteDescription(std::move(offer)));
1899
1900 // Answer should have only a=msid to match the offer.
1901 auto answer = callee->CreateAnswer();
1902 EXPECT_EQ(cricket::kMsidSignalingMediaSection,
1903 answer->description()->msid_signaling());
1904}
1905
Henrik Boström91d039b2018-01-11 17:43:30 +01001906// Sender setups in a call.
1907
Steve Anton3172c032018-05-03 15:30:18 -07001908TEST_P(PeerConnectionRtpTest, CreateTwoSendersWithSameTrack) {
Henrik Boström91d039b2018-01-11 17:43:30 +01001909 auto caller = CreatePeerConnection();
1910 auto callee = CreatePeerConnection();
1911
1912 auto track = caller->CreateAudioTrack("audio_track");
1913 auto sender1 = caller->AddTrack(track);
1914 ASSERT_TRUE(sender1);
1915 // We need to temporarily reset the track for the subsequent AddTrack() to
1916 // succeed.
1917 EXPECT_TRUE(sender1->SetTrack(nullptr));
1918 auto sender2 = caller->AddTrack(track);
1919 EXPECT_TRUE(sender2);
Niels Möllerafb246b2022-04-20 14:26:50 +02001920 EXPECT_TRUE(sender1->SetTrack(track.get()));
Henrik Boström91d039b2018-01-11 17:43:30 +01001921
Florent Castelli15a38de2022-04-06 00:38:21 +02001922 if (sdp_semantics_ == SdpSemantics::kPlanB_DEPRECATED) {
Steve Anton3172c032018-05-03 15:30:18 -07001923 // TODO(hbos): When https://crbug.com/webrtc/8734 is resolved, this should
Artem Titovcfea2182021-08-10 01:22:31 +02001924 // return true, and doing `callee->SetRemoteDescription()` should work.
Steve Anton3172c032018-05-03 15:30:18 -07001925 EXPECT_FALSE(caller->CreateOfferAndSetAsLocal());
1926 } else {
1927 EXPECT_TRUE(caller->CreateOfferAndSetAsLocal());
1928 }
Henrik Boström91d039b2018-01-11 17:43:30 +01001929}
1930
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001931// This test exercises the code path that fires a NegotiationNeeded
1932// notification when the stream IDs of the local description differ from
Guido Urdaneta1ff16c82019-05-20 19:31:53 +02001933// the ones in the transceiver.
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001934TEST_F(PeerConnectionRtpTestUnifiedPlan,
1935 ChangeAssociatedStreamsTriggersRenegotiation) {
1936 auto caller = CreatePeerConnection();
1937 auto callee = CreatePeerConnection();
1938
1939 RtpTransceiverInit init;
1940 init.direction = RtpTransceiverDirection::kSendRecv;
1941 auto transceiver =
1942 caller->AddTransceiver(caller->CreateAudioTrack("a"), init);
Henrik Boströme574a312020-08-25 10:20:11 +02001943 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1944 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001945
1946 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
Henrik Boströme574a312020-08-25 10:20:11 +02001947 caller->observer()->clear_legacy_renegotiation_needed();
1948 caller->observer()->clear_latest_negotiation_needed_event();
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001949
Guido Urdaneta1ff16c82019-05-20 19:31:53 +02001950 transceiver->sender()->SetStreams({"stream3", "stream4", "stream5"});
Henrik Boströme574a312020-08-25 10:20:11 +02001951 EXPECT_TRUE(caller->observer()->legacy_renegotiation_needed());
1952 EXPECT_TRUE(caller->observer()->has_negotiation_needed_event());
Guido Urdaneta1ff16c82019-05-20 19:31:53 +02001953
1954 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
1955 auto callee_streams = callee->pc()->GetReceivers()[0]->streams();
1956 ASSERT_EQ(3u, callee_streams.size());
1957 EXPECT_EQ("stream3", callee_streams[0]->id());
1958 EXPECT_EQ("stream4", callee_streams[1]->id());
1959 EXPECT_EQ("stream5", callee_streams[2]->id());
Guido Urdaneta70c2db12019-04-16 12:24:14 +02001960}
1961
Mirko Bonadeic84f6612019-01-31 12:20:57 +01001962INSTANTIATE_TEST_SUITE_P(PeerConnectionRtpTest,
1963 PeerConnectionRtpTest,
Florent Castelli15a38de2022-04-06 00:38:21 +02001964 Values(SdpSemantics::kPlanB_DEPRECATED,
Mirko Bonadeic84f6612019-01-31 12:20:57 +01001965 SdpSemantics::kUnifiedPlan));
Steve Anton3172c032018-05-03 15:30:18 -07001966
Steve Anton9158ef62017-11-27 13:01:52 -08001967} // namespace webrtc