🍫 Enable support-annotations lint checks

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")
 import("//build/config/zip.gni")
 import("//third_party/ijar/ijar.gni")
 
 assert(is_android)
@@ skipping 62 matching lines @@
       rebase_path(_result_path, root_build_dir),
       "--enable",
     ]
 
     if (defined(invoker.create_cache) && invoker.create_cache) {
       args += [
         "--create-cache",
         "--silent",
       ]
     } else {
-      inputs += [ invoker.jar_path ] + invoker.java_files
+      _rebased_build_config = rebase_path(invoker.build_config, root_build_dir)

[jbudorick, 2016/04/05 13:11:20]

nit: move this down by _rebased_java_files

[agrieve, 2016/04/05 13:40:02]

Done.

+      inputs += invoker.java_files
+      inputs += [
+        invoker.jar_path,
+        invoker.build_config,
+      ]
       deps += [ "//build/android:prepare_android_lint_cache" ]
       _rebased_java_files = rebase_path(invoker.java_files, root_build_dir)
       args += [
         "--jar-path",
         rebase_path(invoker.jar_path, root_build_dir),
         "--java-files=$_rebased_java_files",
+        "--classpath=@FileArg($_rebased_build_config:javac:interface_classpath)",

[jbudorick, 2016/04/05 13:11:20]

Should there be an equivalent change to //build/android/lint_action.gypi?

[agrieve, 2016/04/05 13:40:02]

If we cared about GYP, yes. It doesn't break without it though.

       ]
     }
   }
 }
 
 template("proguard") {
   action(target_name) {
     set_sources_assignment_filter([])
     forward_variables_from(invoker,
                            [
@@ skipping 41 matching lines @@
     if (proguard_verbose) {
       args += [ "--verbose" ]
     }
     if (defined(invoker.args)) {
       args += invoker.args
     }
   }
 }
 
 template("findbugs") {
-  jar_path = invoker.jar_path
-
-  build_config = invoker.build_config
-
   action(target_name) {
     forward_variables_from(invoker,
                            [
                              "deps",
                              "testonly",
                            ])
     script = "//build/android/findbugs_diff.py"
     depfile = "$target_gen_dir/$target_name.d"
-    result_path = "$target_gen_dir/$target_name/result.xml"
-    exclusions_file = "//build/android/findbugs_filter/findbugs_exclude.xml"
+    _result_path = "$target_gen_dir/$target_name/result.xml"
+    _exclusions_file = "//build/android/findbugs_filter/findbugs_exclude.xml"
 
-    rebased_build_config = rebase_path(build_config, root_build_dir)
+    _rebased_build_config = rebase_path(invoker.build_config, root_build_dir)
 
     inputs = [
       "//build/android/pylib/utils/findbugs.py",
-      exclusions_file,
-      jar_path,
+      _exclusions_file,
+      invoker.jar_path,
+      invoker.build_config,
     ]
 
     outputs = [
       depfile,
-      result_path,
+      _result_path,
     ]
 
     args = [
       "--depfile",
       rebase_path(depfile, root_build_dir),
       "--exclude",
-      rebase_path(exclusions_file, root_build_dir),
+      rebase_path(_exclusions_file, root_build_dir),
       "--auxclasspath-gyp",
-      "@FileArg($rebased_build_config:javac:classpath)",
+      "@FileArg($_rebased_build_config:javac:classpath)",
       "--output-file",
-      rebase_path(result_path, root_build_dir),
-      rebase_path(jar_path, root_build_dir),
+      rebase_path(_result_path, root_build_dir),
+      rebase_path(invoker.jar_path, root_build_dir),
     ]
 
     if (findbugs_verbose) {
       args += [ "-vv" ]
     }
   }
 }
 
 # Generates a script in the output bin.java directory to run a java binary.
 #
@@ skipping 1525 matching lines @@
       }
       deps = build_config_deps
     }
   }
 
   if (_supports_android) {
     if (_chromium_code) {
       _final_datadeps += [ ":${_template_name}__lint" ]
       android_lint("${_template_name}__lint") {
         android_manifest = _android_manifest
+        build_config = _build_config
         jar_path = _jar_path
         java_files = _java_files
-        deps = [
-          ":$_compile_java_target",
-        ]
+        deps = build_config_deps + [ ":$_compile_java_target" ]
         if (defined(invoker.deps)) {
           deps += invoker.deps
         }
       }
 
       if (_run_findbugs) {
         _final_datadeps += [ ":${_template_name}__findbugs" ]
         findbugs("${_template_name}__findbugs") {
           build_config = _build_config
           jar_path = _jar_path
-          deps = [
-            ":$_compile_java_target",
-          ]
+          deps = build_config_deps + [ ":$_compile_java_target" ]
         }
       }
     }
 
     _final_deps += [ ":${_template_name}__dex" ]
     dex("${_template_name}__dex") {
       sources = [
         _jar_path,
       ]
       output = _dex_path
@@ skipping 412 matching lines @@
     ]
     args = [
       "--depfile",
       rebase_path(depfile, root_build_dir),
       "--script-output-path",
       rebase_path(generated_script, root_build_dir),
     ]
     args += test_runner_args
   }
 }