memory-infra: Assemble dump results in service

services/resource_coordinator/memory/coordinator/coordinator_impl.h

Index: services/resource_coordinator/memory/coordinator/coordinator_impl.h
diff --git a/services/resource_coordinator/memory/coordinator/coordinator_impl.h b/services/resource_coordinator/memory/coordinator/coordinator_impl.h
index 096847b0f9aca14eb9cda21ab6d67f99d1cca09f..c032f581d4c273baee7ef224d11bbf5e6ea9b294 100644
--- a/services/resource_coordinator/memory/coordinator/coordinator_impl.h
+++ b/services/resource_coordinator/memory/coordinator/coordinator_impl.h
@@ -10,6 +10,7 @@
 #include <set>
 
 #include "base/memory/ref_counted.h"
+#include "base/process/process_handle.h"

[Primiano Tucci (use gerrit), 2017/05/10 17:42:43]

why do we need this here? can it be moved to the .cc file?

 #include "base/threading/thread_checker.h"
 #include "base/trace_event/memory_dump_request_args.h"
 #include "mojo/public/cpp/bindings/binding.h"
@@ -17,6 +18,8 @@
 #include "services/resource_coordinator/public/cpp/memory/coordinator.h"
 #include "services/resource_coordinator/public/interfaces/memory/memory_instrumentation.mojom.h"
 
+using MemoryDumpCallbackResult = base::trace_event::MemoryDumpCallbackResult;

[Primiano Tucci (use gerrit), 2017/05/10 17:42:43]

no "using" in headers 

https://google.github.io/styleguide/cppguide.html#Namespaces

but okay if you move this to the .cc file

+
 namespace memory_instrumentation {
 
 class CoordinatorImpl : public Coordinator, public mojom::Coordinator {