net: Make UploadDataStream::Init() asynchronous.

net/base/upload_data.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/base/upload_data.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/file_util.h"
@@ skipping 102 matching lines @@
 uint64 UploadData::Element::BytesRemaining() {
   return GetContentLength() - offset_;
 }
 
 void UploadData::Element::ResetOffset() {
   offset_ = 0;
 
   // Delete the file stream if already opened, so we can reread the file from
   // the beginning.
   if (file_stream_) {
-    // Temporarily allow until fix: http://crbug.com/72001.
-    base::ThreadRestrictions::ScopedAllowIO allow_io;

[eroman, 2012/08/09 05:29:33]

This removal seems unrelated to this changelist (I don't see any changes made to
CloseSync()), or am I missing something?

[hashimoto, 2012/08/09 13:35:02]

UploadData::ResetOffset() is the only user of
UploadData::Element::ResetOffset(), and UploadDataStream::Init() is the only
user of UploadData::ResetOffset().

|allow_io| here can be removed since UploadDataStream::Init() calls this method
only on IO allowed threads.

     file_stream_->CloseSync();
     delete file_stream_;
     file_stream_ = NULL;
   }
 }
 
 FileStream* UploadData::Element::OpenFileStream() {
   scoped_ptr<FileStream> file(new FileStream(NULL));
   int64 rv = file->OpenSync(
       file_path_,
@@ skipping 164 matching lines @@
     return;
 
   for (size_t i = 0; i < elements_.size(); ++i)
     *content_length += elements_[i].GetContentLength();
 }
 
 UploadData::~UploadData() {
 }
 
 }  // namespace net