Simplify Go's Mojom Type Generator

mojo/public/tools/bindings/mojom_bindings_generator_v1.py

 #!/usr/bin/env python
 # Copyright 2013 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.
 
 """The frontend for the Mojo bindings system."""
 
 
 import argparse
 import imp
@@ skipping 188 matching lines @@
                       help="comma-separated list of generators")
   parser.add_argument("--debug_print_intermediate", action="store_true",
                       help="print the intermediate representation")
   parser.add_argument("-I", dest="import_directories", action="append",
                       metavar="directory", default=[],
                       help="add a directory to be searched for import files")
   parser.add_argument("--use_bundled_pylibs", action="store_true",
                       help="use Python modules bundled in the SDK")
   (args, remaining_args) = parser.parse_known_args(argv)
 
+  # These flags are added to ensure that tests requiring type information pass.
+  # TODO(afandria): The caller of this script should be the one passing these
+  # flags in. https://github.com/domokit/mojo/issues/558

[rudominer, 2015/12/14 21:29:57]

The caller of this script is mojom_bindings_generator.py. Can you not put the
this logic there? We want this flag for mojom_bindings_generator_v2 also because
we want the tests to continue passing when use_new_mojom_compiler=true.

+  remaining_args.append("--go_gen_types")
+
   generator_modules = LoadGenerators(args.generators_string)
 
   fileutil.EnsureDirectoryExists(args.output_dir)
 
   processor = MojomProcessor(lambda filename: filename in args.filename)
   for filename in args.filename:
     processor.ProcessFile(args, remaining_args, generator_modules, filename)
 
   return 0
 
 
 if __name__ == "__main__":
   sys.exit(main(sys.argv[1:]))