[memory-infra] Log reasons for memory dump failure

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 4e2cc606e9b8e04396ea06f6da46a75595fb76ae..3cafe8f6e5839a96c701b97cc5d6abc2c60f3dc0 100644
--- a/base/trace_event/memory_dump_manager.cc
+++ b/base/trace_event/memory_dump_manager.cc
@@ -98,6 +98,10 @@ const char* const MemoryDumpManager::kTraceCategory =
     TRACE_DISABLED_BY_DEFAULT("memory-infra");
 
 // static
+const char* const MemoryDumpManager::kMemoryInfraDumpLogName =
+    "memory-infra dump";
+
+// static
 const int MemoryDumpManager::kMaxConsecutiveFailuresCount = 3;
 
 // static
@@ -337,6 +341,8 @@ void MemoryDumpManager::RequestGlobalDump(
     const MemoryDumpCallback& callback) {
   // Bail out immediately if tracing is not enabled at all.
   if (!UNLIKELY(subtle::NoBarrier_Load(&memory_tracing_enabled_))) {
+    VLOG(1) << kMemoryInfraDumpLogName << " failed because " << kTraceCategory
+            << " tracing category is not enabled";
     if (!callback.is_null())
       callback.Run(0u /* guid */, false /* success */);
     return;
@@ -422,6 +428,14 @@ void MemoryDumpManager::SetupNextMemoryDump(
   // Anyway either tracing is stopped or this was the last hop, create a trace
   // event, add it to the trace and finalize process dump invoking the callback.
   if (!pmd_async_state->dump_thread_task_runner.get()) {
+    if (pmd_async_state->pending_dump_providers.empty()) {
+      VLOG(1) << kMemoryInfraDumpLogName << " failed because dump thread was"
+              << " destroyed before finalizing the dump";
+    } else {
+      VLOG(1) << kMemoryInfraDumpLogName << " failed because dump thread was"
+              << " destroyed before dumping "
+              << pmd_async_state->pending_dump_providers.back().get()->name;
+    }
     pmd_async_state->dump_successful = false;
     pmd_async_state->pending_dump_providers.clear();
   }
@@ -596,8 +610,11 @@ void MemoryDumpManager::FinalizeDumpAndAddToTrace(
 
   bool tracing_still_enabled;
   TRACE_EVENT_CATEGORY_GROUP_ENABLED(kTraceCategory, &tracing_still_enabled);
-  if (!tracing_still_enabled)
+  if (!tracing_still_enabled) {
     pmd_async_state->dump_successful = false;
+    VLOG(1) << kMemoryInfraDumpLogName << " failed because tracing was"
+            << " disabled before the dump was completed";
+  }
 
   if (!pmd_async_state->callback.is_null()) {
     pmd_async_state->callback.Run(dump_guid, pmd_async_state->dump_successful);