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

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

Issue 2123753005: Creating new gn arg: enable_all_proguard_optimizations. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changing from 2 apks to gn arg which controls optimizations Created 4 years, 5 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 # This file contains common system config stuff for the Android build. 5 # This file contains common system config stuff for the Android build.
6 6
7 if (is_android) { 7 if (is_android) {
8 assert(rebase_path("//", root_build_dir) == "../../", 8 assert(rebase_path("//", root_build_dir) == "../../",
9 "Android output directory must be nested 2 levels within src/ (" + 9 "Android output directory must be nested 2 levels within src/ (" +
10 "e.g.: out-gn/Debug). http://crbug.com/412935") 10 "e.g.: out-gn/Debug). http://crbug.com/412935")
(...skipping 99 matching lines...) Expand 10 before | Expand all | Expand 10 after
110 110
111 # EMMA filter string consisting of a list of inclusion/exclusion patterns 111 # EMMA filter string consisting of a list of inclusion/exclusion patterns
112 # separated with whitespace and/or comma. Only has effect if 112 # separated with whitespace and/or comma. Only has effect if
113 # emma_coverage==true 113 # emma_coverage==true
114 emma_filter = "" 114 emma_filter = ""
115 115
116 # Disables process isolation when building _incremental targets. 116 # Disables process isolation when building _incremental targets.
117 # Required for Android M+ due to SELinux policies (stronger sandboxing). 117 # Required for Android M+ due to SELinux policies (stronger sandboxing).
118 disable_incremental_isolated_processes = false 118 disable_incremental_isolated_processes = false
119 119
120 # Enables all ProGuard optimizations. These optimizations must not be
121 # enabled for instrumentation tests, as they break the tests.
agrieve 2016/07/08 14:26:20 nit: change ", as they break the tests" -> "since
smaier 2016/07/08 14:50:07 Done.
122 enable_all_proguard_optimizations = is_official_build
123
120 # Speed up incremental compiles by compiling only changed files. 124 # Speed up incremental compiles by compiling only changed files.
121 enable_incremental_javac = false 125 enable_incremental_javac = false
122 126
123 # Speed up dexing using dx --incremental. 127 # Speed up dexing using dx --incremental.
124 enable_incremental_dx = is_java_debug 128 enable_incremental_dx = is_java_debug
125 129
126 # Neither of these should ever be used for release builds since they are 130 # Neither of these should ever be used for release builds since they are
127 # somewhat experimental and dx --incremental is known to not produce 131 # somewhat experimental and dx --incremental is known to not produce
128 # byte-for-byte identical output. 132 # byte-for-byte identical output.
129 assert(!(enable_incremental_dx && !is_java_debug)) 133 assert(!(enable_incremental_dx && !is_java_debug))
(...skipping 145 matching lines...) Expand 10 before | Expand all | Expand 10 after
275 } else if (current_cpu == "mips64el") { 279 } else if (current_cpu == "mips64el") {
276 android_app_abi = "mips64" 280 android_app_abi = "mips64"
277 } else { 281 } else {
278 assert(false, "Unknown Android ABI: " + current_cpu) 282 assert(false, "Unknown Android ABI: " + current_cpu)
279 } 283 }
280 284
281 if (android_libcpp_lib_dir == "") { 285 if (android_libcpp_lib_dir == "") {
282 android_libcpp_lib_dir = "${android_libcpp_root}/libs/${android_app_abi}" 286 android_libcpp_lib_dir = "${android_libcpp_root}/libs/${android_app_abi}"
283 } 287 }
284 } 288 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698