Fix a bug in the mock sockets. For SSL sockets, when they were...

net/socket/socket_test_util.cc

Index: net/socket/socket_test_util.cc
===================================================================
--- net/socket/socket_test_util.cc	(revision 56039)
+++ net/socket/socket_test_util.cc	(working copy)
@@ -361,12 +361,12 @@
   int rv = transport_->socket()->Connect(connect_callback);
   if (rv == net::OK) {
     delete connect_callback;
+    if (data_->connect.result == net::OK)
+      connected_ = true;

[eroman, 2010/08/21 02:48:09]

It would be better to only set connected_ to true right before we call the user
callback.

     if (data_->connect.async) {
       RunCallbackAsync(callback, data_->connect.result);
       return net::ERR_IO_PENDING;
     }
-    if (data_->connect.result == net::OK)
-      connected_ = true;
     return data_->connect.result;
   }
   return rv;