Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2393)

Unified Diff: base/trace_event/memory_dump_manager.cc

Issue 2549103003: tracing: split trace event filter classes out of TraceLog (Closed)
Patch Set: Remove direct coupling between TraceLog and trace_event_filter_test_utils.h Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: base/trace_event/memory_dump_manager.cc
diff --git a/base/trace_event/memory_dump_manager.cc b/base/trace_event/memory_dump_manager.cc
index 28d5d56c59d4346a9cb740d55fbbeca5aef6b603..e9a9f34dc2e0355209f503993150e7f24a3d8a41 100644
--- a/base/trace_event/memory_dump_manager.cc
+++ b/base/trace_event/memory_dump_manager.cc
@@ -19,6 +19,7 @@
#include "base/threading/thread_task_runner_handle.h"
#include "base/trace_event/heap_profiler.h"
#include "base/trace_event/heap_profiler_allocation_context_tracker.h"
+#include "base/trace_event/heap_profiler_event_filter.h"
#include "base/trace_event/heap_profiler_stack_frame_deduplicator.h"
#include "base/trace_event/heap_profiler_type_name_deduplicator.h"
#include "base/trace_event/malloc_dump_provider.h"
@@ -210,7 +211,7 @@ void MemoryDumpManager::Initialize(MemoryDumpManagerDelegate* delegate,
!(TraceLog::GetInstance()->enabled_modes() & TraceLog::FILTERING_MODE)) {
// Create trace config with heap profiling filter.
TraceConfig::EventFilterConfig heap_profiler_filter_config(
- TraceLog::TraceEventFilter::kHeapProfilerPredicate);
+ HeapProfilerEventFilter::kName);
heap_profiler_filter_config.AddIncludedCategory("*");
heap_profiler_filter_config.AddIncludedCategory(
MemoryDumpManager::kTraceCategory);

Powered by Google App Engine
This is Rietveld 408576698