Mojo JS bindings: fix enum array validation.

mojo/public/tools/bindings/generators/mojom_js_generator.py

 # 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.
 
 """Generates JavaScript source files from a mojom.Module."""
 
 import mojom.generate.generator as generator
 import mojom.generate.module as mojom
 import mojom.generate.pack as pack
 from mojom.generate.template_expander import UseJinja
@@ skipping 216 matching lines @@
   element_size = pack.PackedField.GetSizeForKind(element_kind)
   expected_dimension_sizes = GetArrayExpectedDimensionSizes(
       field.kind)
   element_type = ElementCodecType(element_kind)
   return "%s, %s, %s, %s, 0" % \
       (element_size, element_type, nullable,
        expected_dimension_sizes)
 
 
 def JavaScriptValidateEnumParams(field):
-  nullable = JavaScriptNullableParam(field)
   enum_type = JavaScriptType(field.kind)
-  return "%s, %s" % (enum_type, nullable)
+  return enum_type

[Jay Civelli, 2016/12/21 23:14:52]

Nit: is the local var needed?

[yzshen1, 2016/12/21 23:17:36]

Done.

 
 def JavaScriptValidateStructParams(field):
   nullable = JavaScriptNullableParam(field)
   struct_type = JavaScriptType(field.kind)
   return "%s, %s" % (struct_type, nullable)
 
 def JavaScriptValidateUnionParams(field):
   nullable = JavaScriptNullableParam(field)
   union_type = JavaScriptType(field.kind)
   return "%s, %s" % (union_type, nullable)
@@ skipping 162 matching lines @@
     return self.module.imports
 
   def GetImportedInterfaces(self):
     interface_to_import = {};
     for each_import in self.module.imports:
       for each_interface in each_import["module"].interfaces:
         name = each_interface.name
         interface_to_import[name] = each_import["unique_name"] + "." + name
     return interface_to_import;