Revert "Reland: chrome_repack_locales: compress .pak files on Chrome OS"

This reverts commit e5f79808.

Reason for revert: crbug.com/1023761

Original change's description:
> Reland: chrome_repack_locales: compress .pak files on Chrome OS
> 
> Chromite changes to deploy locales/*.pak.gz files has landed.
> 
> NOTE: This CL depends on the following chromite change (which copies
> .gz locale files to the Chrome OS image):
> https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1903536
> 
> Original CL:
> https://chromium-review.googlesource.com/c/chromium/src/+/1883292/
> 
> Original reviewer:
> TBR=agrieve@chromium.org
> For trivial change to resource_bundle.cc:
> TBR=sky@chromium.org
> 
> Bug: 1017864
> Change-Id: I384f1c2d0a2771e9121723ce8d10519f1ac1af9e
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906944
> Commit-Queue: Steven Bennetts <stevenjb@chromium.org>
> Reviewed-by: Scott Violet <sky@chromium.org>
> Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#714328}

TBR=stevenjb@chromium.org,sky@chromium.org,agrieve@chromium.org

Change-Id: Icbf01b3424729ed7502994c08e026a42e41e9f32
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1017864, 1023761
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911349Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714408}

chrome/chrome_repack_locales.gni

@@ -78,7 +78,6 @@ template("chrome_repack_locales") {
         "//remoting/resources",
         "//ui/chromeos/strings",
       ]
-      compress = true
     }
     if (enable_extensions) {
       source_patterns +=

tools/grit/grit/format/data_pack.py

@@ -215,8 +215,7 @@ def ReadGrdInfo(grd_file):
 
 
 def RePack(output_file, input_files, whitelist_file=None,
-           suppress_removed_key_output=False,
-           output_info_filepath=None):
+           suppress_removed_key_output=False):
   """Write a new data pack file by combining input pack files.
 
   Args:
@@ -227,7 +226,6 @@ def RePack(output_file, input_files, whitelist_file=None,
                       all resources.
       suppress_removed_key_output: allows the caller to suppress the output from
                                    RePackFromDataPackStrings.
-      output_info_file: If not None, specify the output .info filepath.
 
   Raises:
       KeyError: if there are duplicate keys or resource encoding is
@@ -245,9 +243,7 @@ def RePack(output_file, input_files, whitelist_file=None,
   resources, encoding = RePackFromDataPackStrings(
       inputs, whitelist, suppress_removed_key_output)
   WriteDataPack(resources, output_file, encoding)
-  if output_info_filepath is None:
-    output_info_filepath = output_file + '.info'
-  with open(output_info_filepath, 'w') as output_info_file:
+  with open(output_file + '.info', 'w') as output_info_file:
     for filename in input_info_files:
       with open(filename, 'r') as info_file:
         output_info_file.writelines(info_file.readlines())

tools/grit/pak_util.py

@@ -12,12 +12,9 @@ https://dev.chromium.org/developers/design-documents/linuxresourcesandlocalizeds
 from __future__ import print_function
 
 import argparse
-import gzip
 import hashlib
 import os
-import shutil
 import sys
-import tempfile
 
 # Import grit first to get local third_party modules.
 import grit  # pylint: disable=ungrouped-imports,unused-import
@@ -28,24 +25,8 @@ from grit.format import data_pack
 
 
 def _RepackMain(args):
-  output_info_filepath = args.output_pak_file + '.info'
-  if args.compress:
-    # If the file needs to be compressed, call RePack with a tempfile path,
-    # then compress the tempfile to args.output_pak_file.
-    temp_outfile = tempfile.NamedTemporaryFile()
-    out_path = temp_outfile.name
-    # Strip any non .pak extension from the .info output file path.
-    splitext = os.path.splitext(args.output_pak_file)
-    if splitext[1] != '.pak':
-      output_info_filepath = splitext[0] + '.info'
-  else:
-    out_path = args.output_pak_file
-  data_pack.RePack(out_path, args.input_pak_files, args.whitelist,
-                   args.suppress_removed_key_output,
-                   output_info_filepath=output_info_filepath)
-  if args.compress:
-    with gzip.open(args.output_pak_file, 'wb') as output_pak_file:
-      shutil.copyfileobj(temp_outfile, output_pak_file)
+  data_pack.RePack(args.output_pak_file, args.input_pak_files, args.whitelist,
+                   args.suppress_removed_key_output)
 
 
 def _ExtractMain(args):
@@ -159,8 +140,6 @@ def main():
       help='Path to a whitelist used to filter output pak file resource IDs.')
   sub_parser.add_argument('--suppress-removed-key-output', action='store_true',
       help='Do not log which keys were removed by the whitelist.')
-  sub_parser.add_argument('--compress', dest='compress', action='store_true',
-      default=False, help='Compress output_pak_file using gzip.')
   sub_parser.set_defaults(func=_RepackMain)
 
   sub_parser = sub_parsers.add_parser('extract', help='Extracts pak file')

tools/grit/repack.gni

@@ -20,9 +20,6 @@ import("//tools/grit/grit_rule.gni")
 #       Path of the file in the application bundle, defaults to
 #       {{bundle_resources_dir}}/{{source_file_part}}.
 #
-#   compress  [optional]
-#       Gzip the resulting bundle (and append .gz to the output name).
-#
 #   deps  [optional]
 #   public_deps  [optional]
 #   visibility  [optional]
@@ -35,8 +32,6 @@ template("repack") {
     _repack_target_name = "${target_name}__repack"
   }
 
-  _compress = defined(invoker.compress) && invoker.compress
-
   action(_repack_target_name) {
     forward_variables_from(invoker,
                            [
@@ -57,12 +52,8 @@ template("repack") {
     foreach(_invoker_source, invoker.sources) {
       inputs += [ "${_invoker_source}.info" ]
     }
-    _output_pak_file = invoker.output
-    if (_compress) {
-      _output_pak_file += ".gz"
-    }
     outputs = [
-      _output_pak_file,
+      invoker.output,
       "${invoker.output}.info",
     ]
 
@@ -73,11 +64,8 @@ template("repack") {
       args += [ "--whitelist=$_rebased_whitelist" ]
       args += [ "--suppress-removed-key-output" ]
     }
-    args += [ rebase_path(_output_pak_file, root_build_dir) ]
+    args += [ rebase_path(invoker.output, root_build_dir) ]
     args += rebase_path(invoker.sources, root_build_dir)
-    if (_compress) {
-      args += [ "--compress" ]
-    }
   }
 
   if (_copy_data_to_bundle) {
@@ -165,7 +153,6 @@ template("repack_locales") {
                              [
                                "copy_data_to_bundle",
                                "bundle_output",
-                               "compress",
                                "deps",
                                "repack_whitelist",
                                "testonly",

ui/base/resource/resource_bundle.cc

@@ -436,7 +436,7 @@ std::string ResourceBundle::LoadLocaleResources(
 
   std::unique_ptr<DataPack> data_pack(new DataPack(SCALE_FACTOR_100P));
   if (!data_pack->LoadFromPath(locale_file_path)) {
-    LOG(ERROR) << "failed to load locale file: " << locale_file_path;
+    LOG(ERROR) << "failed to load locale.pak";
     NOTREACHED();
     return std::string();
   }