Changed SPDY's ip connection pooling logic to only add the used IP,

net/socket/ssl_client_socket_pool_unittest.cc

Index: net/socket/ssl_client_socket_pool_unittest.cc
===================================================================
--- net/socket/ssl_client_socket_pool_unittest.cc	(revision 92397)
+++ net/socket/ssl_client_socket_pool_unittest.cc	(working copy)
@@ -30,8 +30,6 @@
 
 namespace net {
 
-namespace {

[willchan no longer on Chromium, 2011/07/17 01:23:45]

If you use the SpdySessionPoolPeer, then you can restore this anonymous
namespace.

[ramant (doing other things), 2011/07/17 08:12:30]

Done.

-
 const int kMaxSockets = 32;
 const int kMaxSocketsPerGroup = 6;
 
@@ -657,8 +655,8 @@
     HostPortProxyPair pair;
   } test_hosts[] = {
     { "www.webkit.org",    "192.168.0.1,192.168.0.5" },
-    { "code.google.com",   "192.168.0.2,192.168.0.3,192.168.0.5" },
-    { "js.webkit.org",     "192.168.0.4,192.168.0.5" },
+    { "code.google.com",   "192.168.0.2,192.168.0.3,192.168.0.1" },
+    { "js.webkit.org",     "192.168.0.4,192.168.0.1" },
   };
 
   host_resolver_.set_synchronous_mode(true);
@@ -712,6 +710,8 @@
   EXPECT_EQ(SSLClientSocket::NextProtoFromString(proto),
             SSLClientSocket::kProtoSPDY2);
 
+  session_->spdy_session_pool()->AddAliases(test_hosts[0].pair);

[willchan no longer on Chromium, 2011/07/17 01:23:45]

Rather than doing this, you should probably just use the address that the
MockClientSocket will return, which you can see in socket_test_util.cc's
MockClientSocket::GetPeerAddress() which shows it returns 192.0.2.33. So you
should probably just use that address as an alias for test_hosts[0] and
test_hosts[2].

[ramant (doing other things), 2011/07/17 08:12:30]

Done.

+
   scoped_refptr<SpdySession> spdy_session;
   rv = session_->spdy_session_pool()->GetSpdySessionFromSocket(
     test_hosts[0].pair, handle.release(), BoundNetLog(), 0,
@@ -727,6 +727,4 @@
 
 // It would be nice to also test the timeouts in SSLClientSocketPool.
 
-}  // namespace
-
 }  // namespace net