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

Unified Diff: base/trace_event/heap_profiler_serialization_state.h

Issue 2650863003: [tracing] Switch to new heap dump format. (Closed)
Patch Set: Rebase Created 3 years, 6 months 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/heap_profiler_serialization_state.h
diff --git a/base/trace_event/heap_profiler_serialization_state.h b/base/trace_event/heap_profiler_serialization_state.h
index 3d388b209c580719127eecd8521fc8982f895b59..4c0ffd7a64aebf2efc34a9b863378d1e26d93d67 100644
--- a/base/trace_event/heap_profiler_serialization_state.h
+++ b/base/trace_event/heap_profiler_serialization_state.h
@@ -29,17 +29,18 @@ class BASE_EXPORT HeapProfilerSerializationState
return stack_frame_deduplicator_.get();
}
- void SetStackFrameDeduplicator(
- std::unique_ptr<StackFrameDeduplicator> stack_frame_deduplicator);
-
// Returns the type name deduplicator that should be used by memory dump
// providers when doing a heap dump.
TypeNameDeduplicator* type_name_deduplicator() const {
return type_name_deduplicator_.get();
}
- void SetTypeNameDeduplicator(
- std::unique_ptr<TypeNameDeduplicator> type_name_deduplicator);
+ // Returns generic string deduplicator used by other deduplicators.
+ StringDeduplicator* string_deduplicator() const {
+ return string_deduplicator_.get();
+ }
+
+ void CreateDeduplicators();
void SetAllowedDumpModes(
std::set<MemoryDumpLevelOfDetail> allowed_dump_modes);
@@ -66,6 +67,10 @@ class BASE_EXPORT HeapProfilerSerializationState
// trace is finalized.
std::unique_ptr<TypeNameDeduplicator> type_name_deduplicator_;
+ // Generic string deduplicator, used by other deduplicators; must be defined
+ // after ones that depend on it.
+ std::unique_ptr<StringDeduplicator> string_deduplicator_;
+
uint32_t heap_profiler_breakdown_threshold_bytes_;
};
« no previous file with comments | « base/trace_event/heap_profiler_heap_dump_writer_unittest.cc ('k') | base/trace_event/heap_profiler_serialization_state.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698