diff options
| author | rtenneti@chromium.org <rtenneti@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-19 21:58:04 +0000 |
|---|---|---|
| committer | rtenneti@chromium.org <rtenneti@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-19 21:58:04 +0000 |
| commit | 46da33beb42f48672659fed469da4b15ed927776 (patch) | |
| tree | fb5b8aa70035f58d722ef1211da817b3f5365b00 /net/spdy/spdy_session_pool.h | |
| parent | b0a7510c48c807338edf5ffa376d914e09adbf81 (diff) | |
| download | chromium_src-46da33beb42f48672659fed469da4b15ed927776.zip chromium_src-46da33beb42f48672659fed469da4b15ed927776.tar.gz chromium_src-46da33beb42f48672659fed469da4b15ed927776.tar.bz2 | |
Changed SPDY's ip connection pooling logic to only add the used IP,
rather than all the IPs for the hostname, to the SpdyAliasMap.
This is to fix the following problem:
When we establish a new SPDY connection, we add all IP addresses for
that hostname to our alias map. Therefore, if we connect to host foo
which maps to IPs X, Y, and Z, but the actual IP we connect to is X,
then if we try to get a SPDY connection to host bar which maps to
IPs Y and Z, then currently we will share the SPDY connection to host
foo on IP X.
BUG=89094
R=willchan
TEST=spdy unit tests
Review URL: http://codereview.chromium.org/7349023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93105 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/spdy/spdy_session_pool.h')
| -rw-r--r-- | net/spdy/spdy_session_pool.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/spdy/spdy_session_pool.h b/net/spdy/spdy_session_pool.h index 74b8874..366ede4 100644 --- a/net/spdy/spdy_session_pool.h +++ b/net/spdy/spdy_session_pool.h @@ -163,8 +163,8 @@ class NET_API SpdySessionPool bool LookupAddresses(const HostPortProxyPair& pair, AddressList* addresses) const; - // Add a set of |addresses| as IP-equivalent addresses for |pair|. - void AddAliases(const AddressList& addresses, const HostPortProxyPair& pair); + // Add |endpoint| as an IP-equivalent address for |pair|. + void AddAlias(const addrinfo* address, const HostPortProxyPair& pair); // Remove all aliases for |pair| from the aliases table. void RemoveAliases(const HostPortProxyPair& pair); |