net: Explicitly close a file stream in UploadDataStream.

net/base/upload_data_stream.cc

Index: net/base/upload_data_stream.cc
diff --git a/net/base/upload_data_stream.cc b/net/base/upload_data_stream.cc
index 2d2a3a56fab3c48080cb6bb0de2cd2d57bbb76bb..c777c5eca056bd6a83589971c1ce01f81bf99dfa 100644
--- a/net/base/upload_data_stream.cc
+++ b/net/base/upload_data_stream.cc
@@ -165,7 +165,12 @@ void UploadDataStream::AdvanceToNextElement() {
   ++element_index_;
   element_offset_ = 0;
   element_file_bytes_remaining_ = 0;
-  element_file_stream_.reset();
+  if (element_file_stream_.get()) {
+    // Temporarily allow until fix: http://crbug.com/72001.
+    base::ThreadRestrictions::ScopedAllowIO allow_io;
+    element_file_stream_.Close();

[eroman, 2012/02/01 21:43:28]

The call to Close() isn't strictly necessary right?
Destruction should implicitly close it too.

[satorux1, 2012/02/01 21:46:09]

you are right, but wanted to make it clear that file access is performed here. I
also plan to make file closing asynchronous in the upcoming patches.

btw, element_file_stream_.Close() should have been
element_file_stream_->Close(). patch set 2 fixed this.

+    element_file_stream_.reset();
+  }
 }
 
 bool UploadDataStream::IsEOF() const {