Removing unnecessary calls to AdaptCallbackForRepeating.

TESTED=build

Bug: 730593
Change-Id: Icecb1ce729c82ec02538ed04d642841be12d0799
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463025Reviewed-by: Hidehiko Abe <hidehiko@chromium.org>
Reviewed-by: Sergey Poromov <poromov@chromium.org>
Commit-Queue: Tomasz Tylenda <ttylenda@chromium.org>
Cr-Commit-Position: refs/heads/master@{#815544}

chrome/browser/chromeos/arc/auth/arc_robot_auth_code_fetcher.cc

@@ -73,8 +73,7 @@ void ArcRobotAuthCodeFetcher::Fetch(FetchCallback callback) {
                     ->GetSharedURLLoaderFactory(),
           base::BindOnce(
               &ArcRobotAuthCodeFetcher::OnFetchRobotAuthCodeCompleted,
-              weak_ptr_factory_.GetWeakPtr(),
-              base::AdaptCallbackForRepeating(std::move(callback))));
+              weak_ptr_factory_.GetWeakPtr(), std::move(callback)));
 
   enterprise_management::DeviceServiceApiAccessRequest* request =
       config->request()->mutable_service_api_access_request();

chrome/browser/chromeos/arc/policy/arc_policy_bridge.cc

@@ -463,14 +463,10 @@ void ArcPolicyBridge::GetPolicies(GetPoliciesCallback callback) {
 void ArcPolicyBridge::ReportCompliance(const std::string& request,
                                        ReportComplianceCallback callback) {
   VLOG(1) << "ArcPolicyBridge::ReportCompliance";
-  // TODO(crbug.com/730593): Remove AdaptCallbackForRepeating() by updating
-  // the callee interface.
-  auto repeating_callback =
-      base::AdaptCallbackForRepeating(std::move(callback));
   data_decoder::DataDecoder::ParseJsonIsolated(
       request,
       base::BindOnce(&ArcPolicyBridge::OnReportComplianceParse,
-                     weak_ptr_factory_.GetWeakPtr(), repeating_callback));
+                     weak_ptr_factory_.GetWeakPtr(), std::move(callback)));
 }
 
 void ArcPolicyBridge::ReportCloudDpsRequested(

chrome/browser/chromeos/policy/arc_app_install_event_log_uploader.cc

@@ -75,9 +75,9 @@ void ArcAppInstallEventLogUploader::OnSerialized(
   // base::Unretained() is safe here as the destructor cancels any pending
   // upload, after which the |client_| is guaranteed to not call the callback.
   client_->UploadAppInstallReport(
-      std::move(value_report), base::AdaptCallbackForRepeating(base::BindOnce(
-                                   &ArcAppInstallEventLogUploader::OnUploadDone,
-                                   base::Unretained(this))));
+      std::move(value_report),
+      base::BindOnce(&ArcAppInstallEventLogUploader::OnUploadDone,
+                     base::Unretained(this)));
 }
 
 }  // namespace policy

components/arc/session/arc_data_remover.cc

@@ -57,9 +57,8 @@ void ArcDataRemover::Run(RunCallback callback) {
           .account_id();
   upstart_client->StartJob(
       kArcRemoveDataUpstartJob, {"CHROMEOS_USER=" + account_id},
-      base::AdaptCallbackForRepeating(
-          base::BindOnce(&ArcDataRemover::OnDataRemoved,
-                         weak_factory_.GetWeakPtr(), std::move(callback))));
+      base::BindOnce(&ArcDataRemover::OnDataRemoved, weak_factory_.GetWeakPtr(),
+                     std::move(callback)));
 }
 
 void ArcDataRemover::OnDataRemoved(RunCallback callback, bool success) {