Revert of Make secondary abi work for component build

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("//build/config/android/config.gni")
 import("//build/config/android/internal_rules.gni")
 import("//build/config/dcheck_always_on.gni")
 import("//build/toolchain/toolchain.gni")
 
 assert(is_android)
@@ skipping 1402 matching lines @@
   #       * dependencies of this .so are not automatically included
   #       * ".cr.so" is never added
   #       * they are not side-loaded for _incremental targets.
   #       * load_library_from_apk, use_chromium_linker,
   #         and enable_relocation_packing do not apply
   #     Use this instead of shared_libraries when you are going to load the library
   #     conditionally, and only when shared_libraries doesn't work for you.
   #   shared_libraries: List shared_library targets to bundle. If these
   #     libraries depend on other shared_library targets, those dependencies will
   #     also be included in the apk (e.g. for is_component_build).
-  #   secondary_abi_shared_libraries: secondary abi shared_library targets to
-  #     bundle. If these libraries depend on other shared_library targets, those
-  #     dependencies will also be included in the apk (e.g. for is_component_build).
   #   native_lib_placeholders: List of placeholder filenames to add to the apk
   #     (optional).
   #   apk_under_test: For an instrumentation test apk, this is the target of the
   #     tested apk.
   #   include_all_resources - If true include all resource IDs in all generated
   #     R.java files.
   #   testonly: Marks this target as "test-only".
   #   write_asset_list: Adds an extra file to the assets, which contains a list of
   #     all other asset files.
   #   alternative_locale_resource_dep: The locale resource target which overrides
   #     any exsting locale resources in dep graph.
   #   requires_sdk_api_level_23: If defined and true, the apk is intended for
   #     installation only on Android M or later. In these releases the system
   #     linker does relocation unpacking, so we can enable it unconditionally.
-  #   secondary_native_libs (deprecated): The path of native libraries for secondary
-  #     app abi.
+  #   secondary_native_libs: The path of native libraries for secondary app abi.
   #   run_findbugs_override: Forces run_findbugs on or off. If undefined, the
   #     default will use the build arg run_findbugs.
   #   proguard_jar_path: The path to proguard.jar you wish to use. If undefined,
   #     the proguard used will be the checked in one in //third_party/proguard.
   #
   # Example
   #   android_apk("foo_apk") {
   #     android_manifest = "AndroidManifest.xml"
   #     java_files = [
   #       "android/org/chromium/foo/FooApplication.java",
@@ skipping 95 matching lines @@
                  "or requires_sdk_api_level_23")
     }
     if (_load_library_from_apk) {
       assert(_use_chromium_linker || _requires_sdk_api_level_23,
              "load_library_from_apk requires use_chromium_linker " +
                  "or requires_sdk_api_level_23")
     }
 
     # The dependency that makes the chromium linker, if any is needed.
     _native_libs_deps = []
-    _shared_libraries_is_valid =
-        defined(invoker.shared_libraries) && invoker.shared_libraries != []
-    _secondary_abi_native_libs_deps = []
-    assert(_secondary_abi_native_libs_deps == [])  # mark as used.
-    _secondary_abi_shared_libraries_is_valid =
-        defined(invoker.secondary_abi_shared_libraries) &&
-        invoker.secondary_abi_shared_libraries != []
 
-    if (is_component_build || is_asan) {
-      if (_shared_libraries_is_valid) {
+    if (defined(invoker.shared_libraries) && invoker.shared_libraries != []) {
+      _native_libs_deps += invoker.shared_libraries
+
+      if (is_component_build || is_asan) {
         _native_libs_deps += [ "//build/android:cpplib_stripped" ]
       }
-      if (_secondary_abi_shared_libraries_is_valid) {
-        _secondary_abi_native_libs_deps += [ "//build/android:cpplib_stripped($android_secondary_abi_toolchain)" ]
-      }
-    }
-
-    if (_shared_libraries_is_valid) {
-      _native_libs_deps += invoker.shared_libraries
 
       # To determine the filenames of all dependent shared libraries, write the
       # runtime deps of |shared_libraries| to a file during "gn gen".
       # write_build_config.py will then grep this file for *.so to obtain the
       # complete list.
       _runtime_deps_file =
           "$target_gen_dir/${_template_name}.native.runtimedeps"
       group("${_template_name}__runtime_deps") {
         deps = _native_libs_deps
         write_runtime_deps = _runtime_deps_file
       }
 
       _native_lib_version_rule = ""
       if (defined(invoker.native_lib_version_rule)) {
         _native_lib_version_rule = invoker.native_lib_version_rule
       }
       _native_lib_version_arg = "\"\""
       if (defined(invoker.native_lib_version_arg)) {
         _native_lib_version_arg = invoker.native_lib_version_arg
       }
     }
 
-    if (_secondary_abi_shared_libraries_is_valid) {
-      _secondary_abi_native_libs_deps += invoker.secondary_abi_shared_libraries
-
-      # To determine the filenames of all dependent shared libraries, write the
-      # runtime deps of |shared_libraries| to a file during "gn gen".
-      # write_build_config.py will then grep this file for *.so to obtain the
-      # complete list.
-      _secondary_abi_runtime_deps_file =
-          "$target_gen_dir/${_template_name}.secondary.abi.native.runtimedeps"
-      group("${_template_name}_secondary_abi__runtime_deps") {
-        deps = _secondary_abi_native_libs_deps
-        write_runtime_deps = _secondary_abi_runtime_deps_file
-      }
-    }
-
     if (defined(invoker.deps)) {
       set_sources_assignment_filter([ "*manifest*" ])
       sources = invoker.deps
       set_sources_assignment_filter([])
       if (sources != invoker.deps) {
         _bad_deps = invoker.deps - sources
         assert(
             false,
             "Possible manifest-generating dep found in deps. Use android_manifest_dep for this instead. Found: $_bad_deps")
       }
@@ skipping 58 matching lines @@
         proguard_info = "$_proguard_output_jar_path.info"
       }
 
       # Don't depend on the runtime_deps target in order to avoid having to
       # build the native libraries just to create the .build_config file.
       # The dep is unnecessary since the runtime_deps file is created by gn gen
       # and the runtime_deps file is added to write_build_config.py's depfile.
       if (_native_libs_deps != []) {
         shared_libraries_runtime_deps_file = _runtime_deps_file
       }
-      if (_secondary_abi_native_libs_deps != []) {
-        secondary_abi_shared_libraries_runtime_deps_file =
-            _secondary_abi_runtime_deps_file
-      }
     }
 
     _final_deps = []
 
     _generated_proguard_config = "$base_path.resources.proguard.txt"
     process_resources_target = "${_template_name}__process_resources"
     process_resources(process_resources_target) {
       forward_variables_from(invoker,
                              [
                                "alternative_android_sdk_jar",
@@ skipping 247 matching lines @@
         } else {
           _dex_arg_key =
               "${_rebased_build_config}:final_dex:dependency_dex_files"
         }
         args = [ "--inputs=@FileArg($_dex_arg_key)" ]
       }
     }
 
     _native_libs_file_arg_dep = ":$build_config_target"
     _native_libs_file_arg = "@FileArg($_rebased_build_config:native:libraries)"
-    _secondary_abi_native_libs_file_arg_dep = ":$build_config_target"
-    _secondary_abi_native_libs_file_arg =
-        "@FileArg($_rebased_build_config:native:secondary_abi_libraries)"
-    assert(_secondary_abi_native_libs_file_arg != "" &&
-           _secondary_abi_native_libs_file_arg_dep != "")  # Mark as used.
 
     if (_native_libs_deps != [] && _enable_relocation_packing) {
       _prepare_native_target_name = "${_template_name}__prepare_native"
+      _native_libs_dir = "$gen_dir/packed-libs"
       _native_libs_json = "$gen_dir/packed-libs/filelist.json"
       _rebased_native_libs_json = rebase_path(_native_libs_json, root_build_dir)
+
       _native_libs_file_arg_dep = ":$_prepare_native_target_name"
       _native_libs_file_arg = "@FileArg($_rebased_native_libs_json:files)"
 
-      pack_relocation_section(_prepare_native_target_name) {
+      action(_prepare_native_target_name) {
         forward_variables_from(invoker,
                                [
                                  "deps",
                                  "public_deps",
                                ])
-        file_list_json = _native_libs_json
-        libraries_filearg =
-            "@FileArg(${_rebased_build_config}:native:libraries)"
+        script = "//build/android/gyp/pack_relocations.py"
+        depfile = "$target_gen_dir/$target_name.d"
+        outputs = [
+          _native_libs_json,
+        ]
+
         inputs = [
           _build_config,
         ]
 
         deps += _native_libs_deps
-        deps += [ ":$build_config_target" ]
-      }
-      if (_secondary_abi_native_libs_deps != []) {
-        _prepare_native_target_name =
-            "${_template_name}_secondary_abi__prepare_native"
-        _native_libs_json =
-            "$gen_dir/packed-libs/$android_secondary_abi_cpu/filelist.json"
-        _rebased_native_libs_json =
-            rebase_path(_native_libs_json, root_build_dir)
-        _secondary_abi_native_libs_file_arg_dep =
-            ":$_prepare_native_target_name"
-        _secondary_abi_native_libs_file_arg =
-            "@FileArg($_rebased_native_libs_json:files)"
+        deps += [
+          ":$build_config_target",
+          relocation_packer_target,
+        ]
 
-        pack_relocation_section(_prepare_native_target_name) {
-          forward_variables_from(invoker,
-                                 [
-                                   "deps",
-                                   "public_deps",
-                                 ])
-          file_list_json = _native_libs_json
-          libraries_filearg = "@FileArg(${_rebased_build_config}:native:secondary_abi_libraries)"
-          inputs = [
-            _build_config,
-          ]
-
-          deps += _secondary_abi_native_libs_deps
-          deps += [ ":$build_config_target" ]
-        }
+        args = [
+          "--depfile",
+          rebase_path(depfile, root_build_dir),
+          "--enable-packing=1",
+          "--android-pack-relocations",
+          rebase_path(relocation_packer_exe, root_build_dir),
+          "--stripped-libraries-dir",
+          rebase_path(root_build_dir, root_build_dir),
+          "--packed-libraries-dir",
+          rebase_path(_native_libs_dir, root_build_dir),
+          "--libraries=@FileArg(${_rebased_build_config}:native:libraries)",
+          "--filelistjson=$_rebased_native_libs_json",
+        ]
       }
     }
 
     _extra_native_libs = []
     _extra_native_libs_deps = []
     _extra_native_libs_even_when_incremental = []
     _extra_native_libs_even_when_incremental_deps = []
     assert(_extra_native_libs_even_when_incremental_deps == [])  # Mark as used.
     if (_native_libs_deps != []) {
       # zipalign can't align gdb_server, don't pack gdbserver temporarily.
@@ skipping 74 matching lines @@
           !_create_abi_split) {
         deps += _native_libs_deps + _extra_native_libs_deps +
                 _extra_native_libs_even_when_incremental_deps +
                 [ _native_libs_file_arg_dep ]
         native_libs_filearg = _native_libs_file_arg
         native_libs = _extra_native_libs
         native_libs_even_when_incremental =
             _extra_native_libs_even_when_incremental
       }
 
-      if (_secondary_abi_native_libs_deps != [] && !_create_abi_split) {
-        deps += _secondary_abi_native_libs_deps +
-                [ _secondary_abi_native_libs_file_arg_dep ]
-        secondary_abi_native_libs_filearg = _secondary_abi_native_libs_file_arg
-      }
-
       # Placeholders necessary for some older devices.
       # http://crbug.com/395038
       forward_variables_from(invoker, [ "native_lib_placeholders" ])
     }
 
     if ((_native_libs_deps != [] ||
          _extra_native_libs_even_when_incremental != []) && _create_abi_split) {
       _manifest_rule =
           "${_template_name}__split_manifest_abi_${android_app_abi}"
       generate_split_manifest(_manifest_rule) {
@@ skipping 684 matching lines @@
         # because in practice they seem to contain classes required to be in the
         # classpath.
         deps += _subjar_targets
       }
       if (defined(_res_target_name)) {
         deps += [ ":$_res_target_name" ]
       }
     }
   }
 }