memory-infra: add ProcessType and expose data in RequestGlobalDump()

services/resource_coordinator/memory/coordinator/coordinator_impl_unittest.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "services/resource_coordinator/memory/coordinator/coordinator_impl.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback_forward.h"
 #include "base/memory/ref_counted.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 29 matching lines @@
     base::ThreadTaskRunnerHandle::Get()->PostTask(
         FROM_HERE,
         base::Bind(&CoordinatorImpl::RequestGlobalMemoryDump,
                    base::Unretained(coordinator_.get()), args,
                    base::Bind(&CoordinatorImplTest::OnGlobalMemoryDumpResponse,
                               base::Unretained(this), closure)));
   }
 
   void OnGlobalMemoryDumpResponse(base::Closure closure,
                                   uint64_t dump_guid,
-                                  bool success) {
+                                  bool success,
+                                  mojom::GlobalMemoryDumpPtr) {
     dump_response_args_ = {dump_guid, success};
     closure.Run();
   }
 
  protected:
   struct DumpResponseArgs {
     uint64_t dump_guid;
     bool success;
   };
 
@@ skipping 14 matching lines @@
     test_coordinator->RegisterProcessLocalDumpManager(
         std::move(process_manager));
   }
 
   ~MockDumpManager() override { EXPECT_EQ(0, expected_calls_); }
 
   void RequestProcessMemoryDump(
       const base::trace_event::MemoryDumpRequestArgs& args,
       const RequestProcessMemoryDumpCallback& callback) override {
     expected_calls_--;
-    base::trace_event::MemoryDumpCallbackResult result;
-    callback.Run(args.dump_guid, true, result);
+    callback.Run(args.dump_guid, true, mojom::ProcessMemoryDumpPtr());
   }
 
  private:
   mojo::Binding<mojom::ProcessLocalDumpManager> binding_;
   int expected_calls_;
 };
 
 TEST_F(CoordinatorImplTest, NoProcessLocalManagers) {
   base::RunLoop run_loop;
   base::trace_event::MemoryDumpRequestArgs args = {
@@ skipping 37 matching lines @@
   base::ThreadTaskRunnerHandle::Get()->PostTask(
       FROM_HERE, base::Bind([](std::unique_ptr<MockDumpManager> dm) {},
                             base::Passed(&dump_manager_2)));
 
   run_loop.Run();
 
   EXPECT_EQ(3456U, dump_response_args_.dump_guid);
   EXPECT_FALSE(dump_response_args_.success);
 }
 }  // namespace memory_instrumentation