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

Side by Side Diff: blimp/BUILD.gn

Issue 2387813002: Move session into //blimp/client/app and update GN files. (Closed)
Patch Set: Rebased for good measure Created 4 years, 2 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 | blimp/client/BUILD.gn » ('j') | blimp/client/core/context/BUILD.gn » ('J')
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/chrome_build.gni") 5 import("//build/config/chrome_build.gni")
6 import("//build/config/ui.gni")
7 import("//testing/test.gni") 6 import("//testing/test.gni")
8 7
9 if (is_android) {
10 import("//build/config/android/config.gni")
11 import("//build/config/android/rules.gni")
12 }
13
14 # Convenience meta-target for all of Blimp's production & test code. 8 # Convenience meta-target for all of Blimp's production & test code.
15 group("blimp") { 9 group("blimp") {
16 # In spite of the name, this really just means that non-test targets 10 # In spite of the name, this really just means that non-test targets
17 # cannot depend on this one, and that it can depend on test targets, 11 # cannot depend on this one, and that it can depend on test targets,
18 # which fits how we intend to use it. 12 # which fits how we intend to use it.
19 testonly = true 13 testonly = true
20 14
21 deps = [ 15 deps = [
22 ":blimp_tests", 16 ":blimp_tests",
23 "//blimp/client", 17 "//blimp/client",
24 "//blimp/common", 18 "//blimp/common",
25 ] 19 ]
26 20
27 if (is_android) {
28 deps += [
29 "//blimp/client:blimp_apk",
30 "//blimp/client:blimp_apk_incremental",
31 ]
32 }
33
34 if (is_linux) { 21 if (is_linux) {
35 deps += [ 22 deps += [
36 ":client_engine_integration", 23 ":client_engine_integration",
37 "//blimp/engine", 24 "//blimp/engine",
38 "//blimp/engine:blimp_engine_bundle", 25 "//blimp/engine:blimp_engine_bundle",
39 ] 26 ]
40 27
41 if (is_official_build) { 28 if (is_official_build) {
42 deps += [ "//blimp/engine:blimp_symbols" ] 29 deps += [ "//blimp/engine:blimp_symbols" ]
43 } 30 }
44 } 31 }
45
46 if (is_linux && !is_chromeos && use_x11) {
47 deps += [ "//blimp/client:blimp_shell" ]
48 }
49 } 32 }
50 33
51 # Builds all Blimp test code (unit tests, APKs). 34 # Builds all Blimp test code (unit tests, test APKs).
52 group("blimp_tests") { 35 group("blimp_tests") {
53 testonly = true 36 testonly = true
54 37
55 deps = [ 38 deps = [
56 ":blimp_unittests", 39 ":blimp_unittests",
40 "//blimp/client:test_binaries",
57 ] 41 ]
58 42
59 if (is_android) {
60 deps += [ "//blimp/client:blimp_test_apk" ]
61 }
62
63 if (is_linux) { 43 if (is_linux) {
64 deps += [ ":blimp_browsertests" ] 44 deps += [ ":blimp_browsertests" ]
65 } 45 }
66 } 46 }
67 47
68 test("blimp_unittests") { 48 test("blimp_unittests") {
69 deps = [ 49 deps = [
70 "//blimp/client:app_unit_tests", 50 "//blimp/client:unit_tests",
71 "//blimp/client/core:unit_tests",
72 "//blimp/common:unit_tests", 51 "//blimp/common:unit_tests",
73 "//blimp/net:unit_tests", 52 "//blimp/net:unit_tests",
74 "//blimp/test:run_all_unittests", 53 "//blimp/test:run_all_unittests",
75 ] 54 ]
76 55
77 if (is_linux) { 56 if (is_linux) {
78 deps += [ "//blimp/engine:unit_tests" ] 57 deps += [ "//blimp/engine:unit_tests" ]
79 } 58 }
80 59
81 if (is_android) { 60 if (is_android) {
82 deps += [ ":blimp_unittests_java_deps" ] 61 deps += [ "//blimp/client:blimp_unittests_java_deps" ]
83 } 62 }
84 } 63 }
85 64
86 if (is_android) {
87 # This is the list of targets that the tests need to depend on in order to add
88 # the java classes for their native counterparts to the test apk.
89 # We could directly include them in the test target, but in order to
90 # keep the visibility of the //blimp/client/core:core_java target restricted,
91 # we make it visible to this group (which is testonly) and depend on this
92 # group instead.
93 # We could not add ":blimp_unittests" to the visibility list of
94 # //blimp/client/core:core_java, since the targets that actually need to
95 # depend on it are generated by the test template (see testing/test.gni). So
96 # it was better to add this indirection and expose it to this testonly target,
97 # which can be used in this file only.
98 java_group("blimp_unittests_java_deps") {
99 testonly = true
100 visibility = [ ":*" ]
101
102 deps = [
103 "//base:base_java",
104 "//blimp/client/core:core_java",
105 "//net/android:net_java",
106 "//ui/android:ui_java",
107 ]
108 }
109 }
110
111 if (is_linux) { 65 if (is_linux) {
112 test("blimp_browsertests") { 66 test("blimp_browsertests") {
113 deps = [ 67 deps = [
114 "//blimp/engine:browser_tests", 68 "//blimp/engine:browser_tests",
115 "//tools/xdisplaycheck", 69 "//tools/xdisplaycheck",
116 ] 70 ]
117 data = [ 71 data = [
118 "test/data/", 72 "test/data/",
119 ] 73 ]
120 } 74 }
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
245 "--output", 199 "--output",
246 rebase_path(_output_path, root_build_dir), 200 rebase_path(_output_path, root_build_dir),
247 "--target", 201 "--target",
248 rebase_path(_wrapped_script, root_build_dir), 202 rebase_path(_wrapped_script, root_build_dir),
249 "--output-directory", 203 "--output-directory",
250 rebase_path(root_out_dir, root_build_dir), 204 rebase_path(root_out_dir, root_build_dir),
251 "--flag-name=--output-linux-directory", 205 "--flag-name=--output-linux-directory",
252 ] 206 ]
253 } 207 }
254 } 208 }
OLDNEW
« no previous file with comments | « no previous file | blimp/client/BUILD.gn » ('j') | blimp/client/core/context/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698