Heap Profiling for Oilpan

content/child/web_process_memory_dump_impl.cc

Index: content/child/web_process_memory_dump_impl.cc
diff --git a/content/child/web_process_memory_dump_impl.cc b/content/child/web_process_memory_dump_impl.cc
index a95c9d7a583ef74d00aa98e209049e6430b0f936..01af60d8c3facb875cb1517054ae12118d3bdd34 100644
--- a/content/child/web_process_memory_dump_impl.cc
+++ b/content/child/web_process_memory_dump_impl.cc
@@ -7,6 +7,7 @@
 #include <stddef.h>
 
 #include "base/memory/discardable_memory.h"
+#include "base/strings/stringprintf.h"
 #include "base/trace_event/heap_profiler_heap_dump_writer.h"
 #include "base/trace_event/process_memory_dump.h"
 #include "base/trace_event/trace_event_argument.h"
@@ -177,7 +178,8 @@ void WebProcessMemoryDumpImpl::dumpHeapUsage(
       session_state->stack_frame_deduplicator(),
       session_state->type_name_deduplicator());
   process_memory_dump_->AddHeapDump(allocator_name, heap_dump);
-  overhead.DumpInto("tracing/heap_profiler", process_memory_dump_);
+  std::string base_name = base::StringPrintf("tracing/%s", allocator_name);

[Primiano Tucci (use gerrit), 2016/02/17 10:13:46]

I'd probably keep heap_profiler prefix, to make it clear that it is overhead for
heap_profiler, i.e. "tracing/heap_profiler_%s"

[hajimehoshi, 2016/02/17 10:34:04]

Done.

+  overhead.DumpInto(base_name.c_str(), process_memory_dump_);
 }
 
 }  // namespace content