Move PseudoTCP and channel auth out of LibjingleTransportFactory.

remoting/protocol/ssl_hmac_channel_authenticator_unittest.cc

Index: remoting/protocol/ssl_hmac_channel_authenticator_unittest.cc
diff --git a/remoting/protocol/ssl_hmac_channel_authenticator_unittest.cc b/remoting/protocol/ssl_hmac_channel_authenticator_unittest.cc
index abf288092506a5c5aeaff5db9bd943a3ba4caa26..5a5038771a82ac5ab0273102eb7b05061c9df262 100644
--- a/remoting/protocol/ssl_hmac_channel_authenticator_unittest.cc
+++ b/remoting/protocol/ssl_hmac_channel_authenticator_unittest.cc
@@ -111,12 +111,14 @@ class SslHmacChannelAuthenticatorTest : public testing::Test {
 
   void OnHostConnected(net::Error error,
                        scoped_ptr<net::StreamSocket> socket) {
+    host_auth_.reset();

[Wez, 2014/09/10 02:29:27]

Why's this change required?

[Sergey Ulanov, 2014/09/10 21:50:59]

Previously SslHmacChannelAuthenticator wasn't calling the callback properly
which would lead to crash when it's deleted when the callback is called. This
change increases coverage for this case, but wouldn't catch the bug we had
before. Updated this test now so it crashes without the fix in
ssl_hmac_channel_authenticator.cc .

[Wez, 2014/09/10 22:19:42]

Acknowledged.

     host_callback_.OnDone(error, socket.get());
     host_socket_ = socket.Pass();
   }
 
   void OnClientConnected(net::Error error,
                          scoped_ptr<net::StreamSocket> socket) {
+    client_auth_.reset();
     client_callback_.OnDone(error, socket.get());
     client_socket_ = socket.Pass();
   }