diff options
| author | willchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-26 22:20:54 +0000 |
|---|---|---|
| committer | willchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-26 22:20:54 +0000 |
| commit | 2ff8b3133033d6130ea87df3ab2290a226c7bca7 (patch) | |
| tree | 74ee45d17428c359a0bd8db8b97e51eeb4b7c44c /net/net.gyp | |
| parent | 106113b3dde22f7d8b3a7f9a00a8884668cda782 (diff) | |
| download | chromium_src-2ff8b3133033d6130ea87df3ab2290a226c7bca7.zip chromium_src-2ff8b3133033d6130ea87df3ab2290a226c7bca7.tar.gz chromium_src-2ff8b3133033d6130ea87df3ab2290a226c7bca7.tar.bz2 | |
SPDY: Fix Alternate-Protocol.
(1) In DoInitConnection() we do the existing spdy session check. If it exists there, then we assuem it exists in DoSpdySendRequest(). Unfortunately, we didn't do the same check. Use a member variable to store the HostPortPair.
(2) In DoInitConnection(), we used the scheme://urlhost:urlport as the connection group. With Alternate-Protocol, we used the scheme://urlhost:urlport even though we were connecting to a different port, with a different protocol (TLS). This means we would mix conflicting sockets in the ClientSocketPool. I fix this by dropping scheme://, since it's unnecessary, and would cause us not to share SSL sockets in different connection groups (since the specified scheme might be http://, but due to Alternate-Protocol, we actually do an SSL connect). I also don't use the urlhost:urlport, but use the host:port that we actually connect to.
TODO(willchan):
Fix Alternate-Protocol so it works properly with proxies. I need to change CONNECT for http proxies and patch the SOCKs connects.
Review URL: http://codereview.chromium.org/1755005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45627 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/net.gyp')
| -rw-r--r-- | net/net.gyp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/net.gyp b/net/net.gyp index 2a5dc4d..7ef7bf2 100644 --- a/net/net.gyp +++ b/net/net.gyp @@ -718,6 +718,7 @@ 'spdy/spdy_protocol_test.cc', 'spdy/spdy_session_unittest.cc', 'spdy/spdy_stream_unittest.cc', + 'spdy/spdy_test_util.h', 'url_request/url_request_unittest.cc', 'url_request/url_request_unittest.h', 'websockets/websocket_handshake_draft75_unittest.cc', |