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

Unified Diff: remoting/protocol/jingle_session_unittest.cc

Issue 1099203005: Revert of Use standard ICE in Chromoting. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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 | « remoting/protocol/jingle_session.cc ('k') | remoting/protocol/libjingle_transport_factory.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/jingle_session_unittest.cc
diff --git a/remoting/protocol/jingle_session_unittest.cc b/remoting/protocol/jingle_session_unittest.cc
index 9eb829dd93bdf978da3261c4b1ec6014bb706eef..9ec24b9fe740a4d166cca6fd73888c40dfbe85bb 100644
--- a/remoting/protocol/jingle_session_unittest.cc
+++ b/remoting/protocol/jingle_session_unittest.cc
@@ -105,8 +105,7 @@
host_session_.reset(session);
host_session_->SetEventHandler(&host_session_event_handler_);
- session->set_config(standard_ice_ ? SessionConfig::ForTest()
- : SessionConfig::WithLegacyIceForTest());
+ session->set_config(SessionConfig::ForTest());
}
void DeleteSession() {
@@ -154,7 +153,7 @@
scoped_ptr<TransportFactory> host_transport(new LibjingleTransportFactory(
nullptr,
ChromiumPortAllocator::Create(nullptr, network_settings).Pass(),
- network_settings, TransportRole::SERVER));
+ network_settings));
host_server_.reset(new JingleSessionManager(host_transport.Pass()));
host_server_->Init(host_signal_strategy_.get(), &host_server_listener_);
@@ -168,7 +167,7 @@
scoped_ptr<TransportFactory> client_transport(new LibjingleTransportFactory(
nullptr,
ChromiumPortAllocator::Create(nullptr, network_settings).Pass(),
- network_settings, TransportRole::CLIENT));
+ network_settings));
client_server_.reset(
new JingleSessionManager(client_transport.Pass()));
client_server_->Init(client_signal_strategy_.get(),
@@ -291,8 +290,6 @@
scoped_ptr<base::MessageLoopForIO> message_loop_;
- bool standard_ice_ = true;
-
scoped_ptr<FakeSignalStrategy> host_signal_strategy_;
scoped_ptr<FakeSignalStrategy> client_signal_strategy_;
@@ -355,7 +352,7 @@
const buzz::XmlElement* initiate_xml =
host_signal_strategy_->received_messages().front();
const buzz::XmlElement* jingle_element =
- initiate_xml->FirstNamed(buzz::QName("urn:xmpp:jingle:1", "jingle"));
+ initiate_xml->FirstNamed(buzz::QName(kJingleNamespace, "jingle"));
ASSERT_TRUE(jingle_element);
ASSERT_EQ(kClientJid,
jingle_element->Attr(buzz::QName(std::string(), "initiator")));
@@ -381,23 +378,6 @@
// Verify that data can be sent over stream channel.
TEST_F(JingleSessionTest, TestStreamChannel) {
- CreateSessionManagers(1, FakeAuthenticator::ACCEPT);
- ASSERT_NO_FATAL_FAILURE(
- InitiateConnection(1, FakeAuthenticator::ACCEPT, false));
-
- ASSERT_NO_FATAL_FAILURE(CreateChannel());
-
- StreamConnectionTester tester(host_socket_.get(), client_socket_.get(),
- kMessageSize, kMessages);
- tester.Start();
- message_loop_->Run();
- tester.CheckResults();
-}
-
-// Verify that we can still connect using legacy GICE transport.
-TEST_F(JingleSessionTest, TestLegacyIceConnection) {
- standard_ice_ = false;
-
CreateSessionManagers(1, FakeAuthenticator::ACCEPT);
ASSERT_NO_FATAL_FAILURE(
InitiateConnection(1, FakeAuthenticator::ACCEPT, false));
« no previous file with comments | « remoting/protocol/jingle_session.cc ('k') | remoting/protocol/libjingle_transport_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698