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

Unified Diff: base/trace_event/memory_dump_session_state.h

Issue 2650863003: [tracing] Switch to new heap dump format. (Closed)
Patch Set: Fix StringDeduplicator::Insert Created 3 years, 9 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/memory_dump_session_state.h
diff --git a/base/trace_event/memory_dump_session_state.h b/base/trace_event/memory_dump_session_state.h
index 46092cb4832d7d8d69f51cfadd143760c26b0b15..1ddac72daf95d0badcf6ca4cba1b3535aeec6d90 100644
--- a/base/trace_event/memory_dump_session_state.h
+++ b/base/trace_event/memory_dump_session_state.h
@@ -29,17 +29,18 @@ class BASE_EXPORT MemoryDumpSessionState
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,9 @@ class BASE_EXPORT MemoryDumpSessionState
// trace is finalized.
std::unique_ptr<TypeNameDeduplicator> type_name_deduplicator_;
+ // Generic string deduplicator, used by other deduplicators.
+ std::unique_ptr<StringDeduplicator> string_deduplicator_;
+
std::set<MemoryDumpLevelOfDetail> allowed_dump_modes_;
uint32_t heap_profiler_breakdown_threshold_bytes_;

Powered by Google App Engine
This is Rietveld 408576698