Adds chrome.memoryBenchmarking.heapProfilerDump for the browser process.

content/browser/renderer_host/memory_benchmark_message_filter.cc

Index: content/browser/renderer_host/memory_benchmark_message_filter.cc
diff --git a/content/browser/renderer_host/memory_benchmark_message_filter.cc b/content/browser/renderer_host/memory_benchmark_message_filter.cc
new file mode 100644
index 0000000000000000000000000000000000000000..7d369cc1e0ad092331cb41c00ded88869253b5c2
--- /dev/null
+++ b/content/browser/renderer_host/memory_benchmark_message_filter.cc
@@ -0,0 +1,50 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "content/browser/renderer_host/memory_benchmark_message_filter.h"
+
+#include "content/common/memory_benchmark_messages.h"
+
+#if defined(USE_TCMALLOC)
+#include "third_party/tcmalloc/chromium/src/gperftools/heap-profiler.h"
+#endif
+
+namespace content {
+
+MemoryBenchmarkMessageFilter::MemoryBenchmarkMessageFilter() {
+}
+
+bool MemoryBenchmarkMessageFilter::OnMessageReceived(
+    const IPC::Message& message,
+    bool* message_was_ok) {
+#if defined(USE_TCMALLOC)
+  bool handled = true;
+  IPC_BEGIN_MESSAGE_MAP_EX(MemoryBenchmarkMessageFilter,
+                           message,
+                           *message_was_ok)
+    IPC_MESSAGE_HANDLER(MemoryBenchmarkHostMsg_HeapProfilerDump,
+                        OnHeapProfilerDump)
+    IPC_MESSAGE_UNHANDLED(handled = false)
+  IPC_END_MESSAGE_MAP()
+  return handled;
+#else
+  return false;
+#endif  // defined(USE_TCMALLOC)
+}
+
+MemoryBenchmarkMessageFilter::~MemoryBenchmarkMessageFilter() {
+}
+
+void MemoryBenchmarkMessageFilter::OnHeapProfilerDump(
+    std::string* dump_file_name) {
+#if defined(USE_TCMALLOC)

[piman, 2013/05/10 18:13:52]

I think we need to make sure this can only happen when
--enable-memory-benchmarking is set, e.g. by testing here, or by testing before
adding the filter.

[palmer, 2013/05/10 18:42:17]

Agreed.

[bulach, 2013/05/10 18:46:03]

good point! checking when adding the filter, so that this class assumes it has
what it needs.

+  char dumped_filename_buffer[1000];
+  ::HeapProfilerDumpWithFileName("browser",
+                                 dumped_filename_buffer,
+                                 sizeof(dumped_filename_buffer));
+  dump_file_name->assign(dumped_filename_buffer);
+#endif  // defined(USE_TCMALLOC)
+}
+
+}  // namespace content