OnDownloadStarted takes DownloadItem* instead of DownloadId
PS1@r160378
Reviewers:
sky: content/public/test, content/browser/renderer_host/resource_dispatcher_host_impl.cc
binji: chrome/browser/ui/intents
bauerb: chrome/browser/plugins
finnur: chrome/browser/extensions
Done:
rdsmith: *
Review URL: https://chromiumcodereview.appspot.com/11068027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@160971 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/content/browser/download/download_manager_impl.cc b/content/browser/download/download_manager_impl.cc
index 458f5ca..77b4a20a 100644
--- a/content/browser/download/download_manager_impl.cc
+++ b/content/browser/download/download_manager_impl.cc
@@ -348,7 +348,7 @@
}
// We have received a message from DownloadFileManager about a new download.
-content::DownloadId DownloadManagerImpl::StartDownload(
+DownloadItem* DownloadManagerImpl::StartDownload(
scoped_ptr<DownloadCreateInfo> info,
scoped_ptr<content::ByteStreamReader> stream) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
@@ -380,7 +380,7 @@
(delegate_ && delegate_->GenerateFileHash()), bound_net_log,
callback));
- return download_id;
+ return GetDownload(download_id.local());
}
void DownloadManagerImpl::OnDownloadFileCreated(
