Improve handling of failed integer type conversions.

Source/bindings/scripts/unstable/v8_types.py

Index: Source/bindings/scripts/unstable/v8_types.py
diff --git a/Source/bindings/scripts/unstable/v8_types.py b/Source/bindings/scripts/unstable/v8_types.py
index 131b804ce9e7c792218028e3314687d19c046e8d..c3035360e66715c4a79682c6b529807c57bc0e36 100644
--- a/Source/bindings/scripts/unstable/v8_types.py
+++ b/Source/bindings/scripts/unstable/v8_types.py
@@ -142,6 +142,9 @@ def is_union_type(idl_type):
     return isinstance(idl_type, idl_definitions.IdlUnionType)
 
 
+def is_integer_type(idl_type):
+    return any(idl_type in s for s in [CPP_INT_TYPES, CPP_UNSIGNED_TYPES, CPP_LONG_LONG_TYPES])

[Nils Barth (inactive), 2014/01/06 02:58:09]

Simpler:
return idl_type in CPP_INT_TYPES | CPP_UNSIGNED_TYPES | CPP_LONG_LONG_TYPES

Since this is IDL (independent of C++), could we instead split off INTEGER_TYPES
from BASIC_TYPES, as:
BASIC_TYPES = set([
    # Built-in, non-composite, non-object data types
    # (integer types are listed separately below)
    # http://www.w3.org/TR/WebIDL/#dfn-primitive-type
    'boolean',
    'float',
    # unrestricted float is not supported
    'double',
    # unrestricted double is not supported
    # http://www.w3.org/TR/WebIDL/#idl-types
    'DOMString',
    'Date',
    # http://www.w3.org/TR/WebIDL/#es-type-mapping
    'void',
])
INTEGER_TYPES = set([
    # http://www.w3.org/TR/WebIDL/#dfn-integer-type
    'byte',
    'octet',
    'short',
    'unsigned short',
    # int and unsigned are not IDL types
    'long',
    'unsigned long',
    'long long',
    'unsigned long long',
])
BASIC_TYPES.update(INTEGER_TYPES)

...and then we have these two functions:
def is_basic_type(idl_type):
    return idl_type in BASIC_TYPES

def is_integer_type(idl_type):
    return idl_type in INTEGER_TYPES

[sof, 2014/01/06 21:01:25]

Adopted wholesale; the transformation added here is over IDL types, not over
what it maps to in C++.

+
 ################################################################################
 # V8-specific type handling
 ################################################################################
@@ -237,6 +240,10 @@ CPP_SPECIAL_CONVERSION_RULES = {
     'ScriptValue': 'ScriptValue',
     'boolean': 'bool',
 }
+CPP_LONG_LONG_TYPES = set([

[Nils Barth (inactive), 2014/01/06 02:58:09]

Could you put this set up *above* CPP_SPECIAL_CONVERSION_RULES?
(So the 3 sets of types are together.)

Actually, we don't need this if we instead use INTEGER_TYPES (above),
as this isn't C++-specific.

[sof, 2014/01/06 21:01:25]

Removed.

+    'long long',
+    'unsigned long long',
+])
 
 def cpp_type(idl_type, extended_attributes=None, used_as_argument=False):
     """Returns C++ type corresponding to IDL type."""
@@ -377,8 +384,10 @@ def v8_value_to_cpp_value(idl_type, extended_attributes, v8_value, index):
     add_includes_for_type(idl_type)
 
     if 'EnforceRange' in extended_attributes:
-        arguments = ', '.join([v8_value, 'EnforceRange', 'ok'])
-    else:  # NormalConversion
+        arguments = ', '.join([v8_value, 'EnforceRange', 'exceptionState'])
+    elif is_integer_type(idl_type):  # NormalConversion

[Nils Barth (inactive), 2014/01/06 02:58:09]

Comment '# NormalConversion' should go on the default 'else:' line.

[sof, 2014/01/06 21:01:25]

Isn't this the correct placement for it? i.e., this is for NormalConversion over
integer types.

[Nils Barth (inactive), 2014/01/07 02:11:35]

OIC, you're right - sorry, thanks!
(NormalConversion as opposed to EnforceRange for integer types;
 for non-integer types this doesn't make sense.)

+        arguments = ', '.join([v8_value, 'exceptionState'])
+    else:
         arguments = v8_value
 
     if idl_type in V8_VALUE_TO_CPP_VALUE:
@@ -421,8 +430,8 @@ def v8_value_to_local_cpp_value(idl_type, extended_attributes, v8_value, variabl
     idl_type = preprocess_idl_type(idl_type)
     if idl_type == 'DOMString':
         format_string = 'V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID({cpp_type}, {variable_name}, {cpp_value})'
-    elif 'EnforceRange' in extended_attributes:
-        format_string = 'V8TRYCATCH_WITH_TYPECHECK_VOID({cpp_type}, {variable_name}, {cpp_value}, info.GetIsolate())'
+    elif is_integer_type(idl_type):
+        format_string = 'V8TRYCATCH_EXCEPTION_VOID({cpp_type}, {variable_name}, {cpp_value}, exceptionState)'
     else:
         format_string = 'V8TRYCATCH_VOID({cpp_type}, {variable_name}, {cpp_value})'