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

Side by Side Diff: remoting/protocol/webrtc_transport_unittest.cc

Issue 1510333002: Cleanups in WebrtcTransport. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "remoting/protocol/webrtc_transport.h" 5 #include "remoting/protocol/webrtc_transport.h"
6 6
7 #include "base/message_loop/message_loop.h" 7 #include "base/message_loop/message_loop.h"
8 #include "base/run_loop.h" 8 #include "base/run_loop.h"
9 #include "jingle/glue/thread_wrapper.h" 9 #include "jingle/glue/thread_wrapper.h"
10 #include "net/base/io_buffer.h" 10 #include "net/base/io_buffer.h"
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 ASSERT_TRUE(target_transport); 83 ASSERT_TRUE(target_transport);
84 EXPECT_TRUE((*target_transport) 84 EXPECT_TRUE((*target_transport)
85 ->ProcessTransportInfo(transport_info.get())); 85 ->ProcessTransportInfo(transport_info.get()));
86 } 86 }
87 87
88 protected: 88 protected:
89 void InitializeConnection() { 89 void InitializeConnection() {
90 signal_strategy_.reset(new FakeSignalStrategy(kTestJid)); 90 signal_strategy_.reset(new FakeSignalStrategy(kTestJid));
91 91
92 host_transport_factory_.reset(new WebrtcTransportFactory( 92 host_transport_factory_.reset(new WebrtcTransportFactory(
93 signal_strategy_.get(), 93 jingle_glue::JingleThreadWrapper::current(), signal_strategy_.get(),
94 ChromiumPortAllocatorFactory::Create(network_settings_, nullptr), 94 ChromiumPortAllocatorFactory::Create(network_settings_, nullptr),
95 TransportRole::SERVER)); 95 TransportRole::SERVER));
96 host_transport_ = host_transport_factory_->CreateTransport(); 96 host_transport_ = host_transport_factory_->CreateTransport();
97 host_authenticator_.reset(new FakeAuthenticator( 97 host_authenticator_.reset(new FakeAuthenticator(
98 FakeAuthenticator::HOST, 0, FakeAuthenticator::ACCEPT, false)); 98 FakeAuthenticator::HOST, 0, FakeAuthenticator::ACCEPT, false));
99 99
100 client_transport_factory_.reset(new WebrtcTransportFactory( 100 client_transport_factory_.reset(new WebrtcTransportFactory(
101 signal_strategy_.get(), 101 jingle_glue::JingleThreadWrapper::current(), signal_strategy_.get(),
102 ChromiumPortAllocatorFactory::Create(network_settings_, nullptr), 102 ChromiumPortAllocatorFactory::Create(network_settings_, nullptr),
103 TransportRole::CLIENT)); 103 TransportRole::CLIENT));
104 client_transport_ = client_transport_factory_->CreateTransport(); 104 client_transport_ = client_transport_factory_->CreateTransport();
105 host_authenticator_.reset(new FakeAuthenticator( 105 host_authenticator_.reset(new FakeAuthenticator(
106 FakeAuthenticator::CLIENT, 0, FakeAuthenticator::ACCEPT, false)); 106 FakeAuthenticator::CLIENT, 0, FakeAuthenticator::ACCEPT, false));
107 107
108 // Connect signaling between the two WebrtcTransport objects. 108 // Connect signaling between the two WebrtcTransport objects.
109 host_event_handler_.set_transport_info_callback( 109 host_event_handler_.set_transport_info_callback(
110 base::Bind(&WebrtcTransportTest::ProcessTransportInfo, 110 base::Bind(&WebrtcTransportTest::ProcessTransportInfo,
111 base::Unretained(this), &client_transport_)); 111 base::Unretained(this), &client_transport_));
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
217 const int kMessages = 100; 217 const int kMessages = 100;
218 StreamConnectionTester tester(host_socket_.get(), client_socket_.get(), 218 StreamConnectionTester tester(host_socket_.get(), client_socket_.get(),
219 kMessageSize, kMessages); 219 kMessageSize, kMessages);
220 tester.Start(); 220 tester.Start();
221 message_loop_.Run(); 221 message_loop_.Run();
222 tester.CheckResults(); 222 tester.CheckResults();
223 } 223 }
224 224
225 } // namespace protocol 225 } // namespace protocol
226 } // namespace remoting 226 } // namespace remoting
OLDNEW
« remoting/protocol/webrtc_transport.cc ('K') | « remoting/protocol/webrtc_transport.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698