diff options
author | jgraettinger@chromium.org <jgraettinger@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-05 03:36:24 +0000 |
---|---|---|
committer | jgraettinger@chromium.org <jgraettinger@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-05 03:36:24 +0000 |
commit | 975da41aab312ba7cc270462d02931452f51d953 (patch) | |
tree | d40126c5d42b09caded96a6de8ebeeb042071f3e /url/gurl_unittest.cc | |
parent | da0c65087c4ed0811fbdb00bacc62bc8429b27e7 (diff) | |
download | chromium_src-975da41aab312ba7cc270462d02931452f51d953.zip chromium_src-975da41aab312ba7cc270462d02931452f51d953.tar.gz chromium_src-975da41aab312ba7cc270462d02931452f51d953.tar.bz2 |
Defer SpdySession destruction to support closing writes
Replace the STATE_CLOSED availability state with STATE_DRAINING, where:
* The read-pump will not run, and no frames can be read.
* No new frames may be queued in the sessions's write_queue.
* However, the write-pump will continue to run.
SpdySession's write-pump will destroy the session iff it's draining, the
write queue is empty and no in-flight writes remain. As this (and
~SpdySessionPool()) are now the only locations where SpdySessions may be
destroyed, session lifetime is more know-able and some WeakPtrs are
removed.
Some tests have been updated to include closing RST_STREAMS sent by the
session. This reflects current behavior. A future CL will add GOAWAY
frames upon session errors.
BUG=375033,379469
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273680
Review URL: https://codereview.chromium.org/305823003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275005 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'url/gurl_unittest.cc')
0 files changed, 0 insertions, 0 deletions