diff options
| author | satorux@chromium.org <satorux@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-24 23:41:50 +0000 |
|---|---|---|
| committer | satorux@chromium.org <satorux@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-24 23:41:50 +0000 |
| commit | 75577eca2e0090635fb1aaafe46dd3a6bb0af692 (patch) | |
| tree | a3e963dbb16a137ed112d0a5be5286f6b15c60c8 /net/base | |
| parent | 7a747b5992ad9de44e2653a85a6fd78254cfe535 (diff) | |
| download | chromium_src-75577eca2e0090635fb1aaafe46dd3a6bb0af692.zip chromium_src-75577eca2e0090635fb1aaafe46dd3a6bb0af692.tar.gz chromium_src-75577eca2e0090635fb1aaafe46dd3a6bb0af692.tar.bz2 | |
net: Give more descriptive names for code around the request merging logic.
This is a follow-up patch based on wtc's comments on the earlier patch:
http://codereview.chromium.org/9270030/
BUG=72001
TEST=no logic change
Review URL: http://codereview.chromium.org/9284033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118942 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/base')
| -rw-r--r-- | net/base/upload_data.cc | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/net/base/upload_data.cc b/net/base/upload_data.cc index 019254e..01eaaca 100644 --- a/net/base/upload_data.cc +++ b/net/base/upload_data.cc @@ -178,7 +178,11 @@ uint64 UploadData::GetContentLength() { } bool UploadData::IsInMemory() const { - // Chunks are provided as a stream, hence it's not in memory. + // Chunks are in memory, but UploadData does not have all the chunks at + // once. Chunks are provided progressively with AppendChunk() as chunks + // are ready. Check is_chunked_ here, rather than relying on the loop + // below, as there is a case that is_chunked_ is set to true, but the + // first chunk is not yet delivered. if (is_chunked_) return false; |