[tracing] Sanitize process memory dumps for background mode

third_party/WebKit/Source/platform/heap/BlinkGCMemoryDumpProvider.cpp

 // 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.
 
 #include "platform/heap/BlinkGCMemoryDumpProvider.h"
 
 #include "base/trace_event/heap_profiler_allocation_context_tracker.h"
 #include "base/trace_event/heap_profiler_allocation_register.h"
 #include "base/trace_event/memory_allocator_dump.h"
 #include "base/trace_event/process_memory_dump.h"
@@ skipping 97 matching lines @@
     // TODO(bashi): Change type name of |absoluteName|.
     return m_currentProcessMemoryDump->CreateAllocatorDump(absoluteName.utf8().data());
 }
 
 void BlinkGCMemoryDumpProvider::clearProcessDumpForCurrentGC()
 {
     m_currentProcessMemoryDump->Clear();
 }
 
 BlinkGCMemoryDumpProvider::BlinkGCMemoryDumpProvider()
-    : m_currentProcessMemoryDump(new base::trace_event::ProcessMemoryDump(nullptr))
+    : m_currentProcessMemoryDump(new base::trace_event::ProcessMemoryDump(nullptr, { base::trace_event::MemoryDumpLevelOfDetail::DETAILED }))

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

OH I wonder if for consistency you should reset
m_currentProcessMemoryDump.levelofdetail in the ::OnMemoryDump.

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

This dump is added to trace when detailed dump and this dump is created even
when tracing is not enabled, at every gc.
So, this shouldn't be reset and it should always be detailed.

     , m_isHeapProfilingEnabled(false)
 {
 }
 
 void BlinkGCMemoryDumpProvider::insert(Address address, size_t size, const char* typeName)
 {
     base::trace_event::AllocationContext context = base::trace_event::AllocationContextTracker::GetInstanceForCurrentThread()->GetContextSnapshot();
     context.type_name = typeName;
     MutexLocker locker(m_allocationRegisterMutex);
     if (m_allocationRegister)
         m_allocationRegister->Insert(address, size, context);
 }
 
 void BlinkGCMemoryDumpProvider::remove(Address address)
 {
     MutexLocker locker(m_allocationRegisterMutex);
     if (m_allocationRegister)
         m_allocationRegister->Remove(address);
 }
 
 } // namespace blink