memory-infra: Fill the memory dump callback result (1/2)

base/trace_event/memory_allocator_dump.cc

Index: base/trace_event/memory_allocator_dump.cc
diff --git a/base/trace_event/memory_allocator_dump.cc b/base/trace_event/memory_allocator_dump.cc
index 7583763889ef8b975840d0bcadd12feb359be9a6..528902a08218caf388252f9e139db1f66d29e463 100644
--- a/base/trace_event/memory_allocator_dump.cc
+++ b/base/trace_event/memory_allocator_dump.cc
@@ -29,7 +29,8 @@ MemoryAllocatorDump::MemoryAllocatorDump(const std::string& absolute_name,
       process_memory_dump_(process_memory_dump),
       attributes_(new TracedValue),
       guid_(guid),
-      flags_(Flags::DEFAULT) {
+      flags_(Flags::DEFAULT),
+      size_(0) {
   // The |absolute_name| cannot be empty.
   DCHECK(!absolute_name.empty());
 
@@ -56,9 +57,16 @@ MemoryAllocatorDump::MemoryAllocatorDump(const std::string& absolute_name,
 MemoryAllocatorDump::~MemoryAllocatorDump() {
 }
 
+uint64_t MemoryAllocatorDump::GetSize() {
+  return size_;
+}
+
 void MemoryAllocatorDump::AddScalar(const char* name,
                                     const char* units,
                                     uint64_t value) {
+  if (strcmp(kNameSize, name) == 0) {

[Primiano Tucci (use gerrit), 2017/03/23 15:50:49]

nit: for consistency with the rest of the code in this file, can you drop the
braces for single line if (IIRC our code style allows both but says to stay
locally consistent)

[hjd, 2017/03/23 16:16:39]

Done.

+    size_ = value;
+  }
   SStringPrintf(&string_conversion_buffer_, "%" PRIx64, value);
   attributes_->BeginDictionary(name);
   attributes_->SetString("type", kTypeScalar);