diff options
author | sergeyu <sergeyu@chromium.org> | 2014-09-12 13:43:01 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-09-12 20:49:52 +0000 |
commit | 9cb142f0cb33af89ff478e69e803fdf32f09006e (patch) | |
tree | 20fdc88e708d5b247b6c6f84416c8a7fcabfdfdd /remoting/protocol/authenticator_test_base.cc | |
parent | 02247c02b993b041e0da5a1f03e2f173ff81e888 (diff) | |
download | chromium_src-9cb142f0cb33af89ff478e69e803fdf32f09006e.zip chromium_src-9cb142f0cb33af89ff478e69e803fdf32f09006e.tar.gz chromium_src-9cb142f0cb33af89ff478e69e803fdf32f09006e.tar.bz2 |
Move PseudoTCP and channel auth out of LibjingleTransportFactory.
Previously TransportFactory interface was responsible for creation
and initialization of several protocol layers, including PseudoTCP and
authentication (TLS). Simplified it so now it only creates raw datagram
transport channel. PseudoTcpChannelFactory is now responsible for
setting up PseudoTcpAdapter and AuthenticatingChannelFactory takes care
of channel authentication. Also added DatagramChannelFactory for
Datagram channels.
This change will make it possible to replace PseudoTcpChannelFactory
with an object that creates SCTP-based channels.
Also fixed a bug in SslHmacChannelAuthenticator. It wasn't working
properly when deleted from the callback. (base::Callback objects
shouldn't be deleted while being called because when deleted they
also destroy reference parameters values they are holding).
BUG=402993
Review URL: https://codereview.chromium.org/551173004
Cr-Commit-Position: refs/heads/master@{#294653}
Diffstat (limited to 'remoting/protocol/authenticator_test_base.cc')
-rw-r--r-- | remoting/protocol/authenticator_test_base.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/remoting/protocol/authenticator_test_base.cc b/remoting/protocol/authenticator_test_base.cc index 2ef1658..294dacf 100644 --- a/remoting/protocol/authenticator_test_base.cc +++ b/remoting/protocol/authenticator_test_base.cc @@ -10,6 +10,7 @@ #include "base/path_service.h" #include "base/test/test_timeouts.h" #include "base/timer/timer.h" +#include "net/base/net_errors.h" #include "net/base/test_data_directory.h" #include "remoting/base/rsa_key_pair.h" #include "remoting/protocol/authenticator.h" @@ -157,14 +158,14 @@ void AuthenticatorTestBase::RunChannelAuth(bool expected_fail) { } void AuthenticatorTestBase::OnHostConnected( - net::Error error, + int error, scoped_ptr<net::StreamSocket> socket) { host_callback_.OnDone(error); host_socket_ = socket.Pass(); } void AuthenticatorTestBase::OnClientConnected( - net::Error error, + int error, scoped_ptr<net::StreamSocket> socket) { client_callback_.OnDone(error); client_socket_ = socket.Pass(); |