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

Unified Diff: webrtc/pc/channel_unittest.cc

Issue 2997983002: Completed the functionalities of SrtpTransport. (Closed)
Patch Set: Fix the chromimum issue. Created 3 years, 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/pc/channel.cc ('k') | webrtc/pc/rtptransport.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/pc/channel_unittest.cc
diff --git a/webrtc/pc/channel_unittest.cc b/webrtc/pc/channel_unittest.cc
index 3f30201874ec976f70208a299d977af5ff34d01e..ae34a12de4bd4f8a3b02fff0103d58645331053b 100644
--- a/webrtc/pc/channel_unittest.cc
+++ b/webrtc/pc/channel_unittest.cc
@@ -581,7 +581,7 @@ class ChannelTest : public testing::Test, public sigslot::has_slots<> {
// Basic sanity check.
void TestInit() {
CreateChannels(0, 0);
- EXPECT_FALSE(channel1_->secure());
+ EXPECT_FALSE(channel1_->srtp_active());
EXPECT_FALSE(media_channel1_->sending());
if (verify_playout_) {
EXPECT_FALSE(media_channel1_->playout());
@@ -896,8 +896,8 @@ class ChannelTest : public testing::Test, public sigslot::has_slots<> {
EXPECT_TRUE(channel2_->SetRemoteContent(&content4, CA_ANSWER, NULL));
EXPECT_EQ(0u, media_channel2_->recv_streams().size());
- EXPECT_TRUE(channel1_->secure());
- EXPECT_TRUE(channel2_->secure());
+ EXPECT_TRUE(channel1_->srtp_active());
+ EXPECT_TRUE(channel2_->srtp_active());
SendCustomRtp2(kSsrc2, 0);
WaitForThreads();
EXPECT_TRUE(CheckCustomRtp1(kSsrc2, 0));
@@ -1253,14 +1253,14 @@ class ChannelTest : public testing::Test, public sigslot::has_slots<> {
// Test setting up a call.
void TestCallSetup() {
CreateChannels(0, 0);
- EXPECT_FALSE(channel1_->secure());
+ EXPECT_FALSE(channel1_->srtp_active());
EXPECT_TRUE(SendInitiate());
if (verify_playout_) {
EXPECT_TRUE(media_channel1_->playout());
}
EXPECT_FALSE(media_channel1_->sending());
EXPECT_TRUE(SendAccept());
- EXPECT_FALSE(channel1_->secure());
+ EXPECT_FALSE(channel1_->srtp_active());
EXPECT_TRUE(media_channel1_->sending());
EXPECT_EQ(1U, media_channel1_->codecs().size());
if (verify_playout_) {
@@ -1535,17 +1535,17 @@ class ChannelTest : public testing::Test, public sigslot::has_slots<> {
bool dtls1 = !!(flags1_in & DTLS);
bool dtls2 = !!(flags2_in & DTLS);
CreateChannels(flags1, flags2);
- EXPECT_FALSE(channel1_->secure());
- EXPECT_FALSE(channel2_->secure());
+ EXPECT_FALSE(channel1_->srtp_active());
+ EXPECT_FALSE(channel2_->srtp_active());
EXPECT_TRUE(SendInitiate());
WaitForThreads();
EXPECT_TRUE(channel1_->writable());
EXPECT_TRUE(channel2_->writable());
EXPECT_TRUE(SendAccept());
- EXPECT_TRUE(channel1_->secure());
- EXPECT_TRUE(channel2_->secure());
- EXPECT_EQ(dtls1 && dtls2, channel1_->secure_dtls());
- EXPECT_EQ(dtls1 && dtls2, channel2_->secure_dtls());
+ EXPECT_TRUE(channel1_->srtp_active());
+ EXPECT_TRUE(channel2_->srtp_active());
+ EXPECT_EQ(dtls1 && dtls2, channel1_->dtls_active());
+ EXPECT_EQ(dtls1 && dtls2, channel2_->dtls_active());
SendRtp1();
SendRtp2();
SendRtcp1();
@@ -1564,12 +1564,12 @@ class ChannelTest : public testing::Test, public sigslot::has_slots<> {
// Test that we properly handling SRTP negotiating down to RTP.
void SendSrtpToRtp() {
CreateChannels(SECURE, 0);
- EXPECT_FALSE(channel1_->secure());
- EXPECT_FALSE(channel2_->secure());
+ EXPECT_FALSE(channel1_->srtp_active());
+ EXPECT_FALSE(channel2_->srtp_active());
EXPECT_TRUE(SendInitiate());
EXPECT_TRUE(SendAccept());
- EXPECT_FALSE(channel1_->secure());
- EXPECT_FALSE(channel2_->secure());
+ EXPECT_FALSE(channel1_->srtp_active());
+ EXPECT_FALSE(channel2_->srtp_active());
SendRtp1();
SendRtp2();
SendRtcp1();
@@ -1594,8 +1594,8 @@ class ChannelTest : public testing::Test, public sigslot::has_slots<> {
SSRC_MUX | RTCP_MUX | SECURE);
EXPECT_TRUE(SendOffer());
EXPECT_TRUE(SendProvisionalAnswer());
- EXPECT_TRUE(channel1_->secure());
- EXPECT_TRUE(channel2_->secure());
+ EXPECT_TRUE(channel1_->srtp_active());
+ EXPECT_TRUE(channel2_->srtp_active());
EXPECT_TRUE(channel1_->NeedsRtcpTransport());
EXPECT_TRUE(channel2_->NeedsRtcpTransport());
WaitForThreads(); // Wait for 'sending' flag go through network thread.
@@ -1620,8 +1620,8 @@ class ChannelTest : public testing::Test, public sigslot::has_slots<> {
EXPECT_FALSE(channel2_->NeedsRtcpTransport());
EXPECT_EQ(1, rtcp_mux_activated_callbacks1_);
EXPECT_EQ(1, rtcp_mux_activated_callbacks2_);
- EXPECT_TRUE(channel1_->secure());
- EXPECT_TRUE(channel2_->secure());
+ EXPECT_TRUE(channel1_->srtp_active());
+ EXPECT_TRUE(channel2_->srtp_active());
SendCustomRtcp1(kSsrc1);
SendCustomRtp1(kSsrc1, ++sequence_number1_1);
SendCustomRtcp2(kSsrc2);
« no previous file with comments | « webrtc/pc/channel.cc ('k') | webrtc/pc/rtptransport.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698