Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3037)

Unified Diff: content/browser/download/download_manager_impl.cc

Issue 2689373003: Introduce ParallelDownloadJob. (Closed)
Patch Set: nits. Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/download/download_manager_impl.cc
diff --git a/content/browser/download/download_manager_impl.cc b/content/browser/download/download_manager_impl.cc
index 983e26ae07cec13d81b500b4af0a86ed63977cb3..ae778d2eff9fdf89613a62b18834b2c74d786749 100644
--- a/content/browser/download/download_manager_impl.cc
+++ b/content/browser/download/download_manager_impl.cc
@@ -520,16 +520,6 @@ void DownloadManagerImpl::AddUrlDownloader(
url_downloaders_.push_back(std::move(downloader));
}
-void DownloadManagerImpl::RemoveUrlDownloader(UrlDownloader* downloader) {
- for (auto ptr = url_downloaders_.begin(); ptr != url_downloaders_.end();
- ++ptr) {
- if (ptr->get() == downloader) {
- url_downloaders_.erase(ptr);
- return;
- }
- }
-}
-
// static
DownloadInterruptReason DownloadManagerImpl::BeginDownloadRequest(
std::unique_ptr<net::URLRequest> url_request,
@@ -716,6 +706,24 @@ DownloadItem* DownloadManagerImpl::GetDownloadByGuid(const std::string& guid) {
: nullptr;
}
+void DownloadManagerImpl::OnUrlDownloaderStarted(
+ std::unique_ptr<DownloadCreateInfo> download_create_info,
+ std::unique_ptr<ByteStreamReader> stream_reader,
+ const DownloadUrlParameters::OnStartedCallback& callback) {
+ StartDownload(std::move(download_create_info), std::move(stream_reader),
+ callback);
+}
+
+void DownloadManagerImpl::OnUrlDownloaderStopped(UrlDownloader* downloader) {
+ for (auto ptr = url_downloaders_.begin(); ptr != url_downloaders_.end();
+ ++ptr) {
+ if (ptr->get() == downloader) {
+ url_downloaders_.erase(ptr);
+ return;
+ }
+ }
+}
+
void DownloadManagerImpl::GetAllDownloads(DownloadVector* downloads) {
for (const auto& it : downloads_) {
downloads->push_back(it.second.get());
« no previous file with comments | « content/browser/download/download_manager_impl.h ('k') | content/browser/download/download_manager_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698