Index: webrtc/p2p/base/dtlstransportchannel_unittest.cc |
diff --git a/webrtc/p2p/base/dtlstransportchannel_unittest.cc b/webrtc/p2p/base/dtlstransportchannel_unittest.cc |
index 02b173ab3be1d67bb6f064f6d3252df17be7a337..10640f99e3c130a73ab1f579a3f053dcaa119e9f 100644 |
--- a/webrtc/p2p/base/dtlstransportchannel_unittest.cc |
+++ b/webrtc/p2p/base/dtlstransportchannel_unittest.cc |
@@ -51,7 +51,6 @@ class DtlsTestClient : public sigslot::has_slots<> { |
name_(name), |
signaling_thread_(signaling_thread), |
worker_thread_(worker_thread), |
- protocol_(cricket::ICEPROTO_GOOGLE), |
packet_size_(0), |
use_dtls_srtp_(false), |
ssl_max_version_(rtc::SSL_PROTOCOL_DTLS_10), |
@@ -67,9 +66,6 @@ class DtlsTestClient : public sigslot::has_slots<> { |
const rtc::scoped_refptr<rtc::RTCCertificate>& certificate() { |
return certificate_; |
} |
- void SetIceProtocol(cricket::TransportProtocol proto) { |
- protocol_ = proto; |
- } |
void SetupSrtp() { |
ASSERT(certificate_); |
use_dtls_srtp_ = true; |
@@ -166,10 +162,8 @@ class DtlsTestClient : public sigslot::has_slots<> { |
} |
} |
- std::string transport_type = (protocol_ == cricket::ICEPROTO_GOOGLE) ? |
- cricket::NS_GINGLE_P2P : cricket::NS_JINGLE_ICE_UDP; |
cricket::TransportDescription local_desc( |
- transport_type, std::vector<std::string>(), kIceUfrag1, kIcePwd1, |
+ std::vector<std::string>(), kIceUfrag1, kIcePwd1, |
cricket::ICEMODE_FULL, local_role, |
// If remote if the offerer and has no DTLS support, answer will be |
// without any fingerprint. |
@@ -178,7 +172,7 @@ class DtlsTestClient : public sigslot::has_slots<> { |
cricket::Candidates()); |
cricket::TransportDescription remote_desc( |
- transport_type, std::vector<std::string>(), kIceUfrag1, kIcePwd1, |
+ std::vector<std::string>(), kIceUfrag1, kIcePwd1, |
cricket::ICEMODE_FULL, remote_role, remote_fingerprint.get(), |
cricket::Candidates()); |
@@ -380,10 +374,7 @@ class DtlsTestClient : public sigslot::has_slots<> { |
std::string name_; |
rtc::Thread* signaling_thread_; |
rtc::Thread* worker_thread_; |
- |
rtc::scoped_refptr<rtc::RTCCertificate> certificate_; |
- cricket::TransportProtocol protocol_; |
- |
rtc::scoped_ptr<cricket::FakeTransport> transport_; |
std::vector<cricket::DtlsTransportChannelWrapper*> channels_; |
size_t packet_size_; |
@@ -555,27 +546,6 @@ class DtlsTransportChannelTest : public testing::Test { |
// Test that transport negotiation of ICE, no DTLS works properly. |
TEST_F(DtlsTransportChannelTest, TestChannelSetupIce) { |
- client1_.SetIceProtocol(cricket::ICEPROTO_RFC5245); |
- client2_.SetIceProtocol(cricket::ICEPROTO_RFC5245); |
- Negotiate(); |
- cricket::FakeTransportChannel* channel1 = client1_.GetFakeChannel(0); |
- cricket::FakeTransportChannel* channel2 = client2_.GetFakeChannel(0); |
- ASSERT_TRUE(channel1 != NULL); |
- ASSERT_TRUE(channel2 != NULL); |
- EXPECT_EQ(cricket::ICEROLE_CONTROLLING, channel1->GetIceRole()); |
- EXPECT_EQ(1U, channel1->IceTiebreaker()); |
- EXPECT_EQ(cricket::ICEPROTO_RFC5245, channel1->protocol()); |
- EXPECT_EQ(kIceUfrag1, channel1->ice_ufrag()); |
- EXPECT_EQ(kIcePwd1, channel1->ice_pwd()); |
- EXPECT_EQ(cricket::ICEROLE_CONTROLLED, channel2->GetIceRole()); |
- EXPECT_EQ(2U, channel2->IceTiebreaker()); |
- EXPECT_EQ(cricket::ICEPROTO_RFC5245, channel2->protocol()); |
-} |
- |
-// Test that transport negotiation of GICE, no DTLS works properly. |
-TEST_F(DtlsTransportChannelTest, TestChannelSetupGice) { |
- client1_.SetIceProtocol(cricket::ICEPROTO_GOOGLE); |
- client2_.SetIceProtocol(cricket::ICEPROTO_GOOGLE); |
Negotiate(); |
cricket::FakeTransportChannel* channel1 = client1_.GetFakeChannel(0); |
cricket::FakeTransportChannel* channel2 = client2_.GetFakeChannel(0); |
@@ -583,12 +553,10 @@ TEST_F(DtlsTransportChannelTest, TestChannelSetupGice) { |
ASSERT_TRUE(channel2 != NULL); |
EXPECT_EQ(cricket::ICEROLE_CONTROLLING, channel1->GetIceRole()); |
EXPECT_EQ(1U, channel1->IceTiebreaker()); |
- EXPECT_EQ(cricket::ICEPROTO_GOOGLE, channel1->protocol()); |
EXPECT_EQ(kIceUfrag1, channel1->ice_ufrag()); |
EXPECT_EQ(kIcePwd1, channel1->ice_pwd()); |
EXPECT_EQ(cricket::ICEROLE_CONTROLLED, channel2->GetIceRole()); |
EXPECT_EQ(2U, channel2->IceTiebreaker()); |
- EXPECT_EQ(cricket::ICEPROTO_GOOGLE, channel2->protocol()); |
} |
// Connect without DTLS, and transfer some data. |