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

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

Issue 1952153002: 👪 Strip R.class in android_prebuilt() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Don't strip out *all* R.class, just those associated with known resources 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
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 1197 matching lines...) Expand 10 before | Expand all | Expand 10 after
1208 # ":foo_resources", 1208 # ":foo_resources",
1209 # ":bar_java" 1209 # ":bar_java"
1210 # ] 1210 # ]
1211 # } 1211 # }
1212 template("android_java_prebuilt") { 1212 template("android_java_prebuilt") {
1213 set_sources_assignment_filter([]) 1213 set_sources_assignment_filter([])
1214 java_prebuilt_impl(target_name) { 1214 java_prebuilt_impl(target_name) {
1215 forward_variables_from(invoker, "*") 1215 forward_variables_from(invoker, "*")
1216 supports_android = true 1216 supports_android = true
1217 requires_android = true 1217 requires_android = true
1218 strip_resource_classes = true
1218 } 1219 }
1219 } 1220 }
1220 1221
1221 # Declare an Android apk target 1222 # Declare an Android apk target
1222 # 1223 #
1223 # This target creates an Android APK containing java code, resources, assets, 1224 # This target creates an Android APK containing java code, resources, assets,
1224 # and (possibly) native libraries. 1225 # and (possibly) native libraries.
1225 # 1226 #
1226 # Variables 1227 # Variables
1227 # alternative_android_sdk_jar: The alternative android sdk jar used in 1228 # alternative_android_sdk_jar: The alternative android sdk jar used in
(...skipping 1204 matching lines...) Expand 10 before | Expand all | Expand 10 after
2432 "--target", 2433 "--target",
2433 rebase_path(invoker.target, root_build_dir), 2434 rebase_path(invoker.target, root_build_dir),
2434 "--output-directory", 2435 "--output-directory",
2435 rebase_path(root_out_dir, root_build_dir), 2436 rebase_path(root_out_dir, root_build_dir),
2436 ] 2437 ]
2437 if (defined(invoker.flag_name)) { 2438 if (defined(invoker.flag_name)) {
2438 args += [ "--flag-name=${invoker.flag_name}" ] 2439 args += [ "--flag-name=${invoker.flag_name}" ]
2439 } 2440 }
2440 } 2441 }
2441 } 2442 }
OLDNEW
« build/android/gyp/jar.py ('K') | « build/config/android/internal_rules.gni ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698