Convert base::Closure in chrome/browser/importer

This is a code health CL, no behavior change is expected.

Bug: 1007635
Change-Id: I416970f3ccd80e32c62fc7c5f7f246337d4b7fac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2280142Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Auto-Submit: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#785708}

chrome/browser/importer/importer_list.cc

@@ -194,7 +194,7 @@ ImporterList::~ImporterList() {
 void ImporterList::DetectSourceProfiles(
     const std::string& locale,
     bool include_interactive_profiles,
-    const base::Closure& profiles_loaded_callback) {
+    base::OnceClosure profiles_loaded_callback) {
   DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
   base::ThreadPool::PostTaskAndReplyWithResult(
       FROM_HERE,
@@ -203,7 +203,8 @@ void ImporterList::DetectSourceProfiles(
       base::BindOnce(&DetectSourceProfilesWorker, locale,
                      include_interactive_profiles),
       base::BindOnce(&ImporterList::SourceProfilesLoaded,
-                     weak_ptr_factory_.GetWeakPtr(), profiles_loaded_callback));
+                     weak_ptr_factory_.GetWeakPtr(),
+                     std::move(profiles_loaded_callback)));
 }
 
 const importer::SourceProfile& ImporterList::GetSourceProfileAt(
@@ -213,10 +214,10 @@ const importer::SourceProfile& ImporterList::GetSourceProfileAt(
 }
 
 void ImporterList::SourceProfilesLoaded(
-    const base::Closure& profiles_loaded_callback,
+    base::OnceClosure profiles_loaded_callback,
     const std::vector<importer::SourceProfile>& profiles) {
   DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
 
   source_profiles_.assign(profiles.begin(), profiles.end());
-  profiles_loaded_callback.Run();
+  std::move(profiles_loaded_callback).Run();
 }

chrome/browser/importer/importer_list.h

@@ -35,7 +35,7 @@ class ImporterList {
   // alive, run the callback when the source profile detection finishes.
   void DetectSourceProfiles(const std::string& locale,
                             bool include_interactive_profiles,
-                            const base::Closure& profiles_loaded_callback);
+                            base::OnceClosure profiles_loaded_callback);
 
   // Returns the number of different source profiles you can import from.
   size_t count() const { return source_profiles_.size(); }
@@ -49,7 +49,7 @@ class ImporterList {
   // Called when the source profiles are loaded. Copies the loaded profiles
   // in |profiles| and calls |profiles_loaded_callback|.
   void SourceProfilesLoaded(
-      const base::Closure& profiles_loaded_callback,
+      base::OnceClosure profiles_loaded_callback,
       const std::vector<importer::SourceProfile>& profiles);
 
   // The list of profiles with the default one first.