Make Google API keys available for the Java codebase

chrome/android/BUILD.gn

Index: chrome/android/BUILD.gn
diff --git a/chrome/android/BUILD.gn b/chrome/android/BUILD.gn
index fcf8a1f4e374727cc737d576d556c9221fa305ec..2a56e8fe00c7259e0f93c2ef25f4eaab3e2b9404 100644
--- a/chrome/android/BUILD.gn
+++ b/chrome/android/BUILD.gn
@@ -162,6 +162,7 @@ android_library("chrome_java") {
 
   srcjar_deps = [
     ":chrome_android_java_enums_srcjar",
+    ":chrome_android_java_google_api_keys_srcjar",
     ":chrome_version_srcjar",
     ":custom_tabs_service_aidl",
     ":resource_id_javagen",
@@ -182,6 +183,31 @@ android_library("chrome_java") {
 # GYP: //chrome/chrome_browser.gypi:connection_security_levels_java
 # GYP: //chrome/chrome_browser.gypi:tab_load_status_java
 # GYP: //chrome/chrome_browser.gypi:infobar_action_type_java
+
+action("google_api_keys_java_file") {
+  visibility = [ ":chrome_android_java_google_api_keys_srcjar" ]
+  script = "//build/android/gyp/java_google_api_keys.py"
+  _gen_dir = "${target_gen_dir}/chrome_android_java_google_api_keys_srcjar/java_google_api_keys"
+  _output_path = rebase_path("GoogleAPIKeys.java", ".", _gen_dir)
+  _output_file = get_path_info(_output_path, "abspath")
+  outputs = [
+    _output_file,
+  ]
+  args = [
+    "--out",
+    _output_path,
+  ]
+}
+
+zip("chrome_android_java_google_api_keys_srcjar") {

[agrieve, 2015/11/03 16:22:42]

yep, this looks good :)

One more thing you should do to simplify though, is to fold zip() into
java_google_api_keys.py.

Can you add a --srcjar arg that works similarly to
https://code.google.com/p/chromium/codesearch#chromium/src/build/android/gyp/...

You should be able to avoid writing intermediate files entirely by using
zipfile.writestr().

You should also use HERMETIC_TIMESTAMP from here:
https://code.google.com/p/chromium/codesearch#chromium/src/build/android/gyp/...

(and also HERMETIC_FILE_ATTR) so that identical .srcjar files will be created
given identical inputs.

The other thing this will fix is that the path should include the java package
name in it, or else compilation with enable_incremental_javac = true will fail.
This doesn't exist for GYP, so just leave as-is for GYP.

Note: I see that java_cpp_enum() doesn't follow my advice here, but I will fix
that up :)

[dvh, 2015/11/03 20:03:54]

Done.

+  inputs = get_target_outputs(":google_api_keys_java_file")
+  output = "${target_gen_dir}/${target_name}.srcjar"
+  base_dir = get_label_info(":google_api_keys_java_file", "target_gen_dir")
+  deps = [
+    ":google_api_keys_java_file",
+  ]
+}
+
 java_cpp_enum("chrome_android_java_enums_srcjar") {
   sources = [
     "//chrome/browser/android/activity_type_ids.h",