GN + Android: extract android_standalone_library rule.

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 716 matching lines @@
   }
 }
 
 template("java_library_impl") {
   if (defined(invoker.testonly)) {
     testonly = invoker.testonly
   }
 
   assert(
       defined(invoker.java_files) || defined(invoker.DEPRECATED_java_in_dir) ||
-      defined(invoker.srcjars) || defined(invoker.srcjar_deps))
+      defined(invoker.srcjars) || defined(invoker.srcjar_deps) ||
+      defined(invoker.deps))
   _base_path = "$target_gen_dir/$target_name"
   _jar_path = _base_path + ".jar"
   if (defined(invoker.jar_path)) {
     _jar_path = invoker.jar_path
   }
   _template_name = target_name
 
   _final_deps = []
   _final_datadeps = []
   if (defined(invoker.datadeps)) {
@@ skipping 30 matching lines @@
       }
 
       build_config = _build_config
       jar_path = _jar_path
       if (_supports_android) {
         dex_path = _dex_path
       }
     }
   }
 
-  _chromium_code = true
-  if (defined(invoker.chromium_code)) {
-    _chromium_code = invoker.chromium_code
-  }
-
   _srcjar_deps = []
   if (defined(invoker.srcjar_deps)) {
     _srcjar_deps = invoker.srcjar_deps
   }
 
   _srcjars = []
   if (defined(invoker.srcjars)) {
     _srcjars = invoker.srcjars
   }
 
@@ skipping 12 matching lines @@
     _java_files_build_rel = exec_script(
             "//build/android/gyp/find.py",
             [
               "--pattern",
               "*.java",
               rebase_path(invoker.DEPRECATED_java_in_dir, root_build_dir),
             ],
             "list lines")
     _java_files = rebase_path(_java_files_build_rel, ".", root_build_dir)
   }
-  assert(_java_files != [] || _srcjar_deps != [] || _srcjars != [])
 
-  _final_deps += [ ":${_template_name}__compile_java" ]
-  compile_java("${_template_name}__compile_java") {
-    jar_path = _jar_path
-    build_config = _build_config
-    java_files = _java_files
-    srcjar_deps = _srcjar_deps
-    srcjars = _srcjars
-    chromium_code = _chromium_code
-    android = _requires_android
+  if (_java_files != [] || _srcjar_deps != [] || _srcjars != []) {

[cjhopman, 2014/12/05 19:11:55]

I don't think this is a good idea. The build_config is going to be wrong for
this target (for example, if an android_library depends on this target the build
will fail in a hard to diagnose way).

If we want a target that just does dex merging like this, it should probably not
be implemented with java_library_impl. It should write it's own build_config
(that basically just contains the list of files to dex).

[ppi, 2014/12/08 16:38:27]

Done.

+    _chromium_code = true
+    if (defined(invoker.chromium_code)) {
+      _chromium_code = invoker.chromium_code
+    }
 
-    if (defined(invoker.jar_excluded_patterns)) {
-      jar_excluded_patterns = invoker.jar_excluded_patterns
-    }
-    if (defined(invoker.proguard_preprocess)) {
-      proguard_preprocess = invoker.proguard_preprocess
-    }
-    if (defined(invoker.proguard_config)) {
-      proguard_config = invoker.proguard_config
-    }
-    if (defined(invoker.dist_jar_path)) {
-      dist_jar_path = invoker.dist_jar_path
+    _final_deps += [ ":${_template_name}__compile_java" ]
+    compile_java("${_template_name}__compile_java") {
+      jar_path = _jar_path
+      build_config = _build_config
+      java_files = _java_files
+      srcjar_deps = _srcjar_deps
+      srcjars = _srcjars
+      chromium_code = _chromium_code
+      android = _requires_android
+
+      if (defined(invoker.jar_excluded_patterns)) {
+        jar_excluded_patterns = invoker.jar_excluded_patterns
+      }
+      if (defined(invoker.proguard_preprocess)) {
+        proguard_preprocess = invoker.proguard_preprocess
+      }
+      if (defined(invoker.proguard_config)) {
+        proguard_config = invoker.proguard_config
+      }
+      if (defined(invoker.dist_jar_path)) {
+        dist_jar_path = invoker.dist_jar_path
+      }
     }
   }
 
   if (defined(invoker.main_class)) {
     _final_deps += [ ":${_template_name}__binary_script" ]
     action("${_template_name}__binary_script") {
       script = "//build/android/gyp/create_java_binary_script.py"
       depfile = "$target_gen_dir/$target_name.d"
       java_script = "$root_build_dir/bin/$_template_name"
       inputs = [
@@ skipping 196 matching lines @@
 
     if (defined(invoker.clear_dir) && invoker.clear_dir) {
       args += [ "--clear" ]
     }
 
     if (defined(invoker.args)) {
       args += invoker.args
     }
   }
 }