Index: content/browser/download/download_item.cc |
diff --git a/content/browser/download/download_item.cc b/content/browser/download/download_item.cc |
index e881f0fe7b556431d9d38a1aeb5925bd11a8f4ec..1e2b98bc4f6cfa533a1abf886aa8b8738aa6d34f 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" |
@@ -190,8 +191,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()), |
@@ -224,6 +225,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)); |
@@ -575,16 +580,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) { |
@@ -710,7 +714,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) { |