[Chromoting] Pass list of locale files into build-webapp in a single file.

remoting/webapp/build-webapp.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """Creates a directory with with the unpacked contents of the remoting webapp.
 
 The directory will contain a copy-of or a link-to to all remoting webapp
 resources.  This includes HTML/JS and any plugin binaries. The script also
 massages resulting files appropriately with host plugin data. Finally,
@@ skipping 356 matching lines @@
 
 def main():
   if len(sys.argv) < 6:
     print ('Usage: build-webapp.py '
            '<build-type> <version> <dst> <zip-path> <manifest_template> '
            '<webapp_type> <other files...> '
            '--app_name <name> '
            '--app_description <description> '
            '--app_capabilities <capabilities...> '
            '[--appid <appid>] '
            '[--locales <locales...>] '

[Jamie, 2015/02/05 19:26:20]

Do we still need to support --locales? If not then I think we should delete it
as a command-line option. If it's still needed for CRD, then we should migrate
that to use the same mechanism, then delete it (but that can be a follow-up CL).

[garykac, 2015/02/06 02:13:35]

Done.

+           '[--locale_file <locale-file>] '

[Jamie, 2015/02/05 19:26:20]

s/locale/locales/g (it contains a list of all locales, right?)

[garykac, 2015/02/06 02:13:35]

Done.

            '[--jinja_paths <paths...>] '
            '[--service_environment <service_environment>]')
     return 1
 
   arg_type = ''
   files = []
   locales = []
+  locale_file = ''
   jinja_paths = []
   app_id = None
   app_name = None
   app_description = None
   app_capabilities = set([])
   service_environment = ''
 
   for arg in sys.argv[7:]:
     if arg in ['--locales',
+               '--locale_file',
                '--jinja_paths',
                '--appid',
                '--app_name',
                '--app_description',
                '--app_capabilities',
                '--service_environment']:
       arg_type = arg
     elif arg_type == '--locales':
       locales.append(arg)
+    elif arg_type == '--locale_file':
+      locale_file = arg
+      arg_type = ''
     elif arg_type == '--jinja_paths':
       jinja_paths.append(arg)
     elif arg_type == '--appid':
       app_id = arg
       arg_type = ''
     elif arg_type == '--app_name':
       app_name = arg
       arg_type = ''
     elif arg_type == '--app_description':
       app_description = arg
       arg_type = ''
     elif arg_type == '--app_capabilities':
       app_capabilities.add(arg)
     elif arg_type == '--service_environment':
       service_environment = arg
       arg_type = ''
     else:
       files.append(arg)
 
+  # If the locale files were specified in a listfile, then load them into
+  # the locales array.
+  if locale_file:
+    with open(locale_file) as input:
+      for s in input:
+        locales.append(s.rstrip())
+
   return buildWebApp(sys.argv[1], sys.argv[2], sys.argv[3], sys.argv[4],
                      sys.argv[5], sys.argv[6], app_id, app_name,
                      app_description, app_capabilities, files, locales,
                      jinja_paths, service_environment)
 
 
 if __name__ == '__main__':
   sys.exit(main())