[bindings] Pass NormalConversion to to[U]Int{8/16/32/64}(...) in v8_types.py and reduce conversion …

Source/bindings/scripts/v8_types.py

 # Copyright (C) 2013 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 # copyright notice, this list of conditions and the following disclaimer
@@ skipping 528 matching lines @@
     # Simple types
     idl_type = idl_type.preprocessed_type
     add_includes_for_type(idl_type)
     base_idl_type = idl_type.as_union_type.name if idl_type.is_union_type else idl_type.base_type
 
     if 'EnforceRange' in extended_attributes:
         arguments = ', '.join([v8_value, 'EnforceRange', 'exceptionState'])
     elif 'Clamp' in extended_attributes:
         arguments = ', '.join([v8_value, 'Clamp', 'exceptionState'])
     elif idl_type.v8_conversion_needs_exception_state:
-        arguments = ', '.join([v8_value, 'exceptionState'])
+        if idl_type.is_integer_type:

[Jens Widell, 2015/03/17 11:28:22]

The preceding two cases, EnforceRange and Clamp, also affect only integer types.
I think this code would be slightly better restructured as

  if idl_type.is_integer_type:
    configuration = < 'EnforceRange' or 'Clamp' or 'NormalConversion' >
    arguments = ', '.join([v8_value, configuration, 'exceptionState'])
  elif idl_type.v8_conversion_needs_exception_state:
    ...
  else:
    ...

+            arguments = ', '.join([v8_value, 'NormalConversion', 'exceptionState'])
+        else:
+            arguments = ', '.join([v8_value, 'exceptionState'])
     else:
         arguments = v8_value
     if base_idl_type in V8_VALUE_TO_CPP_VALUE:
         cpp_expression_format = V8_VALUE_TO_CPP_VALUE[base_idl_type]
     elif idl_type.is_array_buffer_or_view:
         cpp_expression_format = (
             '{v8_value}->Is{idl_type}() ? '
             'V8{idl_type}::toImpl(v8::Local<v8::{idl_type}>::Cast({v8_value})) : 0')
     elif idl_type.use_output_parameter_for_result:
         if idl_type.includes_nullable_type:
@@ skipping 406 matching lines @@
     number_of_nullable_member_types_union)
 
 
 def includes_nullable_type_union(idl_type):
     # http://heycam.github.io/webidl/#dfn-includes-a-nullable-type
     return idl_type.number_of_nullable_member_types == 1
 
 IdlTypeBase.includes_nullable_type = False
 IdlNullableType.includes_nullable_type = True
 IdlUnionType.includes_nullable_type = property(includes_nullable_type_union)