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

Side by Side Diff: remoting/BUILD.gn

Issue 1009883002: Fix unresolved gn dependencies when enable_configuration_policy=false (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Patch gn-formatted Created 5 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 unified diff | Download patch
« no previous file with comments | « no previous file | remoting/host/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/features.gni") 5 import("//build/config/features.gni")
6 import("//build/config/ui.gni") 6 import("//build/config/ui.gni")
7 import("//remoting/remoting_version.gni") 7 import("//remoting/remoting_version.gni")
8 import("//testing/test.gni") 8 import("//testing/test.gni")
9 9
10 enable_remoting_host = is_win || is_mac || is_chromeos || use_x11 10 enable_remoting_host = is_win || is_mac || is_chromeos || use_x11
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
54 "test/remote_host_info_fetcher.cc", 54 "test/remote_host_info_fetcher.cc",
55 "test/remote_host_info_fetcher.h", 55 "test/remote_host_info_fetcher.h",
56 "test/test_chromoting_client.cc", 56 "test/test_chromoting_client.cc",
57 "test/test_chromoting_client.h", 57 "test/test_chromoting_client.h",
58 "test/test_video_renderer.cc", 58 "test/test_video_renderer.cc",
59 "test/test_video_renderer.h", 59 "test/test_video_renderer.h",
60 ] 60 ]
61 61
62 deps = [ 62 deps = [
63 "//base", 63 "//base",
64 "//components/policy:test_support",
65 "//net", 64 "//net",
66 "//remoting/base", 65 "//remoting/base",
67 "//remoting/client", 66 "//remoting/client",
68 "//remoting/codec", 67 "//remoting/codec",
69 "//remoting/protocol:test_support", 68 "//remoting/protocol:test_support",
70 "//remoting/resources", 69 "//remoting/resources",
71 "//testing/gmock", 70 "//testing/gmock",
72 "//testing/gtest", 71 "//testing/gtest",
73 ] 72 ]
74 73
74 if (enable_configuration_policy) {
75 deps += [ "//components/policy:test_support" ]
76 }
77
75 if (enable_remoting_host) { 78 if (enable_remoting_host) {
76 deps += [ "//remoting/host:test_support" ] 79 deps += [ "//remoting/host:test_support" ]
77 } 80 }
78 } 81 }
79 82
80 # TODO(GYP) remoting_unittests on Mac/Windows. Currently this fails with a 83 # TODO(GYP) remoting_unittests on Mac/Windows. Currently this fails with a
81 # duplicate resource error on linking on Windows. Just needs to be checked on 84 # duplicate resource error on linking on Windows. Just needs to be checked on
82 # Mac. 85 # Mac.
83 if (!is_win && !is_mac) { 86 if (!is_win && !is_mac) {
84 test("remoting_unittests") { 87 test("remoting_unittests") {
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
130 deps += [ 133 deps += [
131 "//third_party/libjingle:libjingle_webrtc", 134 "//third_party/libjingle:libjingle_webrtc",
132 "//third_party/libjingle:libpeerconnection", 135 "//third_party/libjingle:libpeerconnection",
133 ] 136 ]
134 } 137 }
135 } 138 }
136 } else { 139 } else {
137 group("remoting_unittests") { 140 group("remoting_unittests") {
138 } 141 }
139 } 142 }
OLDNEW
« no previous file with comments | « no previous file | remoting/host/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698