Index: talk/app/webrtc/webrtcsession_unittest.cc |
diff --git a/talk/app/webrtc/webrtcsession_unittest.cc b/talk/app/webrtc/webrtcsession_unittest.cc |
index de10bc40a8b432ba79a248d4a55b112f29f0d2e0..bd49e72b8a80adb6d6736e3d2549cb55d51da199 100644 |
--- a/talk/app/webrtc/webrtcsession_unittest.cc |
+++ b/talk/app/webrtc/webrtcsession_unittest.cc |
@@ -70,8 +70,6 @@ using cricket::BaseSession; |
using cricket::DF_PLAY; |
using cricket::DF_SEND; |
using cricket::FakeVoiceMediaChannel; |
-using cricket::NS_GINGLE_P2P; |
-using cricket::NS_JINGLE_ICE_UDP; |
using cricket::TransportInfo; |
using rtc::SocketAddress; |
using rtc::scoped_ptr; |
@@ -337,8 +335,6 @@ class WebRtcSessionTest : public testing::Test { |
turn_server_(Thread::Current(), kTurnUdpIntAddr, kTurnUdpExtAddr), |
mediastream_signaling_(channel_manager_.get()), |
metrics_observer_(new rtc::RefCountedObject<FakeMetricsObserver>()) { |
- tdesc_factory_->set_protocol(cricket::ICEPROTO_HYBRID); |
- |
cricket::ServerAddresses stun_servers; |
stun_servers.insert(stun_socket_addr_); |
allocator_.reset(new cricket::BasicPortAllocator( |
@@ -1140,13 +1136,6 @@ class WebRtcSessionTest : public testing::Test { |
TestLoopbackCall(config); |
} |
- void VerifyTransportType(const std::string& content_name, |
- cricket::TransportProtocol protocol) { |
- const cricket::Transport* transport = session_->GetTransport(content_name); |
- ASSERT_TRUE(transport != NULL); |
- EXPECT_EQ(protocol, transport->protocol()); |
- } |
- |
// Adds CN codecs to FakeMediaEngine and MediaDescriptionFactory. |
void AddCNCodecs() { |
const cricket::AudioCodec kCNCodec1(102, "CN", 8000, 0, 1, 0); |
@@ -2535,7 +2524,6 @@ TEST_F(WebRtcSessionTest, TestSetRemoteDescriptionWithoutIce) { |
// too short ice ufrag and pwd strings. |
TEST_F(WebRtcSessionTest, TestSetLocalDescriptionInvalidIceCredentials) { |
Init(); |
- tdesc_factory_->set_protocol(cricket::ICEPROTO_RFC5245); |
mediastream_signaling_.SendAudioVideoStream1(); |
rtc::scoped_ptr<SessionDescriptionInterface> offer(CreateOffer()); |
@@ -2561,7 +2549,6 @@ TEST_F(WebRtcSessionTest, TestSetLocalDescriptionInvalidIceCredentials) { |
// too short ice ufrag and pwd strings. |
TEST_F(WebRtcSessionTest, TestSetRemoteDescriptionInvalidIceCredentials) { |
Init(); |
- tdesc_factory_->set_protocol(cricket::ICEPROTO_RFC5245); |
rtc::scoped_ptr<SessionDescriptionInterface> offer(CreateRemoteOffer()); |
std::string sdp; |
// Modifying ice ufrag and pwd in remote offer with strings smaller than the |
@@ -3078,55 +3065,6 @@ TEST_F(WebRtcSessionTest, TestInitiatorFlagAsReceiver) { |
EXPECT_FALSE(session_->initiator()); |
} |
-// This test verifies the ice protocol type at initiator of the call |
-// if |a=ice-options:google-ice| is present in answer. |
-TEST_F(WebRtcSessionTest, TestInitiatorGIceInAnswer) { |
- Init(); |
- mediastream_signaling_.SendAudioVideoStream1(); |
- SessionDescriptionInterface* offer = CreateOffer(); |
- rtc::scoped_ptr<SessionDescriptionInterface> answer( |
- CreateRemoteAnswer(offer)); |
- SetLocalDescriptionWithoutError(offer); |
- std::string sdp; |
- EXPECT_TRUE(answer->ToString(&sdp)); |
- // Adding ice-options to the session level. |
- InjectAfter("t=0 0\r\n", |
- "a=ice-options:google-ice\r\n", |
- &sdp); |
- SessionDescriptionInterface* answer_with_gice = |
- CreateSessionDescription(JsepSessionDescription::kAnswer, sdp, NULL); |
- // Default offer is ICEPROTO_RFC5245, so we expect responder with |
- // only gice to fail. |
- SetRemoteDescriptionAnswerExpectError(kPushDownTDFailed, answer_with_gice); |
-} |
- |
-// This test verifies the ice protocol type at initiator of the call |
-// if ICE RFC5245 is supported in answer. |
-TEST_F(WebRtcSessionTest, TestInitiatorIceInAnswer) { |
- Init(); |
- mediastream_signaling_.SendAudioVideoStream1(); |
- SessionDescriptionInterface* offer = CreateOffer(); |
- SessionDescriptionInterface* answer = CreateRemoteAnswer(offer); |
- SetLocalDescriptionWithoutError(offer); |
- |
- SetRemoteDescriptionWithoutError(answer); |
- VerifyTransportType("audio", cricket::ICEPROTO_RFC5245); |
- VerifyTransportType("video", cricket::ICEPROTO_RFC5245); |
-} |
- |
-// This test verifies the ice protocol type at receiver side of the call if |
-// receiver decides to use ice RFC 5245. |
-TEST_F(WebRtcSessionTest, TestReceiverIceInOffer) { |
- Init(); |
- mediastream_signaling_.SendAudioVideoStream1(); |
- SessionDescriptionInterface* offer = CreateOffer(); |
- SetRemoteDescriptionWithoutError(offer); |
- SessionDescriptionInterface* answer = CreateAnswer(NULL); |
- SetLocalDescriptionWithoutError(answer); |
- VerifyTransportType("audio", cricket::ICEPROTO_RFC5245); |
- VerifyTransportType("video", cricket::ICEPROTO_RFC5245); |
-} |
- |
// Verifing local offer and remote answer have matching m-lines as per RFC 3264. |
TEST_F(WebRtcSessionTest, TestIncorrectMLinesInRemoteAnswer) { |
Init(); |