Add renderer-side client for LeakDetectorRemoteController

chrome/renderer/BUILD.gn

Index: chrome/renderer/BUILD.gn
diff --git a/chrome/renderer/BUILD.gn b/chrome/renderer/BUILD.gn
index 3760d3094befadeab682fdadf7bf5de52c842c04..4cf982284e80c35239024ab2c18a13944e9693c1 100644
--- a/chrome/renderer/BUILD.gn
+++ b/chrome/renderer/BUILD.gn
@@ -182,6 +182,22 @@ static_library("renderer") {
       "spellchecker/hunspell_engine.h",
     ]
   }
+
+  if (is_chromeos) {
+    # TODO(sque): Figure out why the below doesn't cause
+    # leak_detector_remote_client.cc to be built.
+    #sources +=
+    #    rebase_path(gypi_values.chrome_renderer_leak_detector_sources, ".",
+    #                "..")
+    sources += [
+      "leak_detector_remote_client.cc",
+      "leak_detector_remote_client.h",
+    ]
+    deps += [
+      "//components/metrics:interfaces",
+      "//components/metrics:leak_detector",
+    ]

[Ilya Sherman, 2016/06/10 21:27:35]

I might have an oversimplified understanding of this, but I think that anything
that's #included in a header file should go in public_deps rather than deps. 
Otherwise, targets that import the header file will fail to compile in a
component build, because they'll be missing the needed dependencies to resolve
the includes.

[Simon Que, 2016/06/16 07:13:26]

Done

+  }
 }
 
 # In GYP this is part of test_support_common.