Revert of 🍵 Refactor the various locale_paks() templates to be more shared

chrome/chrome_repack_locales.gni

Index: chrome/chrome_repack_locales.gni
diff --git a/chrome/chrome_repack_locales.gni b/chrome/chrome_repack_locales.gni
index a864354ad06cebc8f3af0ef1925dd740d5544602..f4a3cafb8d2affd644abf907c5f74491e6dfbacf 100644
--- a/chrome/chrome_repack_locales.gni
+++ b/chrome/chrome_repack_locales.gni
@@ -7,29 +7,44 @@
 import("//build/config/ui.gni")
 import("//tools/grit/repack.gni")
 
-# Wraps repack_locales(), setting the source_patterns and deps required for
-# Chrome.
-#
-# Extra Parameters:
-#
-#   additional_source_patterns [optional]
-#      Extra source_patterns for repack_locales().
-template("chrome_repack_locales") {
-  repack_locales(target_name) {
-    forward_variables_from(invoker, "*", [ "additional_source_patterns" ])
-
-    # Each input pak file should also have a deps line.
-    source_patterns = [
-      "${root_gen_dir}/chrome/generated_resources_",
-      "${root_gen_dir}/chrome/locale_settings_",
-      "${root_gen_dir}/chrome/platform_locale_settings_",
-      "${root_gen_dir}/components/strings/components_locale_settings_",
-      "${root_gen_dir}/components/strings/components_strings_",
+# Arguments:
+#
+#   locale
+#       Internal name of locale. e.g. "pt-BR"
+#
+#   output
+#       Output file name.
+#
+#   visibility
+#       Normal meaning.
+#
+#   additional_source_patterns
+#      The pattern of additional pak files which need repacked, the name of
+#      pak always ends with '_${locale}.pak' e.g. foo_zh-CN.pak, since the
+#      locale is argument of template, the sources in this list should be
+#      the file name without '_${locale}.pak', tempate expands sources to
+#      the full name, e.g
+#         ${root_gen_dir}/foo expands to ${root_gen_dir}/foo_zh-CN.pak
+#       when locale is zh-CN.
+#
+#   additional_deps
+#      The corresponding deps of additonal_source_patterns.
+#
+template("_repack_one_locale") {
+  locale = invoker.locale
+
+  repack(target_name) {
+    visibility = invoker.visibility
+
+    # Each input pak file should also have a deps line for completeness.
+    sources = [
+      "${root_gen_dir}/chrome/generated_resources_${locale}.pak",
+      "${root_gen_dir}/chrome/locale_settings_${locale}.pak",
+      "${root_gen_dir}/chrome/platform_locale_settings_${locale}.pak",
+      "${root_gen_dir}/components/strings/components_locale_settings_${locale}.pak",
+      "${root_gen_dir}/components/strings/components_strings_${locale}.pak",
     ]
-    if (!defined(deps)) {
-      deps = []
-    }
-    deps += [
+    deps = [
       "//chrome/app:generated_resources",
       "//chrome/app/resources:locale_settings",
       "//chrome/app/resources:platform_locale_settings",
@@ -37,14 +52,21 @@
       "//components/strings:components_strings",
     ]
 
+    if (defined(invoker.additional_source_patterns)) {
+      sources += process_file_template(invoker.additional_source_patterns,
+                                       [ "{{source}}_${locale}.pak" ])
+      deps += invoker.additional_deps
+    }
+
     if (use_ash) {
-      source_patterns += [ "${root_gen_dir}/ash/common/strings/ash_strings_" ]
+      sources +=
+          [ "${root_gen_dir}/ash/common/strings/ash_strings_${locale}.pak" ]
       deps += [ "//ash/common/strings" ]
     }
     if (is_chromeos) {
-      source_patterns += [
-        "${root_gen_dir}/remoting/resources/",
-        "${root_gen_dir}/ui/chromeos/strings/ui_chromeos_strings_",
+      sources += [
+        "${root_gen_dir}/remoting/resources/${locale}.pak",
+        "${root_gen_dir}/ui/chromeos/strings/ui_chromeos_strings_${locale}.pak",
       ]
       deps += [
         "//remoting/resources",
@@ -52,12 +74,12 @@
       ]
     }
     if (!is_ios) {
-      source_patterns += [
-        "${root_gen_dir}/content/app/strings/content_strings_",
-        "${root_gen_dir}/device/bluetooth/strings/bluetooth_strings_",
-        "${root_gen_dir}/third_party/libaddressinput/address_input_strings_",
-        "${root_gen_dir}/ui/strings/app_locale_settings_",
-        "${root_gen_dir}/ui/strings/ui_strings_",
+      sources += [
+        "${root_gen_dir}/content/app/strings/content_strings_${locale}.pak",
+        "${root_gen_dir}/device/bluetooth/strings/bluetooth_strings_${locale}.pak",
+        "${root_gen_dir}/third_party/libaddressinput/address_input_strings_${locale}.pak",
+        "${root_gen_dir}/ui/strings/app_locale_settings_${locale}.pak",
+        "${root_gen_dir}/ui/strings/ui_strings_${locale}.pak",
       ]
       deps += [
         "//content/app/strings",
@@ -68,27 +90,27 @@
       ]
     }
     if (enable_extensions) {
-      source_patterns += [
+      sources += [
         # TODO(jamescook): When Android stops building extensions code move
         # this to the OS != 'ios' and OS != 'android' section.
-        "${root_gen_dir}/extensions/strings/extensions_strings_",
+        "${root_gen_dir}/extensions/strings/extensions_strings_${locale}.pak",
       ]
       deps += [ "//extensions/strings" ]
     }
 
     if (is_chrome_branded) {
-      source_patterns += [
-        "${root_gen_dir}/chrome/google_chrome_strings_",
-        "${root_gen_dir}/components/strings/components_google_chrome_strings_",
+      sources += [
+        "${root_gen_dir}/chrome/google_chrome_strings_${locale}.pak",
+        "${root_gen_dir}/components/strings/components_google_chrome_strings_${locale}.pak",
       ]
       deps += [
         "//chrome/app:google_chrome_strings",
         "//components/strings:components_google_chrome_strings",
       ]
     } else {
-      source_patterns += [
-        "${root_gen_dir}/chrome/chromium_strings_",
-        "${root_gen_dir}/components/strings/components_chromium_strings_",
+      sources += [
+        "${root_gen_dir}/chrome/chromium_strings_${locale}.pak",
+        "${root_gen_dir}/components/strings/components_chromium_strings_${locale}.pak",
       ]
       deps += [
         "//chrome/app:chromium_strings",
@@ -96,16 +118,93 @@
       ]
     }
 
-    if (defined(invoker.additional_source_patterns)) {
-      source_patterns += invoker.additional_source_patterns
-    }
-
-    if (!defined(output_dir)) {
-      if (is_mac || is_ios) {
-        output_dir = "${root_gen_dir}/repack/locales"
-      } else {
-        output_dir = "${root_out_dir}/locales"
-      }
-    }
+    if (enable_resource_whitelist_generation) {
+      repack_whitelist = "$root_gen_dir/chrome/resource_whitelist.txt"
+      deps += [ "//chrome:resource_whitelist" ]
+    }
+    output = invoker.output
   }
 }
+
+# Creates an action to call the repack_locales script.
+#
+# The GYP version generates the locales in the "gen" directory and then copies
+# it to the root build directory. This isn't easy to express in a GN copy
+# rule since the files on Mac have a complex structure. So we generate the
+# files into the final place and skip the "gen" directory.
+#
+# This template uses GN's looping constructs to avoid the complex call to
+# chrome/tools/build/repack_locales.py which wraps the repack commands in the
+# GYP build.
+#
+# Arguments
+#
+#   input_locales
+#       List of locale names to use as inputs.
+#
+#   output_locales
+#       A list containing the corresponding output names for each of the
+#       input names. Mac uses different names in some cases.
+#
+#   visibility
+#
+#   additional_source_patterns
+#      Pass to template("_repack_one_locale")
+#
+#   additional_deps
+#      Pass to template("_repack_one_locale")
+#
+#   output_dir
+#      this is used to override default output dir.
+#
+template("chrome_repack_locales") {
+  # This is the name of the group below that will collect all the invidual
+  # locale targets. External targets will depend on this.
+  group_target_name = target_name
+
+  # GN's subscript is too stupid to do invoker.output_locales[foo] so we need
+  # to make a copy and do output_locales[foo].
+  output_locales = invoker.output_locales
+
+  # Collects all targets the loop generates.
+  locale_targets = []
+
+  # This loop iterates over the input locales and also keeps a counter so it
+  # can simultaneously iterate over the output locales (using GN's very
+  # limited looping capabilities).
+  current_index = 0
+  foreach(input_locale, invoker.input_locales) {
+    output_locale = output_locales[current_index]
+
+    # Compute the name of the target for the current file. Save it for the deps.
+    current_name = "${target_name}_${input_locale}"
+    locale_targets += [ ":$current_name" ]
+
+    _repack_one_locale(current_name) {
+      visibility = [ ":$group_target_name" ]
+      locale = input_locale
+
+      # Compute the output name. Mac uses a different location.
+      if (defined(invoker.output_dir)) {
+        output = "${invoker.output_dir}/${output_locale}.pak"
+      } else if (is_mac || is_ios) {
+        output = "${root_gen_dir}/repack/locales/${output_locale}.pak"
+      } else {
+        output = "${root_out_dir}/locales/${output_locale}.pak"
+      }
+
+      if (defined(invoker.additional_source_patterns)) {
+        additional_source_patterns = invoker.additional_source_patterns
+        additional_deps = invoker.additional_deps
+      }
+    }
+
+    current_index = current_index + 1
+  }
+
+  # The group that external targets depend on which collects all deps.
+  group(group_target_name) {
+    forward_variables_from(invoker, [ "visibility" ])
+    public_deps = locale_targets
+  }
+}