Make the ChromeOS GN debug component build work

remoting/host/BUILD.gn

Index: remoting/host/BUILD.gn
diff --git a/remoting/host/BUILD.gn b/remoting/host/BUILD.gn
index 8e71b25687abf93d4a8898369e618889c2b79642..06409b58d8898a9612eead7f226910b39eff3da2 100644
--- a/remoting/host/BUILD.gn
+++ b/remoting/host/BUILD.gn
@@ -84,6 +84,10 @@ if (is_mac) {  # TODO(GYP) Mac build of remoting host.
     }
 
     if (is_chromeos) {
+      sources -= [
+        "me2me_desktop_environment.cc",
+        "me2me_desktop_environment.h",
+      ]

[Dirk Pranke, 2015/04/23 01:19:17]

I don't understand why, but when these files are linked :host, linking
//chrome:main_dll fails, complaining that remoting::CurtainMode::Create is not
defined.

It seems like these files should perhaps be moved to a different variable that
is only included when enable_me2me_host==true.

I do not understand why this file isn't getting stripped out as dead code by the
linker, though, since :host is a static library.

[garykac, 2015/04/27 18:27:02]

Agreed.  But that seems out of scope for this cl. Can you add a TODO comment
describing the issue?

[Dirk Pranke, 2015/04/27 19:52:25]

Will do.

       deps += [
         "//cc",
         "//ppapi/host",
@@ -146,6 +150,7 @@ if (is_mac) {  # TODO(GYP) Mac build of remoting host.
 
     if (enable_webrtc) {
       deps += [
+        "//third_party/libjingle:libjingle_webrtc",

[Dirk Pranke, 2015/04/23 01:19:17]

libpeerconnection depends //third_party/libjingle:libjingle_webrtc_common , but
those files need someone to define the symbols provided in libjingle_webrtc
(overrides/init_webrtc). Perhaps libpeerconnection should depend on
libjingle_webrtc rather than libjingle_webrtc_common?

         "//third_party/libjingle:libpeerconnection",
         "//third_party/webrtc/modules/desktop_capture",
       ]