summaryrefslogtreecommitdiffstats
path: root/net/spdy/spdy_session_pool.cc
Commit message (Collapse)AuthorAgeFilesLines
* Remove the references to HttpNetworkSession in SpdySessionrch@chromium.org2010-09-251-6/+7
| | | | | | | | | | | | | | | | and replace it with a reference to the SpdySessionPool and SpdySettingStorage, which are actually required. Modified SpdySessionPool to take a pointer to SpdySettingsStorage instead of to HttpNetworkSession in both GetSpdySessionFromSocket() Get(). Modified HttpStreamRequest (and various unit tests) accordingly when calling these methods. Review URL: http://codereview.chromium.org/3400024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60590 0039d316-1c4b-4281-b951-d872f2087c98
* Flush SSL pools after SSL config changes.csilv@chromium.org2010-09-041-1/+10
| | | | | | | | BUG=46043 TEST=Verify that SSL pools are flushed after user makes SSL change. Review URL: http://codereview.chromium.org/3274004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58567 0039d316-1c4b-4281-b951-d872f2087c98
* Add logging to the spdy session pool.mbelshe@chromium.org2010-08-301-2/+16
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3280003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57833 0039d316-1c4b-4281-b951-d872f2087c98
* SpdySessionPool closes down sessions accurately now.erikchen@google.com2010-08-191-14/+30
| | | | | | | | | | This may or may not fix 50265. Regardless, the previous behavior was incorrect. TEST=none BUG=none Review URL: http://codereview.chromium.org/3150023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56696 0039d316-1c4b-4281-b951-d872f2087c98
* SpdySessionPool now identifies SpdySessions by both HostPortPair and proxy ↵erikchen@google.com2010-08-021-22/+30
| | | | | | | | | | | | settings. This ensures SpdySessions are not improperly reused. TEST=net_unittests BUG=49874 Review URL: http://codereview.chromium.org/3047032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54577 0039d316-1c4b-4281-b951-d872f2087c98
* SPDY now always uses http_network_transaction instead of ↵erikchen@google.com2010-07-231-3/+4
| | | | | | | | | | | | | | | | spdy_network_transaction. It was previously possible to use spdy_network_transaction using the command line flags: --use-spdy=no-ssl --use-spdy=no-compress This does not affect instances of chrome that are not run with --use-spdy. Also changed spdy_network_transaction_unittest so that all tests are run with 3 different connection configurations: spdy over npn, spdy over ssl, and spdy without ssl. TEST=net_unittests BUG=49082 Review URL: http://codereview.chromium.org/3048003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53548 0039d316-1c4b-4281-b951-d872f2087c98
* Client attempts to start a new spdy transaction with a session that is ↵erikchen@google.com2010-07-091-5/+12
| | | | | | | | | | | closing down. TEST=net_unittests BUG=47455,48503 Review URL: http://codereview.chromium.org/2841029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52001 0039d316-1c4b-4281-b951-d872f2087c98
* SPDY: Make sure we don't try to send https/wss over an unauthenticated, but ↵willchan@chromium.org2010-06-281-4/+6
| | | | | | | | | | encrypted SSL socket. BUG=46924 Review URL: http://codereview.chromium.org/2805039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50997 0039d316-1c4b-4281-b951-d872f2087c98
* Massively simplify the NetworkChangeNotifier infrastructure:pkasting@chromium.org2010-06-251-7/+8
| | | | | | | | | | | | | | * Use a process-wide object (singleton pattern) * Create/destroy this object on the main thread, make it outlive all consumers * Make observer-related functions threadsafe As a result, the notifier can now be used by any thread (eliminating things like NetworkChangeObserverProxy and NetworkChangeNotifierProxy, and expanding its usefulness); its creation and inner workings are much simplified (eliminating implementation-specific classes); and it is simpler to access (eliminating things like NetworkChangeNotifierThread and a LOT of passing pointers around). BUG=none TEST=Unittests; network changes still trigger notifications Review URL: http://codereview.chromium.org/2802015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50895 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash in GetSpdySessionFromSSLSocket in the case where thembelshe@chromium.org2010-06-241-6/+8
| | | | | | | | | | server immediately hangs up after negotiating SPDY via NPN. BUG=46369 TEST=SpdyNPNServerHangup Review URL: http://codereview.chromium.org/2862027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50689 0039d316-1c4b-4281-b951-d872f2087c98
* SPDY: Minor code cleanup.willchan@chromium.org2010-06-111-1/+3
| | | | | | Review URL: http://codereview.chromium.org/2776008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49590 0039d316-1c4b-4281-b951-d872f2087c98
* Make SpdySessionPool observe network changes.willchan@chromium.org2010-06-071-1/+9
| | | | | | | | BUG=40457 Review URL: http://codereview.chromium.org/2627003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49075 0039d316-1c4b-4281-b951-d872f2087c98
* Fix leaky spdy sessions and remove suppression.vandebo@chromium.org2010-06-031-2/+3
| | | | | | | | | BUG=44738 TEST=heap checker no longer complains and tests still pass Review URL: http://codereview.chromium.org/2477004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48837 0039d316-1c4b-4281-b951-d872f2087c98
* Reland 47342.willchan@chromium.org2010-05-161-2/+2
| | | | | | | | | | | Fixes the merge issue this time. Also fixes a TODO in another change that was blocked on this one. Support SpdySession as a new NetLog source type. Start logging some more SPDY control frames. Original code review: http://codereview.chromium.org/2102003/show BUG=43237 Review URL: http://codereview.chromium.org/2114003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47376 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 47342 - Support SpdySession as a new NetLog source type. Start ↵willchan@chromium.org2010-05-151-2/+2
| | | | | | | | | | | | | logging some more SPDY control frames. BUG=43237 Review URL: http://codereview.chromium.org/2102003 TBR=willchan@chromium.org Review URL: http://codereview.chromium.org/2118003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47343 0039d316-1c4b-4281-b951-d872f2087c98
* Support SpdySession as a new NetLog source type. Start logging some more ↵willchan@chromium.org2010-05-151-2/+2
| | | | | | | | | | SPDY control frames. BUG=43237 Review URL: http://codereview.chromium.org/2102003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47342 0039d316-1c4b-4281-b951-d872f2087c98
* Plumb NetLog into SpdySession.willchan@chromium.org2010-05-121-4/+6
| | | | | | | | TODO(willchan): Start logging events for SpdySession. Review URL: http://codereview.chromium.org/2059004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47047 0039d316-1c4b-4281-b951-d872f2087c98
* Fix cases where SpdySession can double-remove from the SpdySessionPool.mbelshe@chromium.org2010-04-231-0/+1
| | | | | | | | | BUG=42229 TEST=SpdyNetworkTransactionTest.GoAwayWithActiveStream Review URL: http://codereview.chromium.org/1747006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45447 0039d316-1c4b-4281-b951-d872f2087c98
* With the GOAWAY code we now can have a case where we attemptmbelshe@chromium.org2010-04-211-1/+2
| | | | | | | | | | | | | | to remove the session from the pool twice. Recover, instead of abort, on that case. Caught by existing test case already. TBR=willchan BUG=none TEST=SpdyNetworkTransactionTest.GoAwayWithActiveStream git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45211 0039d316-1c4b-4281-b951-d872f2087c98
* Add command line option for changing the max number of SPDY sessions permbelshe@chromium.org2010-03-051-2/+4
| | | | | | | | | | | | | | | domain. The default remains at 1. Command line usage to set it to 13: chrome.exe --use-spdy --max-spdy-sessions-per-domain=13 BUG=none TEST=none Review URL: http://codereview.chromium.org/669169 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40775 0039d316-1c4b-4281-b951-d872f2087c98
* Fix SpdySessionPool to take a host port pair instead of just the host.willchan@chromium.org2010-02-261-26/+23
| | | | | | | | BUG=28595 Review URL: http://codereview.chromium.org/660107 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40077 0039d316-1c4b-4281-b951-d872f2087c98
* Make explicit the assumption that SpdySession::InitializeWithSocket takeswtc@chromium.org2010-02-201-2/+2
| | | | | | | | | | | an SSLClientSocket by renaming the methods and adding a DCHECK. R=willchan BUG=none TEST=No compilation errors. Review URL: http://codereview.chromium.org/650085 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39528 0039d316-1c4b-4281-b951-d872f2087c98
* The great Flip -> Spdy rename.mbelshe@chromium.org2010-02-081-34/+34
| | | | | | | | | BUG=30747 TEST=none Review URL: http://codereview.chromium.org/580009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38386 0039d316-1c4b-4281-b951-d872f2087c98
* Rename all files from flip* to spdy*.mbelshe@chromium.org2010-02-061-0/+121
I haven't yet renamed the classes. BUG=30747 TEST=none Review URL: http://codereview.chromium.org/582001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38315 0039d316-1c4b-4281-b951-d872f2087c98