IDL: Don't lose nullable flag when "preprocessing" type

Source/bindings/scripts/v8_types.py

Index: Source/bindings/scripts/v8_types.py
diff --git a/Source/bindings/scripts/v8_types.py b/Source/bindings/scripts/v8_types.py
index c87b5101cfda7dc1ca4c40d4907d3f025eb221ef..8759fb7c948963e4ba0e45a2b0a9b2c0f1b3329a 100644
--- a/Source/bindings/scripts/v8_types.py
+++ b/Source/bindings/scripts/v8_types.py
@@ -671,10 +671,12 @@ IdlTypeBase.use_output_parameter_for_result = property(use_output_parameter_for_
 ################################################################################
 
 def preprocess_idl_type(idl_type):
+    if idl_type.is_nullable:
+        return IdlNullableType(idl_type.inner_type.preprocessed_type)
     if idl_type.is_enum:
         # Enumerations are internally DOMStrings
         return IdlType('DOMString')
-    if (idl_type.name in ['Any', 'Object'] or idl_type.is_callback_function):

[Jens Widell, 2015/02/16 14:33:54]

Looking at idl_type.name here was wrong; it would be AnyOrNull/ObjectOrNull for
a nullable type (and was in the case of
WebGLRenderingContextBase.getExtension()'s "object?" return value.)

With the other change in this function this code would now have worked, but I
changed it anyway, since I think it's better to look at base_type in most cases;
using name is equivalent except it invites this kind of error.

+    if idl_type.base_type in ['any', 'object'] or idl_type.is_callback_function:
         return IdlType('ScriptValue')
     return idl_type