Convert services/resource_coordinator away from base::Bind/base::Callback

base::Bind/base::Callback are deprecated in favor of either
base::BindOnce/base::OnceCallback or base::BindRepeating/
base::RepeatingCallback (depending on whether the callback
is invoked once or multiple time).

Convert all uses of base::Bind/base::Callback in
services/resource_coordinator to the recommended methods/types.

Bug: 1007831
Change-Id: I85e6a627d8b65feffb9a0d95a0c9620e343f9062
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1840635
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Primiano Tucci <primiano@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#703267}

services/resource_coordinator/memory_instrumentation/coordinator_impl.cc

@@ -372,10 +372,10 @@ void CoordinatorImpl::PerformNextQueuedGlobalMemoryDump() {
   }
 
   auto chrome_callback =
-      base::Bind(&CoordinatorImpl::OnChromeMemoryDumpResponse,
+      base::BindRepeating(&CoordinatorImpl::OnChromeMemoryDumpResponse,
                           weak_ptr_factory_.GetWeakPtr());
   auto os_callback =
-      base::Bind(&CoordinatorImpl::OnOSMemoryDumpResponse,
+      base::BindRepeating(&CoordinatorImpl::OnOSMemoryDumpResponse,
                           weak_ptr_factory_.GetWeakPtr(), request->dump_guid);
   QueuedRequestDispatcher::SetUpAndDispatch(request, clients, chrome_callback,
                                             os_callback);

services/resource_coordinator/memory_instrumentation/queued_request_dispatcher.cc

@@ -238,7 +238,7 @@ void QueuedRequestDispatcher::SetUpAndDispatch(
           {client_info.pid, ResponseType::kChromeDump});
       client->RequestChromeMemoryDump(
           request->GetRequestArgs(),
-          base::BindOnce(std::move(chrome_callback), client_info.pid));
+          base::BindOnce(chrome_callback, client_info.pid));
     }
 
 // On most platforms each process can dump data about their own process

services/resource_coordinator/public/cpp/memory_instrumentation/client_process_impl.cc

@@ -76,7 +76,7 @@ void ClientProcessImpl::RequestChromeMemoryDump(
       pending_chrome_callbacks_.emplace(args.dump_guid, std::move(callback));
   DCHECK(it_and_inserted.second) << "Duplicated request id " << args.dump_guid;
   base::trace_event::MemoryDumpManager::GetInstance()->CreateProcessDump(
-      args, base::Bind(&ClientProcessImpl::OnChromeMemoryDumpDone,
+      args, base::BindOnce(&ClientProcessImpl::OnChromeMemoryDumpDone,
                            base::Unretained(this)));
 }