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

base/trace_event/memory_dump_manager.h

Index: base/trace_event/memory_dump_manager.h
diff --git a/base/trace_event/memory_dump_manager.h b/base/trace_event/memory_dump_manager.h
index 79e8dfbf4b8d01a480a5dbe394f8ea980b5420a8..bcd4b47a4c463962e958758a9d1f21d7cd9227d0 100644
--- a/base/trace_event/memory_dump_manager.h
+++ b/base/trace_event/memory_dump_manager.h
@@ -297,6 +297,8 @@ class BASE_EXPORT MemoryDumpManager : public TraceLog::EnabledStateObserver {
   ~MemoryDumpManager() override;
 
   static void SetInstanceForTesting(MemoryDumpManager* instance);
+  static uint32_t SumSizesInKbForPrefix(const std::string&,

[Primiano Tucci (use gerrit), 2017/03/22 17:19:52]

I'd call this "GetDumpsSumKb" (to make clear that this is about summing the
"dump", and use an explicit |prefix| in the string var name (so you don't end up
repeating "prefix" in the .cc file)

Also I'd add a similar comment with todo saying this is transitional with link
to the bug

[hjd, 2017/03/22 19:24:20]

Done.

+                                        const ProcessMemoryDump*);
   static void FinalizeDumpAndAddToTrace(
       std::unique_ptr<ProcessMemoryDumpAsyncState> pmd_async_state);