Revert of 🎊 Have build_config targets depend only on other build_config targets

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")
 import("//build/config/sanitizers/sanitizers.gni")
 
 assert(is_android)
 
-# These identify targets that have .build_config files (except for android_apk,
-# java_binary, resource_rewriter, since we never need to depend on these).
-_java_target_whitelist = [
-  "*:*_java",
-  "*:*_javalib",
-  "*:*_java_*",  # e.g. java_test_support
-  "*:java",
-  "*:junit",
-  "*:junit_*",
-  "*:*_junit_*",
-  "*:*javatests",
-  "*:*_assets",
-  "*android*:assets",
-  "*:*_apk_*resources",
-  "*android*:resources",
-  "*:*_resources",
-  "*:*_grd",
-  "*:*locale_paks",
-
-  # TODO(agrieve): Rename targets below to match above patterns.
-  "//android_webview/glue:glue",
-  "//build/android/pylib/device/commands:chromium_commands",
-  "//build/android/rezip:rezip",
-  "//chrome/test/android/cast_emulator:cast_emulator",
-  "//components/cronet/android:cronet_api",
-  "//components/cronet/android:cronet_javadoc_classpath",
-  "//components/policy:app_restrictions_resources",
-  "//device/battery/android:battery_monitor_android",
-  "//device/vibration/android:vibration_manager_android",
-  "//mojo/public/java:bindings",
-  "//mojo/public/java:system",
-  "//third_party/android_tools:emma_device",
-  "//third_party/cardboard-java:cardboard-java",
-  "//third_party/custom_tabs_client:custom_tabs_client_shared_lib",
-  "//third_party/custom_tabs_client:custom_tabs_support_lib",
-  "//third_party/errorprone:chromium_errorprone",
-  "//third_party/haha:haha",
-  "//third_party/junit:hamcrest",
-  "//third_party/netty4:netty_all",
-  "//third_party/netty-tcnative:netty-tcnative",
-  "//third_party/robolectric:android-all-4.3_r2-robolectric-0",
-  "//third_party/robolectric:json-20080701",
-  "//third_party/robolectric:tagsoup-1.2",
-]
-
-# Targets that match the whitelist but are not actually java targets.
-_java_target_blacklist = [
-  "//chrome:packed_extra_resources",
-  "//chrome:packed_resources",
-  "//remoting/android:remoting_android_raw_resources",
-]
-
 # Write the target's .build_config file. This is a json file that contains a
 # dictionary of information about how to build this target (things that
 # require knowledge about this target's dependencies and cannot be calculated
 # at gn-time). There is a special syntax to add a value in that dictionary to
 # an action/action_foreachs args:
 #   --python-arg=@FileArg($rebased_build_config_path:key0:key1)
 # At runtime, such an arg will be replaced by the value in the build_config.
 # See build/android/gyp/write_build_config.py and
 # build/android/gyp/util/build_utils.py:ExpandFileArgs
 template("write_build_config") {
-  type = invoker.type
-
-  # Don't need to enforce naming scheme for these targets since we never
-  # consider them in dependency chains.
-  if (type != "android_apk" && type != "java_binary" &&
-      type != "resource_rewriter") {
-    set_sources_assignment_filter(_java_target_whitelist)
-    _parent_invoker = invoker.invoker
-    _target_label =
-        get_label_info(":${_parent_invoker.target_name}", "label_no_toolchain")
-    sources = [
-      _target_label,
-    ]
-    if (sources != []) {
-      set_sources_assignment_filter(_java_target_blacklist)
-      sources = []
-      sources = [
-        _target_label,
-      ]
-      if (sources != []) {
-        assert(false, "Invalid java target name: $_target_label")
-      }
-    }
-    sources = []
-  }
-
   action(target_name) {
     set_sources_assignment_filter([])
+    type = invoker.type
     build_config = invoker.build_config
 
     assert(type == "android_apk" || type == "java_library" ||
            type == "android_resources" || type == "deps_dex" ||
            type == "android_assets" || type == "resource_rewriter" ||
            type == "java_binary" || type == "group")
 
     forward_variables_from(invoker,
                            [
                              "deps",
                              "testonly",
+                             "visibility",
                            ])
     if (!defined(deps)) {
       deps = []
     }
 
     script = "//build/android/gyp/write_build_config.py"
     depfile = "$target_gen_dir/$target_name.d"
     inputs = []
 
-    _deps_configs = []
-    if (defined(invoker.possible_config_deps)) {
-      foreach(_possible_dep, invoker.possible_config_deps) {
-        set_sources_assignment_filter(_java_target_whitelist)
-        _target_label = get_label_info(_possible_dep, "label_no_toolchain")
-        sources = [
-          _target_label,
-        ]
-        if (sources == []) {
-          set_sources_assignment_filter(_java_target_blacklist)
-          sources = []
-          sources = [
-            _target_label,
-          ]
-          if (sources != []) {
-            deps += [ "${_target_label}__build_config" ]
-            _dep_gen_dir = get_label_info(_possible_dep, "target_gen_dir")
-            _dep_name = get_label_info(_possible_dep, "name")
-            _deps_configs += [ "$_dep_gen_dir/$_dep_name.build_config" ]
-          }
-        }
-        sources = []
-      }
-      set_sources_assignment_filter([])
+    possible_deps_configs = []
+    foreach(d, deps) {
+      dep_gen_dir = get_label_info(d, "target_gen_dir")
+      dep_name = get_label_info(d, "name")
+      possible_deps_configs += [ "$dep_gen_dir/$dep_name.build_config" ]
     }
-    _rebased_deps_configs = rebase_path(_deps_configs, root_build_dir)
+    rebase_possible_deps_configs =
+        rebase_path(possible_deps_configs, root_build_dir)
 
     outputs = [
       depfile,
       build_config,
     ]
 
     args = [
       "--type",
       type,
       "--depfile",
       rebase_path(depfile, root_build_dir),
-      "--deps-configs=$_rebased_deps_configs",
+      "--possible-deps-configs=$rebase_possible_deps_configs",
       "--build-config",
       rebase_path(build_config, root_build_dir),
     ]
 
     is_java = type == "java_library" || type == "java_binary"
     is_apk = type == "android_apk"
     is_android_assets = type == "android_assets"
     is_android_resources = type == "android_resources"
     is_deps_dex = type == "deps_dex"
     is_group = type == "group"
@@ skipping 49 matching lines @@
       apk_under_test_config =
           "$apk_under_test_gen_dir/$apk_under_test_name.build_config"
       args += [
         "--tested-apk-config",
         rebase_path(apk_under_test_config, root_build_dir),
       ]
     }
 
     if (is_android_assets) {
       if (defined(invoker.asset_sources)) {
+        inputs += invoker.asset_sources
         _rebased_asset_sources =
             rebase_path(invoker.asset_sources, root_build_dir)
         args += [ "--asset-sources=$_rebased_asset_sources" ]
       }
       if (defined(invoker.asset_renaming_sources)) {
+        inputs += invoker.asset_renaming_sources
         _rebased_asset_renaming_sources =
             rebase_path(invoker.asset_renaming_sources, root_build_dir)
         args += [ "--asset-renaming-sources=$_rebased_asset_renaming_sources" ]
 
         # These are zip paths, so no need to rebase.
         args += [ "--asset-renaming-destinations=${invoker.asset_renaming_destinations}" ]
       }
       if (defined(invoker.disable_compression) && invoker.disable_compression) {
         args += [ "--disable-asset-compression" ]
       }
@@ skipping 1434 matching lines @@
     if (_supports_android) {
       _dex_target_name = "${_template_name}__dex"
     }
 
     write_build_config(_build_config_target_name) {
       type = "java_library"
       supports_android = _supports_android
       requires_android =
           defined(invoker.requires_android) && invoker.requires_android
 
-      if (defined(invoker.deps)) {
-        possible_config_deps = _deps
-      }
+      deps = _deps
       build_config = _build_config
       jar_path = _jar_path
       if (_supports_android) {
         dex_path = _dex_path
       }
     }
 
     process_java_prebuilt(_process_jar_target_name) {
       forward_variables_from(invoker,
                              [
@@ skipping 386 matching lines @@
 
     # Define build_config_deps which will be a list of targets required to
     # build the _build_config.
     if (defined(invoker.override_build_config)) {
       _build_config = invoker.override_build_config
     } else {
       _build_config = _base_path + ".build_config"
       build_config_target_name = "${_template_name}__build_config"
 
       write_build_config(build_config_target_name) {
+        deps = _accumulated_deps
         if (defined(invoker.is_java_binary) && invoker.is_java_binary) {
           type = "java_binary"
         } else {
           type = "java_library"
         }
-        if (defined(invoker.deps)) {
-          possible_config_deps = invoker.deps
-        }
         supports_android = _supports_android
         requires_android = _requires_android
         bypass_platform_checks = defined(invoker.bypass_platform_checks) &&
                                  invoker.bypass_platform_checks
 
         build_config = _build_config
         jar_path = _jar_path
         if (_supports_android) {
           dex_path = _dex_path
         }
@@ skipping 281 matching lines @@
     }
   }
 
   # Produces a single .dex.jar out of a set of Java dependencies.
   template("deps_dex") {
     set_sources_assignment_filter([])
     build_config = "$target_gen_dir/${target_name}.build_config"
     build_config_target_name = "${target_name}__build_config"
 
     write_build_config(build_config_target_name) {
-      forward_variables_from(invoker, [ "dex_path" ])
-      if (defined(invoker.deps)) {
-        possible_config_deps = invoker.deps
-      }
+      forward_variables_from(invoker,
+                             [
+                               "deps",
+                               "dex_path",
+                             ])
       type = "deps_dex"
       build_config = build_config
     }
 
     rebased_build_config = rebase_path(build_config, root_build_dir)
     dex(target_name) {
       inputs = [
         build_config,
       ]
       output = invoker.dex_path
@@ skipping 57 matching lines @@
       outputs = [
         depfile,
         invoker.out_manifest,
       ]
       inputs = [
         invoker.main_manifest,
       ]
     }
   }
 }