Replace the DownloadFileManager with direct ownership of DownloadFileImpl

content/browser/download/base_file.cc

Index: content/browser/download/base_file.cc
diff --git a/content/browser/download/base_file.cc b/content/browser/download/base_file.cc
index cc7871fcedde3d1a34ba8488651b4e4f96a7b20a..0a1eae0d131da286b5255827ec384dd9c0cbcc9c 100644
--- a/content/browser/download/base_file.cc
+++ b/content/browser/download/base_file.cc
@@ -220,7 +220,6 @@ BaseFile::BaseFile(const FilePath& full_path,
       calculate_hash_(calculate_hash),
       detached_(false),
       bound_net_log_(bound_net_log) {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
   memcpy(sha256_hash_, kEmptySha256Hash, kSha256HashLen);
   if (file_stream_.get()) {

[asanka, 2012/10/10 16:04:40]

Suggestion: If the constructor can be called from any thread, shall we move this
use of file_stream_ to Initialize() or Open()? Open() already calls
SetBoundNetLogSource().

[Randy Smith (Not in Mondays), 2012/10/16 18:30:29]

Done.

     file_stream_->SetBoundNetLogSource(bound_net_log_);