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

Side by Side Diff: build/config/android/rules.gni

Issue 2000333003: Allow test and unittest_apk GN targets to use proguard (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove the "+" Created 4 years, 7 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 | « AUTHORS ('k') | testing/test.gni » ('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 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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("//base/android/linker/config.gni") 5 import("//base/android/linker/config.gni")
6 import("//build/config/android/config.gni") 6 import("//build/config/android/config.gni")
7 import("//build/config/android/internal_rules.gni") 7 import("//build/config/android/internal_rules.gni")
8 import("//build/config/sanitizers/sanitizers.gni") 8 import("//build/config/sanitizers/sanitizers.gni")
9 import("//build/toolchain/toolchain.gni") 9 import("//build/toolchain/toolchain.gni")
10 import("//third_party/android_platform/config.gni") 10 import("//third_party/android_platform/config.gni")
(...skipping 2116 matching lines...) Expand 10 before | Expand all | Expand 10 after
2127 testonly = true 2127 testonly = true
2128 2128
2129 assert(defined(unittests_dep), "Need unittests_dep for $target_name") 2129 assert(defined(unittests_dep), "Need unittests_dep for $target_name")
2130 2130
2131 test_suite_name = get_label_info(unittests_dep, "name") 2131 test_suite_name = get_label_info(unittests_dep, "name")
2132 2132
2133 # This trivial assert is needed in case both unittests_binary and apk_name 2133 # This trivial assert is needed in case both unittests_binary and apk_name
2134 # are defined, as otherwise test_suite_name would not be used. 2134 # are defined, as otherwise test_suite_name would not be used.
2135 assert(test_suite_name != "") 2135 assert(test_suite_name != "")
2136 2136
2137 if (defined(invoker.proguard_enabled) && invoker.proguard_enabled) {
2138 assert(invoker.proguard_configs != [])
2139 proguard_enabled = true
2140 proguard_configs = invoker.proguard_configs
2141 }
2142
2137 if (!defined(apk_name)) { 2143 if (!defined(apk_name)) {
2138 apk_name = test_suite_name 2144 apk_name = test_suite_name
2139 } 2145 }
2140 2146
2141 if (!defined(android_manifest)) { 2147 if (!defined(android_manifest)) {
2142 android_manifest = 2148 android_manifest =
2143 "//testing/android/native_test/java/AndroidManifest.xml" 2149 "//testing/android/native_test/java/AndroidManifest.xml"
2144 } 2150 }
2145 2151
2146 if (!defined(unittests_binary)) { 2152 if (!defined(unittests_binary)) {
(...skipping 280 matching lines...) Expand 10 before | Expand all | Expand 10 after
2427 "--target", 2433 "--target",
2428 rebase_path(invoker.target, root_build_dir), 2434 rebase_path(invoker.target, root_build_dir),
2429 "--output-directory", 2435 "--output-directory",
2430 rebase_path(root_out_dir, root_build_dir), 2436 rebase_path(root_out_dir, root_build_dir),
2431 ] 2437 ]
2432 if (defined(invoker.flag_name)) { 2438 if (defined(invoker.flag_name)) {
2433 args += [ "--flag-name=${invoker.flag_name}" ] 2439 args += [ "--flag-name=${invoker.flag_name}" ]
2434 } 2440 }
2435 } 2441 }
2436 } 2442 }
OLDNEW
« no previous file with comments | « AUTHORS ('k') | testing/test.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698