diff options
| author | mmenke@chromium.org <mmenke@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-29 22:34:51 +0000 |
|---|---|---|
| committer | mmenke@chromium.org <mmenke@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-29 22:34:51 +0000 |
| commit | 32aa415f8eab2790052277fecfb843e8ad2ba38c (patch) | |
| tree | d315900a8d6f25f527feeab9ba2df8b975dfb107 /net/spdy/spdy_http_stream.cc | |
| parent | 3abba2674d55de917a2ac4d3d2534746845dd076 (diff) | |
| download | chromium_src-32aa415f8eab2790052277fecfb843e8ad2ba38c.zip chromium_src-32aa415f8eab2790052277fecfb843e8ad2ba38c.tar.gz chromium_src-32aa415f8eab2790052277fecfb843e8ad2ba38c.tar.bz2 | |
Remove HttpStream::GetResponseInfo, which is not currently used.
HttpStream::GetResponseInfo returned a pointer to an object not owned
by the HttpStream, so could result in returning freed memory, in the
case the stream outlives its client. This happens in the case an
HttpResponseBodyDrainer is used.
As the method isn't used anywhere, seems a good idea to get rid of it.
BUG=none
Review URL: https://codereview.chromium.org/303443009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273629 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/spdy/spdy_http_stream.cc')
| -rw-r--r-- | net/spdy/spdy_http_stream.cc | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/net/spdy/spdy_http_stream.cc b/net/spdy/spdy_http_stream.cc index a327768..055d46f 100644 --- a/net/spdy/spdy_http_stream.cc +++ b/net/spdy/spdy_http_stream.cc @@ -89,10 +89,6 @@ int SpdyHttpStream::InitializeStream(const HttpRequestInfo* request_info, return rv; } -const HttpResponseInfo* SpdyHttpStream::GetResponseInfo() const { - return response_info_; -} - UploadProgress SpdyHttpStream::GetUploadProgress() const { if (!request_info_ || !HasUploadData()) return UploadProgress(); |