Change DeterministicMockTCPClientSocket to set the peer address from the connect data.

net/socket/socket_test_util.cc

Index: net/socket/socket_test_util.cc
diff --git a/net/socket/socket_test_util.cc b/net/socket/socket_test_util.cc
index d51d5c65388ee5d2f63d76f88b191d9fe1c8b78a..ef6d38c2e6c5c67955e5ebb45140ccfdfc32102a 100644
--- a/net/socket/socket_test_util.cc
+++ b/net/socket/socket_test_util.cc
@@ -942,7 +942,9 @@ DeterministicMockTCPClientSocket::DeterministicMockTCPClientSocket(
       read_buf_len_(0),
       read_pending_(false),
       data_(data),
-      was_used_to_convey_data_(false) {}
+      was_used_to_convey_data_(false) {
+  peer_addr_ = data->connect_data().peer_addr;

[Ryan Sleevi, 2012/07/11 21:34:14]

protected members make me sad. Took me much too long to find:

1) DeterministicMockTCPClientSocket is-a MockClientSocket (due to the newline
wrapping in the .h that puts AsyncSocket on the new line, rather than the
MockClientSocket - line 683)
2) Where this member was actually defined.

Could you also fix #1 with this change? (and MockUDPClientSocket at line 789 of
the .h)

[Ryan Hamilton, 2012/07/11 21:51:14]

Done, I think.  Would you mind taking a look to confirm?

+}
 
 DeterministicMockTCPClientSocket::~DeterministicMockTCPClientSocket() {}