Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1121)

Unified Diff: remoting/host/linux/BUILD.gn

Issue 2438773003: Update CRD ChromeOS Host build files to be better in line with GN style. (Closed)
Patch Set: Moving unicode_to_keysym to x11, that is the only place the class is used. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/host/chromeos/BUILD.gn ('k') | remoting/host/mac/constants_mac.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/linux/BUILD.gn
diff --git a/remoting/host/linux/BUILD.gn b/remoting/host/linux/BUILD.gn
index 0c7ed70e6f60d5c47c1ed4fec8eba44a4f0f79d3..8a763368da0d9dd11e4ae0c481c9132d75094f36 100644
--- a/remoting/host/linux/BUILD.gn
+++ b/remoting/host/linux/BUILD.gn
@@ -56,22 +56,30 @@ source_set("linux") {
"audio_pipe_reader.h",
"certificate_watcher.cc",
"certificate_watcher.h",
- "unicode_to_keysym.cc",
- "unicode_to_keysym.h",
]
deps = [
"//remoting/protocol",
"//third_party/webrtc/modules/desktop_capture",
]
+ public_deps = []
if (use_x11) {
deps += [ ":x11" ]
}
+ if (is_desktop_linux) {
+ deps += [ "//build/config/linux/gtk" ]
+ }
}
source_set("x11") {
sources = [
+ "unicode_to_keysym.cc",
+ "unicode_to_keysym.h",
+ "x11_character_injector.cc",
+ "x11_character_injector.h",
+ "x11_keyboard_impl.cc",
+ "x11_keyboard_impl.h",
"x11_util.cc",
"x11_util.h",
"x_server_clipboard.cc",
@@ -108,3 +116,39 @@ target("executable", "remoting_native_messaging_host") {
[ "VERSION=" + "$chrome_version_major" + "." + "$remoting_version_patch" +
"." + "$chrome_version_build" + "." + "$chrome_version_patch" ]
}
+
+source_set("unit_tests") {
+ testonly = true
+
+ sources = [
+ "audio_pipe_reader_unittest.cc",
+ "certificate_watcher_unittest.cc",
+ "unicode_to_keysym_unittest.cc",
+ "x11_character_injector_unittest.cc",
+ "x_server_clipboard_unittest.cc",
+ ]
+
+ if (!use_x11) {
+ sources -= [ "unicode_to_keysym_unittest.cc" ]
+ }
+
+ configs += [ "//remoting/build/config:version" ]
+
+ deps = [
+ "//remoting/host",
+ "//remoting/host:test_support",
+ "//remoting/host/it2me:common",
+ "//remoting/host/native_messaging",
+ "//remoting/host/security_key:unit_tests",
+ "//remoting/host/setup",
+ "//remoting/proto",
+ "//remoting/resources",
+ "//skia",
+ "//testing/gmock",
+ "//testing/gtest",
+ ]
+
+ if (!is_ios) {
+ deps += [ "//components/policy/core/browser:test_support" ]
+ }
+}
« no previous file with comments | « remoting/host/chromeos/BUILD.gn ('k') | remoting/host/mac/constants_mac.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698