Index: content/browser/download/download_item.cc |
diff --git a/content/browser/download/download_item.cc b/content/browser/download/download_item.cc |
index 080987289601b82facb9fbacb99319b1b820f7ad..7f1b5a5d040a15d48667d697e81d2b1419f99c18 100644 |
--- a/content/browser/download/download_item.cc |
+++ b/content/browser/download/download_item.cc |
@@ -19,6 +19,7 @@ |
#include "content/browser/download/download_file.h" |
#include "content/browser/download/download_create_info.h" |
#include "content/browser/download/download_file_manager.h" |
+#include "content/browser/download/download_id.h" |
#include "content/browser/download/download_manager.h" |
#include "content/browser/download/download_manager_delegate.h" |
#include "content/browser/download/download_persistent_store_info.h" |
@@ -191,8 +192,8 @@ DownloadItem::DownloadItem(DownloadManager* download_manager, |
const FilePath& path, |
const GURL& url, |
bool is_otr, |
- int download_id) |
- : download_id_(download_id), |
+ DownloadId download_id) |
+ : download_id_(download_id.local()), |
full_path_(path), |
url_chain_(1, url), |
referrer_url_(GURL()), |
@@ -225,6 +226,10 @@ DownloadItem::~DownloadItem() { |
download_manager_->AssertQueueStateConsistent(this); |
} |
+DownloadId DownloadItem::global_id() const { |
+ return DownloadId(download_manager_, id()); |
+} |
+ |
void DownloadItem::AddObserver(Observer* observer) { |
// TODO(rdsmith): Change to DCHECK after http://crbug.com/85408 resolved. |
CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
@@ -577,16 +582,15 @@ void DownloadItem::OnDownloadCompleting(DownloadFileManager* file_manager) { |
if (NeedsRename()) { |
BrowserThread::PostTask(BrowserThread::FILE, FROM_HERE, |
NewRunnableMethod(file_manager, |
- &DownloadFileManager::RenameCompletingDownloadFile, id(), |
+ &DownloadFileManager::RenameCompletingDownloadFile, global_id(), |
GetTargetFilePath(), safety_state() == SAFE)); |
return; |
} |
Completed(); |
- BrowserThread::PostTask(BrowserThread::FILE, FROM_HERE, |
- NewRunnableMethod(file_manager, &DownloadFileManager::CompleteDownload, |
- id())); |
+ BrowserThread::PostTask(BrowserThread::FILE, FROM_HERE, NewRunnableMethod( |
+ file_manager, &DownloadFileManager::CompleteDownload, global_id())); |
} |
void DownloadItem::OnDownloadRenamedToFinalName(const FilePath& full_path) { |
@@ -712,7 +716,7 @@ void DownloadItem::OffThreadCancel(DownloadFileManager* file_manager) { |
BrowserThread::PostTask( |
BrowserThread::FILE, FROM_HERE, |
NewRunnableMethod( |
- file_manager, &DownloadFileManager::CancelDownload, download_id_)); |
+ file_manager, &DownloadFileManager::CancelDownload, global_id())); |
} |
void DownloadItem::Init(bool active) { |