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

Unified Diff: chrome/browser/safe_browsing/incident_reporting/download_metadata_manager.cc

Issue 2821193005: Rewrite base::Bind to base::BindOnce with base_bind_rewriters in //chrome/browser/safe_browsing (Closed)
Patch Set: Created 3 years, 8 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: chrome/browser/safe_browsing/incident_reporting/download_metadata_manager.cc
diff --git a/chrome/browser/safe_browsing/incident_reporting/download_metadata_manager.cc b/chrome/browser/safe_browsing/incident_reporting/download_metadata_manager.cc
index d503f1e12162de7f586fcf922c05fc2c1b713f1f..b57d1842a33347ff67bec90986f9dde0782e0256 100644
--- a/chrome/browser/safe_browsing/incident_reporting/download_metadata_manager.cc
+++ b/chrome/browser/safe_browsing/incident_reporting/download_metadata_manager.cc
@@ -403,10 +403,11 @@ void DownloadMetadataManager::GetDownloadDetails(
// Fire off a task to load the details and return them to the caller.
DownloadMetadata* metadata = new DownloadMetadata();
read_runner_->PostTaskAndReply(
- FROM_HERE, base::Bind(&ReadMetadataOnWorkerPool,
- GetMetadataPath(browser_context), metadata),
- base::Bind(&ReturnResults, callback,
- base::Passed(base::WrapUnique(metadata))));
+ FROM_HERE,
+ base::BindOnce(&ReadMetadataOnWorkerPool,
+ GetMetadataPath(browser_context), metadata),
+ base::BindOnce(&ReturnResults, callback,
+ base::Passed(base::WrapUnique(metadata))));
}
content::DownloadManager*
@@ -566,18 +567,17 @@ void DownloadMetadataManager::ManagerContext::ReadMetadata() {
// Do not block shutdown on this read since nothing will come of it.
read_runner_->PostTaskAndReply(
FROM_HERE,
- base::Bind(&ReadMetadataOnWorkerPool, metadata_path_, metadata),
- base::Bind(&DownloadMetadataManager::ManagerContext::OnMetadataReady,
- weak_factory_.GetWeakPtr(),
- base::Passed(base::WrapUnique(metadata))));
+ base::BindOnce(&ReadMetadataOnWorkerPool, metadata_path_, metadata),
+ base::BindOnce(&DownloadMetadataManager::ManagerContext::OnMetadataReady,
+ weak_factory_.GetWeakPtr(),
+ base::Passed(base::WrapUnique(metadata))));
}
void DownloadMetadataManager::ManagerContext::WriteMetadata() {
write_runner_->PostTask(
FROM_HERE,
- base::Bind(&WriteMetadataOnWorkerPool,
- metadata_path_,
- base::Owned(new DownloadMetadata(*download_metadata_))));
+ base::BindOnce(&WriteMetadataOnWorkerPool, metadata_path_,
+ base::Owned(new DownloadMetadata(*download_metadata_))));
}
void DownloadMetadataManager::ManagerContext::RemoveMetadata() {
@@ -591,7 +591,7 @@ void DownloadMetadataManager::ManagerContext::RemoveMetadata() {
// Remove any metadata.
download_metadata_.reset();
write_runner_->PostTask(
- FROM_HERE, base::Bind(&DeleteMetadataOnWorkerPool, metadata_path_));
+ FROM_HERE, base::BindOnce(&DeleteMetadataOnWorkerPool, metadata_path_));
// Run callbacks (only present in case of a transition to LOAD_COMPLETE).
RunCallbacks();
}

Powered by Google App Engine
This is Rietveld 408576698