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

Side by Side Diff: testing/test.gni

Issue 2082453003: 🎅 Use GN's dependency info for native libraries in write_build_config.py (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix osmesa on non-android Created 4 years, 6 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
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 # ============================================================================== 5 # ==============================================================================
6 # TEST SETUP 6 # TEST SETUP
7 # ============================================================================== 7 # ==============================================================================
8 8
9 template("_gen_isolate") { 9 template("_gen_isolate") {
10 testonly = true 10 testonly = true
(...skipping 169 matching lines...) Expand 10 before | Expand all | Expand 10 after
180 _apk_specific_vars + _wrapper_script_vars + 180 _apk_specific_vars + _wrapper_script_vars +
181 _gen_isolate_vars + [ "visibility" ]) 181 _gen_isolate_vars + [ "visibility" ])
182 182
183 if (!defined(invoker.use_default_launcher) || 183 if (!defined(invoker.use_default_launcher) ||
184 invoker.use_default_launcher) { 184 invoker.use_default_launcher) {
185 deps += [ "//testing/android/native_test:native_test_native_code" ] 185 deps += [ "//testing/android/native_test:native_test_native_code" ]
186 } 186 }
187 } 187 }
188 unittest_apk(_apk_target) { 188 unittest_apk(_apk_target) {
189 forward_variables_from(invoker, _apk_specific_vars + [ "deps" ]) 189 forward_variables_from(invoker, _apk_specific_vars + [ "deps" ])
190 unittests_dep = ":$_library_target" 190 shared_library = ":$_library_target"
191 apk_name = invoker.target_name 191 apk_name = invoker.target_name
192 if (defined(invoker.output_name)) { 192 if (defined(invoker.output_name)) {
193 apk_name = invoker.output_name 193 apk_name = invoker.output_name
194 unittests_binary = "lib${apk_name}.so"
195 install_script_name = "install_${invoker.output_name}" 194 install_script_name = "install_${invoker.output_name}"
196 } 195 }
197 deps += [ ":$_library_target" ]
198 196
199 # TODO(agrieve): Remove this data_dep once bots don't build the _apk 197 # TODO(agrieve): Remove this data_dep once bots don't build the _apk
200 # target (post-GYP). 198 # target (post-GYP).
201 # It's a bit backwards for the apk to depend on the runner script, since 199 # It's a bit backwards for the apk to depend on the runner script, since
202 # the apk is conceptually a runtime_dep of the script. However, it is 200 # the apk is conceptually a runtime_dep of the script. However, it is
203 # currently necessary because the bots build this _apk target directly 201 # currently necessary because the bots build this _apk target directly
204 # rather than the group() below. 202 # rather than the group() below.
205 data_deps = [ 203 data_deps = [
206 ":$_test_runner_target", 204 ":$_test_runner_target",
207 ] 205 ]
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
359 if (defined(invoker.output_name) && target_name != invoker.output_name) { 357 if (defined(invoker.output_name) && target_name != invoker.output_name) {
360 group("${invoker.output_name}_run") { 358 group("${invoker.output_name}_run") {
361 testonly = true 359 testonly = true
362 deps = [ 360 deps = [
363 ":${invoker.target_name}", 361 ":${invoker.target_name}",
364 ] 362 ]
365 } 363 }
366 } 364 }
367 } 365 }
368 } 366 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698