[memory-infra] Remove MemoryDumpManagerDelegate

services/resource_coordinator/BUILD.gn

 # 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.
 
 # There should be only one memory instrumentaiton coordinator. It is currently
 # in the browser process. So, only //content/browser should link to this target.
 # Others modules should only need the public targets.
 source_set("lib") {

[Primiano Tucci (use gerrit), 2017/04/21 17:11:51]

I think this needs to become component("lib") otherwise you'll be bitten by all
sort of ODR violations. this lib is linked in by different linker unit in
component builds.

   sources = [
     "memory/coordinator/coordinator_impl.cc",
     "memory/coordinator/coordinator_impl.h",
+    "resource_coordinator_export.h",
   ]
 
   public_deps = [
     "//base",
     "//mojo/public/cpp/bindings",
-    "//services/resource_coordinator/public/cpp",
+    "//services/resource_coordinator/public/cpp:resource_coordinator_cpp",
     "//services/resource_coordinator/public/interfaces",
   ]
 }
 
 source_set("tests") {
   testonly = true
 
   sources = [
     "memory/coordinator/coordinator_impl_unittest.cc",
-    "public/cpp/memory/memory_dump_manager_delegate_impl_unittest.cc",
+    "public/cpp/memory/process_local_dump_manager_impl_unittest.cc",
   ]
 
   deps = [
     ":lib",
     "//base",
     "//mojo/public/cpp/bindings",
-    "//services/resource_coordinator/public/cpp",
+    "//services/resource_coordinator/public/cpp:resource_coordinator_cpp",
     "//services/resource_coordinator/public/interfaces",
     "//testing/gtest",
   ]
 }