GN: Add missing deps for incremental apks.

build/config/android/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("//base/android/linker/config.gni")
 import("//build/config/android/config.gni")
 import("//build/config/android/internal_rules.gni")
 import("//build/toolchain/toolchain.gni")
 import("//third_party/android_platform/config.gni")
 import("//tools/grit/grit_rule.gni")
@@ skipping 1296 matching lines @@
 
     deps = _chromium_linker_dep + _android_manifest_deps
     if (defined(invoker.deps)) {
       deps += invoker.deps
     }
 
     native_libs = _native_libs
   }
 
   _final_deps = []
-  _incremental_final_deps = []
 
   process_resources_target = "${_template_name}__process_resources"
   process_resources(process_resources_target) {
     forward_variables_from(invoker, [ "include_all_resources" ])
     srcjar_path = "${target_gen_dir}/${target_name}.srcjar"
     r_text_path = "${target_gen_dir}/${target_name}_R.txt"
     android_manifest = _android_manifest
     resource_dirs = [ "//build/android/ant/empty/res" ]
     zip_path = resources_zip_path
     all_resources_zip_path = _all_resources_zip_path
@@ skipping 116 matching lines @@
     ]
     output = final_dex_path
     _dex_arg_key = "${_rebased_build_config}:final_dex:dependency_dex_files"
     args = [
       "--inputs=@FileArg($_dex_arg_key)",
       _rebased_lib_dex_path,
     ]
   }
 
   if (_native_libs != []) {
-    action("${_template_name}__prepare_native") {
+    _prepare_native_target_name = "${_template_name}__prepare_native"
+    action(_prepare_native_target_name) {
       forward_variables_from(invoker,
                              [
                                "data_deps",
                                "public_deps",
                              ])
       script = "//build/android/gyp/pack_relocations.py"
       packed_libraries_dir = "$_native_libs_dir/$android_app_abi"
       depfile = "$target_gen_dir/$target_name.d"
       outputs = [
         depfile,
@@ skipping 37 matching lines @@
 
       if (is_debug) {
         rebased_gdbserver = rebase_path([ android_gdbserver ], root_build_dir)
         inputs += [ android_gdbserver ]
         args += [ "--libraries=$rebased_gdbserver" ]
       }
     }
   }
 
   _final_deps += [ ":${_template_name}__create" ]
-  _incremental_final_deps += [ ":${_template_name}__create_incremental" ]
   create_apk("${_template_name}__create") {
-    forward_variables_from(invoker, [ "language_splits" ])
+    deps = []
+    forward_variables_from(invoker,
+                           [
+                             "asset_location",
+                             "deps",
+                             "extensions_to_not_compress",
+                             "language_splits",
+                           ])
     apk_path = _final_apk_path
     android_manifest = _android_manifest
     resources_zip = _all_resources_zip_path
     dex_path = final_dex_path
     load_library_from_apk = _load_library_from_apk
     create_density_splits = _create_density_splits
-    if (defined(invoker.extensions_to_not_compress)) {
-      extensions_to_not_compress = invoker.extensions_to_not_compress
-    } else {
+
+    if (!defined(extensions_to_not_compress)) {
       # Allow icu data, v8 snapshots, and pak files to be loaded directly from
       # the .apk.
       # Note: These are actually suffix matches, not necessarily extensions.
       extensions_to_not_compress = ".dat,.bin,.pak"
     }
 
     version_code = _version_code
     version_name = _version_name
 
     keystore_name = _keystore_name
     keystore_path = _keystore_path
     keystore_password = _keystore_password
 
+    # Incremental apk does not use native libs nor final dex.
+    incremental_deps =
+        deps + _android_manifest_deps + [ ":$process_resources_target" ]
+
     # This target generates the input file _all_resources_zip_path.
-    deps = _android_manifest_deps + [
-             ":$process_resources_target",
-             ":$final_dex_target_name",
-           ]
-    incremental_deps = _android_manifest_deps + [ ":$process_resources_target" ]
-    if (defined(invoker.deps)) {
-      deps += invoker.deps
-      incremental_deps += invoker.deps
-    }
-
-    if (defined(invoker.asset_location)) {
-      asset_location = invoker.asset_location
-
-      # We don't know the exact dependencies that create the assets in
-      # |asset_location|; we depend on all caller deps until a better solution
-      # is figured out (http://crbug.com/433330).
-      if (defined(invoker.deps)) {
-        deps += invoker.deps
-      }
-    }
+    deps += _android_manifest_deps + [
+              ":$process_resources_target",
+              ":$final_dex_target_name",
+            ]
 
     if (_native_libs != [] && !_create_abi_split) {
       native_libs_dir = _native_libs_dir
-      deps += [ ":${_template_name}__prepare_native" ]
+      deps += [ ":$_prepare_native_target_name" ]
     }
   }
 
   if (_native_libs != [] && _create_abi_split) {
     _manifest_rule = "${_template_name}__split_manifest_abi_${android_app_abi}"
     generate_split_manifest(_manifest_rule) {
       main_manifest = _android_manifest
       out_manifest =
           "$gen_dir/split-manifests/${android_app_abi}/AndroidManifest.xml"
       split_name = "abi_${android_app_abi}"
       deps = _android_manifest_deps
     }
 
     _apk_rule = "${_template_name}__split_apk_abi_${android_app_abi}"
     _final_deps += [ ":$_apk_rule" ]
 
-    # Don't add this to _incremental_final_deps since for incremental installs
-    # we skip abi splits altogether.
     create_apk(_apk_rule) {
       apk_path = "${_final_apk_path_no_ext}-abi-${android_app_abi}.apk"
       base_path = "$gen_dir/$_apk_rule"
 
       manifest_outputs = get_target_outputs(":${_manifest_rule}")
       android_manifest = manifest_outputs[1]
       load_library_from_apk = _load_library_from_apk
 
       version_code = _version_code
       version_name = _version_name
 
       keystore_name = _keystore_name
       keystore_path = _keystore_path
       keystore_password = _keystore_password
 
       native_libs_dir = _native_libs_dir
       deps = [
-        ":${_template_name}__prepare_native",
+        ":${_prepare_native_target_name}",
         ":${_manifest_rule}",
       ]
       incremental_deps = deps
     }
   }
 
   _create_incremental_script_rule_name = "${_template_name}__incremental_script"
-  _incremental_final_deps += [ ":${_create_incremental_script_rule_name}" ]
   action(_create_incremental_script_rule_name) {
     script = "//build/android/incremental_install/create_install_script.py"
     depfile = "$target_gen_dir/$target_name.d"
     deps = [
       ":$build_config_target",
     ]
 
     _generated_script_path =
         "${root_out_dir}/bin/install_incremental_${_template_name}"
     outputs = [
@@ skipping 26 matching lines @@
     }
   }
 
   group(target_name) {
     forward_variables_from(invoker, [ "data_deps" ])
     public_deps = _final_deps
   }
   group("${target_name}_incremental") {
     data_deps = []
     forward_variables_from(invoker, [ "data_deps" ])
+
+    # device/commands is used by the installer script to push files via .zip.
     data_deps += [ "//build/android/pylib/device/commands" ]
-    public_deps = _incremental_final_deps
+
+    # Since the _incremental.apk does not include use .so nor .dex from the
+    # actual target, but instead loads them at runtime, we need to explicitly
+    # depend on them here.

[pkotwicz, 2015/09/25 14:19:30]

Thanks for the comment!

+    public_deps = [
+      ":${_template_name}__create_incremental",
+      ":${_create_incremental_script_rule_name}",
+      ":${java_target}",
+    ]
+    if (_native_libs != []) {
+      public_deps += [ ":$_prepare_native_target_name" ]
+    }
   }
 }
 
 # Declare an Android instrumentation test apk
 #
 # This target creates an Android instrumentation test apk.
 #
 # Variables
 #   android_manifest: Path to AndroidManifest.xml.
 #   data_deps: List of dependencies needed at runtime. These will be built but
@@ skipping 369 matching lines @@
 
 # TODO(GYP): implement this.
 template("uiautomator_test") {
   set_sources_assignment_filter([])
   forward_variables_from(invoker, [ "testonly" ])
   assert(target_name != "")
   assert(invoker.deps != [] || true)
   group(target_name) {
   }
 }