diff options
| author | erikchen@google.com <erikchen@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-27 20:04:03 +0000 |
|---|---|---|
| committer | erikchen@google.com <erikchen@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-27 20:04:03 +0000 |
| commit | 6c6ea177b4c8196bbf5ea214c9082e5027997a5a (patch) | |
| tree | 610f59a269c5ac50a86b8f2386cd368aff8ce56c /net/spdy/spdy_session_pool.h | |
| parent | fab8d14e7b94307a6f9965ccb59b882b9cc79ff6 (diff) | |
| download | chromium_src-6c6ea177b4c8196bbf5ea214c9082e5027997a5a.zip chromium_src-6c6ea177b4c8196bbf5ea214c9082e5027997a5a.tar.gz chromium_src-6c6ea177b4c8196bbf5ea214c9082e5027997a5a.tar.bz2 | |
SPDY sends RST_STREAM upon cancelling request, or bad header parse data.
Also fix tsan failure for spdy_http_stream_unittest.
Attempted to commit in http://codereview.chromium.org/3014030/show, ran into
different tsan failure.
TEST=net_unittests, tsan on windows for SpdyHttpStreamTest.
BUG=46589, 47478, 50198
Review URL: http://codereview.chromium.org/2811072
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53829 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/spdy/spdy_session_pool.h')
| -rw-r--r-- | net/spdy/spdy_session_pool.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/spdy/spdy_session_pool.h b/net/spdy/spdy_session_pool.h index 362fdc9..1c18438 100644 --- a/net/spdy/spdy_session_pool.h +++ b/net/spdy/spdy_session_pool.h @@ -71,7 +71,8 @@ class SpdySessionPool // Close all Spdy Sessions; used for debugging. void CloseAllSessions(); - // Removes a SpdySession from the SpdySessionPool. + // Removes a SpdySession from the SpdySessionPool. This should only be called + // by SpdySession, because otherwise session->state_ is not set to CLOSED. void Remove(const scoped_refptr<SpdySession>& session); // NetworkChangeNotifier::Observer methods: |