[memory-infra] Remove MemoryDumpManagerDelegate

base/trace_event/memory_dump_manager.cc

Index: base/trace_event/memory_dump_manager.cc
diff --git a/base/trace_event/memory_dump_manager.cc b/base/trace_event/memory_dump_manager.cc
index 5f217eaf9a0aa45f6b9b8e6337eea76dc0b1bf8c..221ead4c13c4212ab3a20f5f16b7a7c0487a3544 100644
--- a/base/trace_event/memory_dump_manager.cc
+++ b/base/trace_event/memory_dump_manager.cc
@@ -184,7 +184,8 @@ void MemoryDumpManager::SetInstanceForTesting(MemoryDumpManager* instance) {
 }
 
 MemoryDumpManager::MemoryDumpManager()
-    : memory_tracing_enabled_(0),
+    : is_coordinator_(false),
+      memory_tracing_enabled_(0),
       tracing_process_id_(kInvalidTracingProcessId),
       dumper_registrations_ignored_for_testing_(false),
       heap_profiling_enabled_(false) {
@@ -241,12 +242,14 @@ void MemoryDumpManager::EnableHeapProfilingIfNeeded() {
 }
 
 void MemoryDumpManager::Initialize(
-    std::unique_ptr<MemoryDumpManagerDelegate> delegate) {
+    RequestGlobalDumpCallback request_dump_callback,
+    bool is_coordinator) {
   {
     AutoLock lock(lock_);
-    DCHECK(delegate);
-    DCHECK(!delegate_);
-    delegate_ = std::move(delegate);
+    DCHECK(!request_dump_callback.is_null());
+    DCHECK(request_dump_callback_.is_null());
+    request_dump_callback_ = request_dump_callback;
+    is_coordinator_ = is_coordinator;
     EnableHeapProfilingIfNeeded();
   }
 
@@ -465,10 +468,10 @@ void MemoryDumpManager::RequestGlobalDump(
       MemoryDumpLevelOfDetailToString(level_of_detail));
   GlobalMemoryDumpCallback wrapped_callback = Bind(&OnGlobalDumpDone, callback);
 
-  // The delegate will coordinate the IPC broadcast and at some point invoke
+  // The embedder will coordinate the IPC broadcast and at some point invoke
   // CreateProcessDump() to get a dump for the current process.
   MemoryDumpRequestArgs args = {guid, dump_type, level_of_detail};
-  delegate_->RequestGlobalMemoryDump(args, wrapped_callback);
+  request_dump_callback_.Run(args, wrapped_callback);
 }
 
 void MemoryDumpManager::GetDumpProvidersForPolling(
@@ -865,7 +868,8 @@ void MemoryDumpManager::OnTraceLogEnabled() {
 
   AutoLock lock(lock_);
 
-  DCHECK(delegate_);  // At this point we must have a delegate.
+  // At this point we must have the ability to request global dumps.
+  DCHECK(!request_dump_callback_.is_null());
   session_state_ = session_state;
 
   DCHECK(!dump_thread_);
@@ -905,7 +909,7 @@ void MemoryDumpManager::OnTraceLogEnabled() {
 
       // When peak detection is enabled, trigger a dump straight away as it
       // gives a good reference point for analyzing the trace.
-      if (delegate_->IsCoordinator()) {
+      if (is_coordinator_) {
         dump_thread_->task_runner()->PostTask(
             FROM_HERE, BindRepeating(&OnPeakDetected, trigger.level_of_detail));
       }
@@ -913,7 +917,7 @@ void MemoryDumpManager::OnTraceLogEnabled() {
   }
 
   // Only coordinator process triggers periodic global memory dumps.
-  if (delegate_->IsCoordinator() && !periodic_config.triggers.empty()) {
+  if (is_coordinator_ && !periodic_config.triggers.empty()) {
     MemoryDumpScheduler::GetInstance()->Start(periodic_config,
                                               dump_thread_->task_runner());
   }