Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(412)

Unified Diff: net/url_request/url_request.h

Issue 11419034: net: Move ownership of UploadDataStream from URLRequestHttpJob to URLRequest (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Remove a local variable Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/url_request/url_fetcher_core.cc ('k') | net/url_request/url_request.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/url_request/url_request.h
diff --git a/net/url_request/url_request.h b/net/url_request/url_request.h
index b886df3ab2d4b8f8ce9ae471bd2890b2cff240b4..ab17a2c4f58e6f6daff47039b35c54795994a505 100644
--- a/net/url_request/url_request.h
+++ b/net/url_request/url_request.h
@@ -92,6 +92,7 @@ class IOBuffer;
class SSLCertRequestInfo;
class SSLInfo;
class UploadData;
+class UploadDataStream;
class URLRequestContext;
class URLRequestJob;
class X509Certificate;
@@ -398,16 +399,6 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
// and it is permissible for it to be null.
void set_delegate(Delegate* delegate);
- // The data comprising the request message body is specified as a sequence of
- // data segments and/or files containing data to upload. These methods may
- // be called to construct the data sequence to upload, and they may only be
- // called before Start() is called. For POST requests, the user must call
- // SetRequestHeaderBy{Id,Name} to set the Content-Type of the request to the
- // appropriate value before calling Start().
- //
- // When uploading data, bytes_len must be non-zero.
- void AppendBytesToUpload(const char* bytes, int bytes_len); // takes a copy
-
// Indicates that the request body should be sent using chunked transfer
// encoding. This method may only be called before Start() is called.
void EnableChunkedUpload();
@@ -425,8 +416,7 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
void set_upload(UploadData* upload);
// Get the upload data directly.
- const UploadData* get_upload() const;
- UploadData* get_upload_mutable();
+ const UploadDataStream* get_upload() const;
// Returns true if the request has a non-empty message body to upload.
bool has_upload() const;
@@ -763,6 +753,7 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
scoped_refptr<URLRequestJob> job_;
scoped_refptr<UploadData> upload_;
+ scoped_ptr<UploadDataStream> upload_data_stream_;
std::vector<GURL> url_chain_;
GURL first_party_for_cookies_;
GURL delegate_redirect_url_;
« no previous file with comments | « net/url_request/url_fetcher_core.cc ('k') | net/url_request/url_request.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698