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

Unified Diff: base/trace_event/memory_allocator_dump_guid.cc

Issue 2923123004: [memory-infra] Add API to ProcessMemoryDump to create ownership edges for base::SharedMemory (Closed)
Patch Set: nits. 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/memory_allocator_dump_guid.cc
diff --git a/base/trace_event/memory_allocator_dump_guid.cc b/base/trace_event/memory_allocator_dump_guid.cc
index bf4389a4c759f5ae29bc6fda2cb8c4b4cc536871..ddec816a80499a0320b093e791fb009ab4bae41d 100644
--- a/base/trace_event/memory_allocator_dump_guid.cc
+++ b/base/trace_event/memory_allocator_dump_guid.cc
@@ -20,6 +20,13 @@ uint64_t HashString(const std::string& str) {
}
} // namespace
+// static
+bool MemoryAllocatorDumpGuid::UseSharedMemoryBasedGUIDs() {
+ // TODO(hajimehoshi): This should just become the default behavior once the
+ // Mojo GUID (crbug.com/604726) is fixed.
+ return false;
+}
+
MemoryAllocatorDumpGuid::MemoryAllocatorDumpGuid(uint64_t guid) : guid_(guid) {}
MemoryAllocatorDumpGuid::MemoryAllocatorDumpGuid()

Powered by Google App Engine
This is Rietveld 408576698