Add create_density_splits option to java_apk.gypi / android_apk (gn)

build/config/android/internal_rules.gni

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/android/config.gni")
 
 assert(is_android)
 
 rebased_android_sdk = rebase_path(android_sdk, root_build_dir)
 rebased_android_sdk_root = rebase_path(android_sdk_root, root_build_dir)
@@ skipping 420 matching lines @@
     ]
   }
 
   group(target_name) {
     deps = [
       ":${target_name}__jar_toc",
     ]
   }
 }
 
+template("finalize_apk") {
+  action(target_name) {
+    script = "//build/android/gyp/finalize_apk.py"
+    depfile = "$target_gen_dir/$target_name.d"
+
+    sources = [
+      invoker.input_apk_path,
+    ]
+    inputs = [
+      invoker.keystore_path,
+    ]
+    outputs = [
+      depfile,
+      invoker.output_apk_path,
+    ]
+
+    args = [
+      "--depfile",
+      rebase_path(depfile, root_build_dir),
+      "--zipalign-path",
+      rebase_path(zipalign_path, root_build_dir),
+      "--unsigned-apk-path",
+      rebase_path(invoker.input_apk_path, root_build_dir),
+      "--final-apk-path",
+      rebase_path(invoker.output_apk_path, root_build_dir),
+      "--key-path",
+      rebase_path(invoker.keystore_path, root_build_dir),
+      "--key-name",
+      invoker.keystore_name,
+      "--key-passwd",
+      invoker.keystore_password,
+    ]
+    if (defined(invoker.rezip_apk) && invoker.rezip_apk) {
+      _rezip_jar_path = "$root_build_dir/lib.java/rezip_apk.jar"
+      inputs += [ _rezip_jar_path ]
+      args += [
+        "--load-library-from-zip-file=1",
+        "--rezip-apk-jar-path",
+        rebase_path(_rezip_jar_path, root_build_dir),
+      ]
+    }
+  }
+}
+
 # Packages resources, assets, dex, and native libraries into an apk. Signs and
 # zipaligns the apk.
 template("create_apk") {
   set_sources_assignment_filter([])
   if (defined(invoker.testonly)) {
     testonly = invoker.testonly
   }
 
   _android_manifest = invoker.android_manifest
   _base_path = invoker.base_path
   _final_apk_path = invoker.apk_path
 
   if (defined(invoker.resources_zip)) {
     _resources_zip = invoker.resources_zip
   }
   if (defined(invoker.dex_path)) {
     _dex_path = invoker.dex_path
   }
-  _keystore_path = invoker.keystore_path
-  _keystore_name = invoker.keystore_name
-  _keystore_password = invoker.keystore_password
   _load_library_from_apk = invoker.load_library_from_apk
 
-  _deps = []
+  _package_deps = []
   if (defined(invoker.deps)) {
-    _deps = invoker.deps
+    _package_deps = invoker.deps
   }
 
   _native_libs_dir = "//build/android/empty/res"
   if (defined(invoker.native_libs_dir)) {
     _native_libs_dir = invoker.native_libs_dir
   }
 
   if (defined(invoker.asset_location)) {
     _asset_location = invoker.asset_location
   }
 
   _version_code = invoker.version_code
   _version_name = invoker.version_name
 
   _base_apk_path = _base_path + ".apk_intermediates"
 
   _resource_packaged_apk_path = _base_apk_path + ".ap_"
   _packaged_apk_path = _base_apk_path + ".unfinished.apk"
   _shared_resources =
       defined(invoker.shared_resources) && invoker.shared_resources
 
   _configuration_name = "Release"
   if (is_debug) {
     _configuration_name = "Debug"
   }
 
-  action("${target_name}__package_resources") {
-    deps = _deps
+  _create_density_splits =
+      defined(invoker.create_density_splits) && invoker.create_density_splits
+  if (_create_density_splits) {
+    _split_densities = [
+      "hdpi",
+      "xhdpi",
+      "xxhdpi",
+      "tvdpi",
+    ]
+  }
+
+  _package_resources_target_name = "${target_name}__package_resources"
+  action(_package_resources_target_name) {
+    deps = _package_deps
 
     script = "//build/android/gyp/package_resources.py"
     depfile = "${target_gen_dir}/${target_name}.d"
     inputs = [
       _android_manifest,
     ]
     if (defined(_resources_zip)) {
       inputs += [ _resources_zip ]
     }
     outputs = [
@@ skipping 27 matching lines @@
     }
     if (defined(_resources_zip)) {
       args += [
         "--resource-zips",
         rebase_path(_resources_zip, root_build_dir),
       ]
     }
     if (_shared_resources) {
       args += [ "--shared-resources" ]
     }
+    if (_create_density_splits) {
+      args += [ "--create-density-splits" ]
+      foreach(_density, _split_densities) {
+        outputs += [ "${_resource_packaged_apk_path}-${_density}" ]
+      }
+    }
   }
 
   action("${target_name}__package") {
     script = "//build/android/gyp/ant.py"
     _ant_script = "//build/android/ant/apk-package.xml"
 
+    deps = [
+      ":${_package_resources_target_name}",
+    ]
     depfile = "$target_gen_dir/$target_name.d"
 
     inputs = [
       _resource_packaged_apk_path,
       _ant_script,
     ]
     if (defined(_dex_path)) {
       inputs += [ _dex_path ]
     }
 
@@ skipping 29 matching lines @@
       _rebased_dex_path = rebase_path(_dex_path, root_build_dir)
       args += [
         "-DDEX_FILE_PATH=$_rebased_dex_path",
         "-DHAS_CODE=true",
       ]
     } else {
       args += [ "-DHAS_CODE=false" ]
     }
   }
 
-  action("${target_name}__finalize") {
-    script = "//build/android/gyp/finalize_apk.py"
-    depfile = "$target_gen_dir/$target_name.d"
+  finalize_apk("${target_name}__finalize") {
+    input_apk_path = _packaged_apk_path
+    output_apk_path = _final_apk_path
+    keystore_path = invoker.keystore_path
+    keystore_name = invoker.keystore_name
+    keystore_password = invoker.keystore_password
+    rezip_apk = _load_library_from_apk
+  }
 
-    sources = [
-      _packaged_apk_path,
-    ]
-    inputs = [
-      _keystore_path,
-    ]
-    outputs = [
-      depfile,
-      _final_apk_path,
-    ]
+  _final_deps = [ ":${target_name}__finalize" ]
 
-    args = [
-      "--depfile",
-      rebase_path(depfile, root_build_dir),
-      "--zipalign-path",
-      rebase_path(zipalign_path, root_build_dir),
-      "--unsigned-apk-path",
-      rebase_path(_packaged_apk_path, root_build_dir),
-      "--final-apk-path",
-      rebase_path(_final_apk_path, root_build_dir),
-      "--key-path",
-      rebase_path(_keystore_path, root_build_dir),
-      "--key-name",
-      _keystore_name,
-      "--key-passwd",
-      _keystore_password,
-    ]
-    if (_load_library_from_apk) {
-      _rezip_jar_path = "$root_build_dir/lib.java/rezip_apk.jar"
-      inputs += [ _rezip_jar_path ]
-      args += [
-        "--load-library-from-zip-file=1",
-        "--rezip-apk-jar-path",
-        rebase_path(_rezip_jar_path, root_build_dir),
-      ]
+  if (_create_density_splits) {
+    foreach(_density, _split_densities) {
+      finalize_apk("${target_name}__finalize_${_density}_split") {

[cjhopman, 2015/05/20 01:51:51]

nit: I'd extract this target name to a variable.

[agrieve, 2015/05/20 14:49:36]

Done.

+        input_apk_path = "${_resource_packaged_apk_path}-${_density}"
+        _output_paths = process_file_template(
+                [ _final_apk_path ],
+                "{{source_dir}}/{{source_name_part}}-${_density}.apk")
+        output_apk_path = _output_paths[0]
+        keystore_path = invoker.keystore_path
+        keystore_name = invoker.keystore_name
+        keystore_password = invoker.keystore_password
+      }
+      _final_deps += [ ":${target_name}__finalize_${_density}_split" ]
     }
   }
 
   group(target_name) {
-    deps = [
-      ":${target_name}__finalize",
-      ":${target_name}__package_resources",
-    ]
+    deps = _final_deps
   }
 }
 
 template("java_prebuilt_impl") {
   set_sources_assignment_filter([])
   if (defined(invoker.testonly)) {
     testonly = invoker.testonly
   }
   _supports_android =
       defined(invoker.supports_android) && invoker.supports_android
@@ skipping 594 matching lines @@
     script = "//build/android/gyp/generate_split_manifest.py"
     outputs = [
       depfile,
       invoker.out_manifest,
     ]
     inputs = [
       invoker.main_manifest,
     ]
   }
 }