[tracing] Sanitize process memory dumps for background mode

base/trace_event/memory_dump_provider.h

Index: base/trace_event/memory_dump_provider.h
diff --git a/base/trace_event/memory_dump_provider.h b/base/trace_event/memory_dump_provider.h
index 79ab7934866a5bce8b7a26cf95fb962325b0920d..405e1bb8ce94d0361e1b4d91479e26d018247bed 100644
--- a/base/trace_event/memory_dump_provider.h
+++ b/base/trace_event/memory_dump_provider.h
@@ -8,19 +8,11 @@
 #include "base/base_export.h"
 #include "base/macros.h"
 #include "base/process/process_handle.h"
-#include "base/trace_event/memory_dump_request_args.h"
+#include "base/trace_event/process_memory_dump.h"

[Primiano Tucci (use gerrit), 2016/06/02 20:24:04]

hmm I wonder if this can end up pulling too many headers in blink and cause some
weird side effect due to some name collision. Maybe safer if you move MDArgs to
memory_dump_request_args which is a cleaner header.

[ssid, 2016/06/03 01:59:46]

Done.

 
 namespace base {
 namespace trace_event {
 
-class ProcessMemoryDump;
-
-// Args passed to OnMemoryDump(). This is to avoid rewriting all the subclasses
-// in the codebase when extending the MemoryDumpProvider API.
-struct MemoryDumpArgs {
-  MemoryDumpLevelOfDetail level_of_detail;
-};
-
 // The contract interface that memory dump providers must implement.
 class BASE_EXPORT MemoryDumpProvider {
  public: