[Android] Workaround of an android platform bug.

build/java_apk.gypi

 # Copyright (c) 2012 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.
 
 # This file is meant to be included into a target to provide a rule
 # to build Android APKs in a consistent manner.
 #
 # To use this, create a gyp target with the following form:
 # {
 #   'target_name': 'my_package_apk',
@@ skipping 67 matching lines @@
     'compile_input_paths': [],
     'package_input_paths': [],
     'ordered_libraries_file': '<(intermediate_dir)/native_libraries.json',
     'native_libraries_template': '<(DEPTH)/base/android/java/templates/NativeLibraries.template',
     'native_libraries_java_dir': '<(intermediate_dir)/native_libraries_java/',
     'native_libraries_java_file': '<(native_libraries_java_dir)/NativeLibraries.java',
     'native_libraries_java_stamp': '<(intermediate_dir)/native_libraries_java.stamp',
     'native_libraries_template_data_dir': '<(intermediate_dir)/native_libraries/',
     'native_libraries_template_data_file': '<(native_libraries_template_data_dir)/native_libraries_array.h',
     'native_libraries_template_version_file': '<(native_libraries_template_data_dir)/native_libraries_version.h',
+    'native_libraries_cleanup_stamp': '<(intermediate_dir)/native_libraries_cleanup.stamp',
     'compile_stamp': '<(intermediate_dir)/compile.stamp',
     'lint_stamp': '<(intermediate_dir)/lint.stamp',
     'lint_result': '<(intermediate_dir)/lint_result.xml',
     'lint_config': '<(intermediate_dir)/lint_config.xml',
     'never_lint%': 0,
     'instr_stamp': '<(intermediate_dir)/instr.stamp',
     'jar_stamp': '<(intermediate_dir)/jar.stamp',
     'obfuscate_stamp': '<(intermediate_dir)/obfuscate.stamp',
     'strip_stamp': '<(intermediate_dir)/strip.stamp',
     'classes_dir': '<(intermediate_dir)/classes',
@@ skipping 78 matching lines @@
     ['native_lib_target != "" and component == "shared_library"', {
       'dependencies': [
         '<(DEPTH)/build/android/setup.gyp:copy_system_libraries',
       ],
     }],
     ['use_chromium_linker == 1', {
       'dependencies': [
         '<(DEPTH)/base/base.gyp:chromium_android_linker',
       ],
     }],
+    ['native_lib_target != "" and component != "shared_library" and use_chromium_linker != 1', {

[cjhopman, 2014/03/17 18:32:35]

I think this would be better if it just explicitly listed the libraries to keep
and then deleted everything else.

Also, this shouldn't be disabled when use_chromium_linker == 1 (since
use_chromium_linker==1 for chrome_apk).

[Feng Qian, 2014/03/18 00:07:01]

Listing libraries to keep will have issues with building multiple targets, if
the list only has libraries of current target, it will delete libraries of other
targets.

I am not sure if it will work.

Removing 'use_chromium_linker' check. It reminds me that I forgot to include
changes in Linker.java :)

On 2014/03/17 18:32:35, cjhopman wrote:

[cjhopman, 2014/03/18 00:16:55]

Oh right, because this is deleting the ones in <(SHARED_LIB_DIR) too. How about,
along with making this only affect libchromeview we also move the action into
that target or the chrome_apk target (i.e. it shouldn't really be needed for all
apks).

+      # cleanup old libraries in PRODUCT_DIR and package lib directories
+      'variables': {
+        'package_input_paths': [ '<(native_libraries_cleanup_stamp)' ],
+      },
+      'actions': [
+        {
+          'action_name': 'cleanup_old_libraries',
+          'inputs': [
+            '<(DEPTH)/chrome/VERSION',
+            '<(DEPTH)/build/android/gyp/delete_files.py',
+          ],
+          'outputs': [
+            '<(native_libraries_cleanup_stamp)',
+          ],
+          'action': [
+            'python', '<(DEPTH)/build/android/gyp/delete_files.py',
+            '--stamp=<(native_libraries_cleanup_stamp)',
+            '--pattern=<(native_lib_target).*.so*',
+            '--keep=<(native_lib_target).>(android_product_extension)*',
+            '<(SHARED_LIB_DIR)',
+            '<(apk_package_native_libs_dir)/<(android_app_abi)',
+          ],
+        },
+      ],
+    }],
     ['native_lib_target != ""', {
       'variables': {
         'compile_input_paths': [ '<(native_libraries_java_stamp)' ],
         'generated_src_dirs': [ '<(native_libraries_java_dir)' ],
         'native_libs_paths': [
           '<(SHARED_LIB_DIR)/<(native_lib_target).>(android_product_extension)'
         ],
         'package_input_paths': [
           '<(apk_package_native_libs_dir)/<(android_app_abi)/gdbserver',
         ],
@@ skipping 560 matching lines @@
         '<(DEPTH)/build/android/ant/apk-package.xml',
 
         # Add list of inputs to the command line, so if inputs change
         # (e.g. if a Java file is removed), the command will be re-run.
         # TODO(newt): remove this once crbug.com/177552 is fixed in ninja.
         '-DTHIS_IS_IGNORED=>!(echo \'>(_inputs)\' | md5sum)',
       ]
     },
   ],
 }