diff options
author | willchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-14 18:03:09 +0000 |
---|---|---|
committer | willchan@chromium.org <willchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-14 18:03:09 +0000 |
commit | 2681688fc8449fe1cc6b2f66267fcf04f0327ea1 (patch) | |
tree | 424611ca258e9699de6b63349e1d0fba118144a3 /net/http/http_network_transaction.h | |
parent | 0f337f026ad2bca4d3eda9bb1cb8e5c9dc2dd7f3 (diff) | |
download | chromium_src-2681688fc8449fe1cc6b2f66267fcf04f0327ea1.zip chromium_src-2681688fc8449fe1cc6b2f66267fcf04f0327ea1.tar.gz chromium_src-2681688fc8449fe1cc6b2f66267fcf04f0327ea1.tar.bz2 |
Cleanup StreamFactory && StreamRequest APIs.
Stop refcounting StreamRequest. Establish a clear ownership of the StreamRequest. Deletion implies cancellation. Use ScopedRunnableMethodFactory::NewRunnableMethod() instead of NewRunnableMethod().
Remove Start() from StreamRequest. This is an implementation detail of HttpStreamRequest, so it only exists there now.
BUG=59103
TEST=existing, this is a pure refactor.
Review URL: http://codereview.chromium.org/3746002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62581 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http/http_network_transaction.h')
-rw-r--r-- | net/http/http_network_transaction.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/net/http/http_network_transaction.h b/net/http/http_network_transaction.h index 7ddee29..a8d371d 100644 --- a/net/http/http_network_transaction.h +++ b/net/http/http_network_transaction.h @@ -33,7 +33,7 @@ class SSLHostInfo; struct HttpRequestInfo; class HttpNetworkTransaction : public HttpTransaction, - public StreamFactory::StreamRequestDelegate { + public StreamRequest::Delegate { public: explicit HttpNetworkTransaction(HttpNetworkSession* session); @@ -58,7 +58,7 @@ class HttpNetworkTransaction : public HttpTransaction, virtual uint64 GetUploadProgress() const; virtual void SetSSLHostInfo(SSLHostInfo* host_info); - // StreamRequestDelegate methods: + // StreamRequest::Delegate methods: virtual void OnStreamReady(HttpStream* stream); virtual void OnStreamFailed(int status); virtual void OnCertificateError(int status, const SSLInfo& ssl_info); @@ -211,7 +211,7 @@ class HttpNetworkTransaction : public HttpTransaction, ProxyInfo proxy_info_; - scoped_refptr<StreamFactory::StreamRequestJob> stream_request_; + scoped_ptr<StreamRequest> stream_request_; scoped_ptr<HttpStream> stream_; // True if we've validated the headers that the stream parser has returned. |