Add renderer-side client for LeakDetectorRemoteController

chrome/renderer/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/features.gni")
 import("//tools/grit/grit_rule.gni")
 
 gypi_values = exec_script("//build/gypi_to_gn.py",
                           [ rebase_path("../chrome_renderer.gypi") ],
                           "scope",
@@ skipping 164 matching lines @@
 
   if (is_win) {
     deps += [ "//third_party/wtl" ]
   }
   if (is_android) {
     sources -= [
       "spellchecker/hunspell_engine.cc",
       "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.
 source_set("test_support") {
   testonly = true
   visibility = [ "//chrome/test:test_support" ]
 
   sources = [
     "chrome_mock_render_thread.cc",
     "chrome_mock_render_thread.h",
@@ skipping 21 matching lines @@
     ]
   }
 
   if (is_android) {
     sources -= [
       "safe_browsing/mock_feature_extractor_clock.cc",
       "safe_browsing/mock_feature_extractor_clock.h",
     ]
   }
 }