Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10)

Side by Side Diff: webrtc/api/peerconnectioninterface_unittest.cc

Issue 2023373002: Separating internal and external methods of RtpSender/RtpReceiver. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 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 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 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 1028 matching lines...) Expand 10 before | Expand all | Expand 10 after
1039 pc_factory_->CreateLocalMediaStream(kStreamLabel1)); 1039 pc_factory_->CreateLocalMediaStream(kStreamLabel1));
1040 std::vector<MediaStreamInterface*> stream_list; 1040 std::vector<MediaStreamInterface*> stream_list;
1041 stream_list.push_back(stream.get()); 1041 stream_list.push_back(stream.get());
1042 scoped_refptr<AudioTrackInterface> audio_track( 1042 scoped_refptr<AudioTrackInterface> audio_track(
1043 pc_factory_->CreateAudioTrack("audio_track", nullptr)); 1043 pc_factory_->CreateAudioTrack("audio_track", nullptr));
1044 scoped_refptr<VideoTrackInterface> video_track(pc_factory_->CreateVideoTrack( 1044 scoped_refptr<VideoTrackInterface> video_track(pc_factory_->CreateVideoTrack(
1045 "video_track", 1045 "video_track",
1046 pc_factory_->CreateVideoSource(new cricket::FakeVideoCapturer()))); 1046 pc_factory_->CreateVideoSource(new cricket::FakeVideoCapturer())));
1047 auto audio_sender = pc_->AddTrack(audio_track, stream_list); 1047 auto audio_sender = pc_->AddTrack(audio_track, stream_list);
1048 auto video_sender = pc_->AddTrack(video_track, stream_list); 1048 auto video_sender = pc_->AddTrack(video_track, stream_list);
1049 EXPECT_EQ(kStreamLabel1, audio_sender->stream_id()); 1049 EXPECT_EQ(1UL, audio_sender->stream_ids().size());
1050 EXPECT_EQ(kStreamLabel1, audio_sender->stream_ids()[0]);
1050 EXPECT_EQ("audio_track", audio_sender->id()); 1051 EXPECT_EQ("audio_track", audio_sender->id());
1051 EXPECT_EQ(audio_track, audio_sender->track()); 1052 EXPECT_EQ(audio_track, audio_sender->track());
1052 EXPECT_EQ(kStreamLabel1, video_sender->stream_id()); 1053 EXPECT_EQ(1UL, video_sender->stream_ids().size());
1054 EXPECT_EQ(kStreamLabel1, video_sender->stream_ids()[0]);
1053 EXPECT_EQ("video_track", video_sender->id()); 1055 EXPECT_EQ("video_track", video_sender->id());
1054 EXPECT_EQ(video_track, video_sender->track()); 1056 EXPECT_EQ(video_track, video_sender->track());
1055 1057
1056 // Now create an offer and check for the senders. 1058 // Now create an offer and check for the senders.
1057 std::unique_ptr<SessionDescriptionInterface> offer; 1059 std::unique_ptr<SessionDescriptionInterface> offer;
1058 ASSERT_TRUE(DoCreateOffer(&offer, nullptr)); 1060 ASSERT_TRUE(DoCreateOffer(&offer, nullptr));
1059 1061
1060 const cricket::ContentInfo* audio_content = 1062 const cricket::ContentInfo* audio_content =
1061 cricket::GetFirstAudioContent(offer->description()); 1063 cricket::GetFirstAudioContent(offer->description());
1062 const cricket::AudioContentDescription* audio_desc = 1064 const cricket::AudioContentDescription* audio_desc =
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
1115 auto audio_sender = 1117 auto audio_sender =
1116 pc_->AddTrack(audio_track, std::vector<MediaStreamInterface*>()); 1118 pc_->AddTrack(audio_track, std::vector<MediaStreamInterface*>());
1117 auto video_sender = 1119 auto video_sender =
1118 pc_->AddTrack(video_track, std::vector<MediaStreamInterface*>()); 1120 pc_->AddTrack(video_track, std::vector<MediaStreamInterface*>());
1119 EXPECT_EQ("audio_track", audio_sender->id()); 1121 EXPECT_EQ("audio_track", audio_sender->id());
1120 EXPECT_EQ(audio_track, audio_sender->track()); 1122 EXPECT_EQ(audio_track, audio_sender->track());
1121 EXPECT_EQ("video_track", video_sender->id()); 1123 EXPECT_EQ("video_track", video_sender->id());
1122 EXPECT_EQ(video_track, video_sender->track()); 1124 EXPECT_EQ(video_track, video_sender->track());
1123 // If the ID is truly a random GUID, it should be infinitely unlikely they 1125 // If the ID is truly a random GUID, it should be infinitely unlikely they
1124 // will be the same. 1126 // will be the same.
1125 EXPECT_NE(video_sender->stream_id(), audio_sender->stream_id()); 1127 EXPECT_NE(video_sender->stream_ids(), audio_sender->stream_ids());
1126 } 1128 }
1127 1129
1128 TEST_F(PeerConnectionInterfaceTest, CreateOfferReceiveAnswer) { 1130 TEST_F(PeerConnectionInterfaceTest, CreateOfferReceiveAnswer) {
1129 InitiateCall(); 1131 InitiateCall();
1130 WaitAndVerifyOnAddStream(kStreamLabel1); 1132 WaitAndVerifyOnAddStream(kStreamLabel1);
1131 VerifyRemoteRtpHeaderExtensions(); 1133 VerifyRemoteRtpHeaderExtensions();
1132 } 1134 }
1133 1135
1134 TEST_F(PeerConnectionInterfaceTest, CreateOfferReceivePrAnswerAndAnswer) { 1136 TEST_F(PeerConnectionInterfaceTest, CreateOfferReceivePrAnswerAndAnswer) {
1135 CreatePeerConnection(); 1137 CreatePeerConnection();
(...skipping 1525 matching lines...) Expand 10 before | Expand all | Expand 10 after
2661 FakeConstraints updated_answer_c; 2663 FakeConstraints updated_answer_c;
2662 answer_c.SetMandatoryReceiveAudio(false); 2664 answer_c.SetMandatoryReceiveAudio(false);
2663 answer_c.SetMandatoryReceiveVideo(false); 2665 answer_c.SetMandatoryReceiveVideo(false);
2664 2666
2665 cricket::MediaSessionOptions updated_answer_options; 2667 cricket::MediaSessionOptions updated_answer_options;
2666 EXPECT_TRUE( 2668 EXPECT_TRUE(
2667 ParseConstraintsForAnswer(&updated_answer_c, &updated_answer_options)); 2669 ParseConstraintsForAnswer(&updated_answer_c, &updated_answer_options));
2668 EXPECT_TRUE(updated_answer_options.has_audio()); 2670 EXPECT_TRUE(updated_answer_options.has_audio());
2669 EXPECT_TRUE(updated_answer_options.has_video()); 2671 EXPECT_TRUE(updated_answer_options.has_video());
2670 } 2672 }
OLDNEW
« no previous file with comments | « webrtc/api/peerconnection.cc ('k') | webrtc/api/proxy.h » ('j') | webrtc/api/proxy.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698