[tracing] Sanitize process memory dumps for background mode

base/trace_event/memory_dump_provider.h

 // Copyright 2015 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.
 
 #ifndef BASE_TRACE_EVENT_MEMORY_DUMP_PROVIDER_H_
 #define BASE_TRACE_EVENT_MEMORY_DUMP_PROVIDER_H_
 
 #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:
   // Optional arguments for MemoryDumpManager::RegisterDumpProvider().
   struct Options {
     Options()
         : target_pid(kNullProcessId),
           dumps_on_single_thread_task_runner(false) {}
 
     // If the dump provider generates dumps on behalf of another process,
@@ skipping 27 matching lines @@
  protected:
   MemoryDumpProvider() {}
 
   DISALLOW_COPY_AND_ASSIGN(MemoryDumpProvider);
 };
 
 }  // namespace trace_event
 }  // namespace base
 
 #endif  // BASE_TRACE_EVENT_MEMORY_DUMP_PROVIDER_H_