Reland of Deduplicate Monochrome locale .paks

chrome/chrome_paks.gni

 # Copyright 2016 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/locales.gni")
 import("//chrome/common/features.gni")
 import("//extensions/features/features.gni")
 import("//ui/base/ui_features.gni")
 import("chrome_repack_locales.gni")
 
@@ skipping 166 matching lines @@
 # Defines repack() targets used by Chrome. Specifically:
 # * chrome_100_percent.pak
 # * chrome_200_percent.pak (optionally)
 # * resources.pak
 # * locale .pak files
 #
 # Paramters:
 #   output_dir [required]: Directory to output .pak files. Locale .pak files
 #       will always be place in $output_dir/locales
 #   additional_extra_paks: List of extra .pak sources for resources.pak.
-#   exclude_locale_paks: if set to true, skip chrome_repack_locales.
+#   locale_whitelist: if set, override repack_whitelist for locale .pak files.
 #   copy_data_to_bundle:
 #   deps:
 #   output_dir:
 #   public_deps:
 #   repack_whitelist:
 #   visibility:
 #     Normal meanings.
 #
 template("chrome_paks") {
   chrome_repack_percent("${target_name}_100_percent") {
@@ skipping 29 matching lines @@
                              "deps",
                              "output_dir",
                              "repack_whitelist",
                              "visibility",
                            ])
     if (defined(invoker.additional_extra_paks)) {
       additional_paks = invoker.additional_extra_paks
     }
   }
 
-  if (!defined(invoker.exclude_locale_paks) || !invoker.exclude_locale_paks) {
-    chrome_repack_locales("${target_name}_locales") {
-      forward_variables_from(invoker,
-                             [
-                               "copy_data_to_bundle",
-                               "deps",
-                               "repack_whitelist",
-                               "visibility",
-                             ])
+  chrome_repack_locales("${target_name}_locales") {
+    forward_variables_from(invoker,
+                           [
+                             "copy_data_to_bundle",
+                             "deps",
+                             "visibility",
+                           ])
+    if (defined(invoker.locale_whitelist)) {
+      repack_whitelist = invoker.locale_whitelist
+    } else if (defined(invoker.repack_whitelist)) {
+      repack_whitelist = invoker.repack_whitelist
+    }
 
-      input_locales = locales
-      output_dir = "${invoker.output_dir}/locales"
+    input_locales = locales
+    output_dir = "${invoker.output_dir}/locales"
 
-      if (is_mac) {
-        output_locales = locales_as_mac_outputs
-      } else {
-        output_locales = locales
-      }
+    if (is_mac) {
+      output_locales = locales_as_mac_outputs
+    } else {
+      output_locales = locales
     }
   }
 
   group(target_name) {
     forward_variables_from(invoker, [ "deps" ])
     public_deps = [
       ":${target_name}_100_percent",
       ":${target_name}_extra",
+      ":${target_name}_locales",
     ]
-    if (!defined(invoker.exclude_locale_paks) || !invoker.exclude_locale_paks) {
-      public_deps += [ ":${target_name}_locales" ]
-    }
     if (enable_hidpi) {
       public_deps += [ ":${target_name}_200_percent" ]
     }
     if (defined(invoker.public_deps)) {
       public_deps += invoker.public_deps
     }
   }
 }