Make the ChromeOS GN debug component build work

remoting/host/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("//build/config/ui.gni")
 import("//remoting/remoting_host.gni")
 import("//remoting/remoting_locales.gni")
 import("//remoting/remoting_srcs.gni")
 import("//remoting/remoting_version.gni")
@@ skipping 66 matching lines @@
       if (is_linux) {
         # These will already be filtered out on non-Linux.
         sources -= [
           "linux/x_server_clipboard.cc",
           "linux/x_server_clipboard.h",
         ]
       }
     }
 
     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",
         "//skia",
         "//ui/aura",
         "//ui/compositor",
         "//ui/events",
         "//ui/views",
       ]
 
@@ skipping 42 matching lines @@
 
     if (is_win) {
       deps += [
         ":messages",
         ":remoting_lib_idl",
       ]
     }
 
     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",
       ]
 
       sources +=
           rebase_path(remoting_host_srcs_gypi_values.remoting_cast_sources,
                       ".",
                       "//remoting")
     }
   }
@@ skipping 254 matching lines @@
           "host_main.h",
         ]
 
         deps = [
           ":remoting_me2me_host_static",
         ]
       }
     }
   }
 }