Properly set up classpath for Android Lint (and fix its stderr filter)

build/config/android/internal_rules.gni

Index: build/config/android/internal_rules.gni
diff --git a/build/config/android/internal_rules.gni b/build/config/android/internal_rules.gni
index b94134c07cddeeff62877b278a089571b968db96..140bd4854f95e2a6af309beb7354259f139a070a 100644
--- a/build/config/android/internal_rules.gni
+++ b/build/config/android/internal_rules.gni
@@ -60,6 +60,7 @@ template("android_lint") {
       rebase_path(_cache_dir, root_build_dir),
       "--platform-xml-path",
       rebase_path(_platform_xml_path, root_build_dir),
+      "--android-sdk-version=${android_sdk_version}",
       "--depfile",
       rebase_path(depfile, root_build_dir),
       "--config-path",
@@ -92,8 +93,7 @@ template("android_lint") {
         "--jar-path",
         rebase_path(invoker.jar_path, root_build_dir),
         "--java-files=$_rebased_java_files",
-        "--classpath=['$rebased_android_sdk_jar']",

[agrieve, 2016/04/08 13:40:29]

do we not need to pass android.jar at all now? I think the .gyp version will
still have it.

[mlopatkin, 2016/04/08 16:55:16]

That's the point of the CL. We should not pass android.jar in --libraries; we
specify target in project.properties instead and then Lint itself find
android.jar in the SDK and adds it to the classpath. So Lint doesn't run
class-level checks on android.jar.

-        "--classpath=@FileArg($_rebased_build_config:javac:interface_classpath)",
+        "--classpath=@FileArg($_rebased_build_config:javac:classpath)",
       ]
     }
   }