[remoting android] Make Android credits a GN template.

remoting/android/BUILD.gn

 # Copyright 2015 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/android/rules.gni")
 import("//remoting/android/client_java_tmpl.gni")
+import("//remoting/android/credits_tmpl.gni")
 import("//remoting/android/remoting_apk_tmpl.gni")
 import("//remoting/remoting_options.gni")
 import("//remoting/tools/build/remoting_localize.gni")
 
 generate_jni("jni_headers") {
   sources = [
     "java/src/org/chromium/chromoting/jni/Client.java",
     "java/src/org/chromium/chromoting/jni/GlDisplay.java",
     "java/src/org/chromium/chromoting/jni/JniInterface.java",
     "java/src/org/chromium/chromoting/jni/TouchEventData.java",
@@ skipping 39 matching lines @@
   ]
   libs = [
     "android",
     "OpenSLES",
     "EGL",
   ]
   configs += [ "//remoting/build/config:enable_webrtc_remoting_client" ]
   assert_no_deps = [ "//third_party/ffmpeg:*" ]
 }
 
-action("credits") {
-  credits_html_file = "$target_gen_dir/credits.html"
-  script = "//tools/licenses.py"
-  depfile = "$target_gen_dir/$target_name.d"
-  credits_template = "//remoting/credits/credits.tmpl"
-  credits_entry_template = "//remoting/credits/credits_entry.tmpl"
-  inputs = [
-    credits_template,
-    credits_entry_template,
-  ]
-  outputs = [
-    credits_html_file,
-  ]
-  args = [
-    "credits",
-    rebase_path(credits_html_file, root_build_dir),
-    "--file-template",
-    rebase_path(credits_template, root_build_dir),
-    "--entry-template",
-    rebase_path(credits_entry_template, root_build_dir),
-    "--depfile",
-    rebase_path(depfile, root_build_dir),
-    "--gn-target",
-    "//remoting/android:remoting_apk",
-    "--gn-out-dir",
-    rebase_path(root_build_dir),
-  ]
-}
-
-_raw_resources_base_dir = "$target_gen_dir/remoting_android_raw_resources/res"
-copy("remoting_android_raw_resources") {
-  _credits_html = get_label_info("//remoting/android:credits",
-                                 "target_gen_dir") + "/credits.html"
-  sources = [
-    "//remoting/credits/credits_css.css",
-    "//remoting/credits/credits_js.js",
-    _credits_html,
-  ]
-  outputs = [
-    "$_raw_resources_base_dir/raw/{{source_file_part}}",
-  ]
-  deps = [
-    ":credits",
-  ]
+remoting_android_credits("credits") {
+  gn_target = "//remoting/android:remoting_apk"
 }
 
 remoting_localize("remoting_apk_manifest") {
   sources = [
     "java/AndroidManifest.xml.jinja2",
   ]
   locales = [ "en" ]
   variables = [ rebase_path(branding_path) ]
   output = "$root_gen_dir/remoting/android/{{source_name_part}}"
 }
 
 android_resources("remoting_android_client_java_resources") {
   custom_package = "org.chromium.chromoting"
   resource_dirs = [ "java/res" ]
-  generated_resource_dirs = [ _raw_resources_base_dir ]
-  generated_resource_files =
-      get_target_outputs(":remoting_android_raw_resources")
-
+  generated_resource_dirs =
+      [ get_label_info(":credits_resources_raw", "target_gen_dir") +
+        "/credits_resources_raw/res" ]
+  generated_resource_files = get_target_outputs(":credits_resources_raw")
   deps = [
-    ":remoting_android_raw_resources",
+    ":credits_resources_raw",

[agrieve, 2017/03/21 02:07:13]

It's a little bit bad form to depend on a generated target name from outside of
the template that defines it. Instead, I'd recommend:

template("remoting_android_credits") {
  action("${target_name}__generate") {
    ...
  }
  copy(target_name) {
    deps = [ ":${target_name}__generate" ]
  }
}

That said... I think it would be cleaner to define this android_resources()
target in the same place as the copy (since they both point to the same
directory & list of files). So, I'd either move the copy out, or move the
android_resources() in (and convert this target to a java_group() that just
depends on it, if you need to maintain it)

[Lambros, 2017/03/22 21:31:00]

Done (added __generate to the action target)

     "//remoting/resources:strings_java",
     "//third_party/android_tools:android_support_v7_appcompat_java",
   ]
 }
 
 remoting_android_client_java_tmpl("remoting_android_client_java") {
   remoting_google_play_services_library = google_play_services_library
 }
 
 remoting_apk_tmpl("remoting_apk") {
@@ skipping 30 matching lines @@
     "javatests/src/org/chromium/chromoting/TouchInputStrategyTest.java",
     "javatests/src/org/chromium/chromoting/test/util/MutableReference.java",
   ]
   deps = [
     ":remoting_android_client_java",
     "//base:base_java",
     "//base:base_java_test_support",
     "//third_party/android_support_test_runner:runner_java",
   ]
 }