Remove unnecessary base::Passed() with base::BindOnce()

The documentation on base::Callback<...> (src/docs/callback.md)
recommends against using base::Passed(...) with base::BindOnce.

> Avoid using `base::Passed()` with `base::BindOnce()`, as `std::move()`
> does the same thing and is more familiar.

Also convert uses of base::Bind() that can be trivially converted to
use base::BindOnce() so that the base::Passed() can be removed from
the binding.

This CL was uploaded by git cl split.

Bug: 812523
Change-Id: I575592bb0d3d316f47c4e654f800ee51f66f2538
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1611883
Commit-Queue: Satoru Takabayashi <satorux@chromium.org>
Reviewed-by: Satoru Takabayashi <satorux@chromium.org>
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#660735}

chrome/browser/chromeos/app_mode/kiosk_app_data_base.cc

@@ -102,10 +102,9 @@ void KioskAppDataBase::SaveIcon(const SkBitmap& icon,
 
   const base::FilePath icon_path =
       cache_dir.AppendASCII(app_id_).AddExtension(kIconFileExtension);
-  base::PostTaskWithTraits(
-      FROM_HERE, {base::MayBlock()},
-      base::BindOnce(&SaveIconToLocalOnBlockingPool, icon_path,
-                     base::Passed(std::move(image_data))));
+  base::PostTaskWithTraits(FROM_HERE, {base::MayBlock()},
+                           base::BindOnce(&SaveIconToLocalOnBlockingPool,
+                                          icon_path, std::move(image_data)));
 
   icon_path_ = icon_path;
 }

chrome/browser/chromeos/customization/customization_document.cc

@@ -869,17 +869,16 @@ void ServicesCustomizationDocument::CheckAndApplyWallpaper() {
 
   std::unique_ptr<bool> exists(new bool(false));
 
-  base::Closure check_file_exists =
-      base::Bind(&CheckWallpaperCacheExists,
-                 GetCustomizedWallpaperDownloadedFileName(),
-                 base::Unretained(exists.get()));
-  base::Closure on_checked_closure = base::Bind(
+  base::OnceClosure check_file_exists = base::BindOnce(
+      &CheckWallpaperCacheExists, GetCustomizedWallpaperDownloadedFileName(),
+      base::Unretained(exists.get()));
+  base::OnceClosure on_checked_closure = base::BindOnce(
       &ServicesCustomizationDocument::OnCheckedWallpaperCacheExists,
       weak_ptr_factory_.GetWeakPtr(), base::Passed(std::move(exists)),
-      base::Passed(std::move(applying)));
+      std::move(applying));
   base::PostTaskWithTraitsAndReply(
       FROM_HERE, {base::MayBlock(), base::TaskPriority::BEST_EFFORT},
-      check_file_exists, on_checked_closure);
+      std::move(check_file_exists), std::move(on_checked_closure));
 }
 
 void ServicesCustomizationDocument::OnCheckedWallpaperCacheExists(

chrome/browser/chromeos/customization/customization_wallpaper_downloader.cc

@@ -135,7 +135,7 @@ void CustomizationWallpaperDownloader::Start() {
                      base::Unretained(success.get()));
   base::OnceClosure on_created_closure = base::BindOnce(
       &CustomizationWallpaperDownloader::OnWallpaperDirectoryCreated,
-      weak_factory_.GetWeakPtr(), base::Passed(std::move(success)));
+      weak_factory_.GetWeakPtr(), std::move(success));
   base::PostTaskWithTraitsAndReply(
       FROM_HERE, {base::MayBlock(), base::TaskPriority::BEST_EFFORT},
       std::move(mkdir_closure), std::move(on_created_closure));
@@ -169,9 +169,9 @@ void CustomizationWallpaperDownloader::OnSimpleLoaderComplete(
   base::OnceClosure rename_closure = base::BindOnce(
       &RenameTemporaryFile, response_path, wallpaper_downloaded_file_,
       base::Unretained(success.get()));
-  base::OnceClosure on_rename_closure = base::BindOnce(
-      &CustomizationWallpaperDownloader::OnTemporaryFileRenamed,
-      weak_factory_.GetWeakPtr(), base::Passed(std::move(success)));
+  base::OnceClosure on_rename_closure =
+      base::BindOnce(&CustomizationWallpaperDownloader::OnTemporaryFileRenamed,
+                     weak_factory_.GetWeakPtr(), std::move(success));
   base::PostTaskWithTraitsAndReply(
       FROM_HERE, {base::MayBlock(), base::TaskPriority::BEST_EFFORT},
       std::move(rename_closure), std::move(on_rename_closure));

chrome/browser/chromeos/ownership/owner_settings_service_chromeos.cc

@@ -124,10 +124,9 @@ void ContinueLoadPrivateKeyOnIOThread(
            base::TaskShutdownBehavior::SKIP_ON_SHUTDOWN});
   task_runner->PostTask(
       FROM_HERE,
-      base::BindOnce(
-          &LoadPrivateKeyByPublicKeyOnWorkerThread, owner_key_util,
-          base::Passed(crypto::GetPublicSlotForChromeOSUser(username_hash)),
-          base::Passed(std::move(private_slot)), callback));
+      base::BindOnce(&LoadPrivateKeyByPublicKeyOnWorkerThread, owner_key_util,
+                     crypto::GetPublicSlotForChromeOSUser(username_hash),
+                     std::move(private_slot), callback));
 }
 
 void LoadPrivateKeyOnIOThread(const scoped_refptr<OwnerKeyUtil>& owner_key_util,