| 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();
|
|
|