Simplify Go's Mojom Type Generator

mojo/public/tools/bindings/mojom_bindings_generator_v1.py

Index: mojo/public/tools/bindings/mojom_bindings_generator_v1.py
diff --git a/mojo/public/tools/bindings/mojom_bindings_generator_v1.py b/mojo/public/tools/bindings/mojom_bindings_generator_v1.py
index 6b455b3d54496401635272a4efd70f7a465a2a65..92b5f6700923b3b6c0d8493312e5af8af2201b4d 100755
--- a/mojo/public/tools/bindings/mojom_bindings_generator_v1.py
+++ b/mojo/public/tools/bindings/mojom_bindings_generator_v1.py
@@ -206,6 +206,11 @@ def main(argv):
                       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)