net: Give more descriptive names for code around the request merging logic.

net/base/upload_data.cc

Index: net/base/upload_data.cc
diff --git a/net/base/upload_data.cc b/net/base/upload_data.cc
index 019254e48878c431c31304373c0ef5a9bd5041e7..01eaacacc4eee4248e67900f4e6a9793149c96ed 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.

[wtc, 2012/01/24 20:10:57]

Thank you for explaining this to me.

   if (is_chunked_)
     return false;