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

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

Issue 2181053002: android: dcheck_always_on sets BuildConfig.IS_DCHECK (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix findbugs Created 4 years, 4 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("//build/config/android/config.gni") 5 import("//build/config/android/config.gni")
6 import("//build/config/android/internal_rules.gni") 6 import("//build/config/android/internal_rules.gni")
7 import("//build/config/dcheck_always_on.gni")
7 import("//build/toolchain/toolchain.gni") 8 import("//build/toolchain/toolchain.gni")
8 9
9 assert(is_android) 10 assert(is_android)
10 11
11 # Creates a dist directory for a native executable. 12 # Creates a dist directory for a native executable.
12 # 13 #
13 # Running a native executable on a device requires all the shared library 14 # Running a native executable on a device requires all the shared library
14 # dependencies of that executable. To make it easier to install and run such an 15 # dependencies of that executable. To make it easier to install and run such an
15 # executable, this will create a directory containing the native exe and all 16 # executable, this will create a directory containing the native exe and all
16 # it's library dependencies. 17 # it's library dependencies.
(...skipping 1730 matching lines...) Expand 10 before | Expand all | Expand 10 after
1747 java_cpp_template("${_template_name}__build_config_java") { 1748 java_cpp_template("${_template_name}__build_config_java") {
1748 package_name = "org/chromium/base" 1749 package_name = "org/chromium/base"
1749 sources = [ 1750 sources = [
1750 "//base/android/java/templates/BuildConfig.template", 1751 "//base/android/java/templates/BuildConfig.template",
1751 ] 1752 ]
1752 1753
1753 defines = [] 1754 defines = []
1754 if (enable_multidex) { 1755 if (enable_multidex) {
1755 defines += [ "ENABLE_MULTIDEX" ] 1756 defines += [ "ENABLE_MULTIDEX" ]
1756 } 1757 }
1757 if (!is_java_debug) { 1758 if (is_java_debug || dcheck_always_on) {
1758 defines += [ "NDEBUG" ] 1759 defines += [ "_DCHECK_IS_ON" ]
1759 } 1760 }
1760 } 1761 }
1761 _srcjar_deps += [ ":${_template_name}__build_config_java" ] 1762 _srcjar_deps += [ ":${_template_name}__build_config_java" ]
1762 } 1763 }
1763 1764
1764 java_target = "${_template_name}__java" 1765 java_target = "${_template_name}__java"
1765 java_library_impl(java_target) { 1766 java_library_impl(java_target) {
1766 forward_variables_from(invoker, 1767 forward_variables_from(invoker,
1767 [ 1768 [
1768 "chromium_code", 1769 "chromium_code",
(...skipping 894 matching lines...) Expand 10 before | Expand all | Expand 10 after
2663 android_manifest = "${_output_path}/AndroidManifest.xml" 2664 android_manifest = "${_output_path}/AndroidManifest.xml"
2664 v14_skip = true 2665 v14_skip = true
2665 } 2666 }
2666 } 2667 }
2667 2668
2668 java_group(target_name) { 2669 java_group(target_name) {
2669 deps = _sub_target_names 2670 deps = _sub_target_names
2670 } 2671 }
2671 } 2672 }
2672 } 2673 }
OLDNEW
« no previous file with comments | « build/config/android/config.gni ('k') | chrome/android/java/src/org/chromium/chrome/browser/ChromeStrictMode.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698