[Tracing] Disable registration of regular dump providers during tests

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 a2fa95396360caad11ccf78d58c19457f9f42098..05d9017761806fc082815d7215e16c5bce5de348 100644
--- a/base/trace_event/memory_dump_manager.cc
+++ b/base/trace_event/memory_dump_manager.cc
@@ -98,8 +98,6 @@ MemoryDumpManager* MemoryDumpManager::GetInstance() {
 
 // static
 void MemoryDumpManager::SetInstanceForTesting(MemoryDumpManager* instance) {
-  if (instance)
-    instance->skip_core_dumpers_auto_registration_for_testing_ = true;
   g_instance_for_testing = instance;
 }
 
@@ -108,7 +106,7 @@ MemoryDumpManager::MemoryDumpManager()
       is_coordinator_(false),
       memory_tracing_enabled_(0),
       tracing_process_id_(kInvalidTracingProcessId),
-      skip_core_dumpers_auto_registration_for_testing_(false) {
+      dumper_registrations_ignored_for_testing_(false) {
   g_next_guid.GetNext();  // Make sure that first guid is not zero.
 }
 
@@ -127,7 +125,6 @@ void MemoryDumpManager::Initialize(MemoryDumpManagerDelegate* delegate,
   }
 
   // Enable the core dump providers.
-  if (!skip_core_dumpers_auto_registration_for_testing_) {
 #if !defined(OS_NACL)
     RegisterDumpProvider(ProcessMemoryTotalsDumpProvider::GetInstance());
 #endif
@@ -144,7 +141,6 @@ void MemoryDumpManager::Initialize(MemoryDumpManagerDelegate* delegate,
 #if defined(OS_WIN)
     RegisterDumpProvider(WinHeapDumpProvider::GetInstance());
 #endif
-  }  // !skip_core_dumpers_auto_registration_for_testing_
 
   // If tracing was enabled before initializing MemoryDumpManager, we missed the
   // OnTraceLogEnabled() event. Synthetize it so we can late-join the party.
@@ -158,6 +154,9 @@ void MemoryDumpManager::Initialize(MemoryDumpManagerDelegate* delegate,
 void MemoryDumpManager::RegisterDumpProvider(
     MemoryDumpProvider* mdp,
     const scoped_refptr<SingleThreadTaskRunner>& task_runner) {
+  if (dumper_registrations_ignored_for_testing_)
+    return;
+
   MemoryDumpProviderInfo mdp_info(mdp, task_runner);
   AutoLock lock(lock_);
   auto iter_new = dump_providers_.insert(mdp_info);