IDL: Support argument default values

Source/bindings/tests/results/V8TestInterfaceNamedConstructor.cpp

Index: Source/bindings/tests/results/V8TestInterfaceNamedConstructor.cpp
diff --git a/Source/bindings/tests/results/V8TestInterfaceNamedConstructor.cpp b/Source/bindings/tests/results/V8TestInterfaceNamedConstructor.cpp
index 0557751dbdd5eb190f8143f58d53a75f9ba1faac..8e4ffa9330f5fc9b820fe52a848d3e85f23ebdd6 100644
--- a/Source/bindings/tests/results/V8TestInterfaceNamedConstructor.cpp
+++ b/Source/bindings/tests/results/V8TestInterfaceNamedConstructor.cpp
@@ -110,7 +110,11 @@ static void V8TestInterfaceNamedConstructorConstructorCallback(const v8::Functio
         TONATIVE_VOID_INTERNAL(defaultUndefinedOptionalBooleanArg, info[1]->BooleanValue());
         TONATIVE_VOID_EXCEPTIONSTATE_INTERNAL(defaultUndefinedOptionalLongArg, toInt32(info[2], exceptionState), exceptionState);
         TOSTRING_VOID_INTERNAL(defaultUndefinedOptionalStringArg, info[3]);
-        TOSTRING_VOID_INTERNAL(defaultNullStringOptionalstringArg, argumentOrNull(info, 4));

[Jens Widell, 2014/06/16 13:22:29]

The generated code changes like this everywhere "= null" is currently used on an
optional DOMString argument (all modified by
https://codereview.chromium.org/312683005/).

That's the only code generation change right now.

+        if (info.Length() > 4) {
+            TOSTRING_VOID_INTERNAL(defaultNullStringOptionalstringArg, info[4]);
+        } else {
+            defaultNullStringOptionalstringArg = nullptr;
+        }
         if (UNLIKELY(info.Length() <= 5)) {
             RefPtr<TestInterfaceNamedConstructor> impl = TestInterfaceNamedConstructor::createForJSConstructor(stringArg, defaultUndefinedOptionalBooleanArg, defaultUndefinedOptionalLongArg, defaultUndefinedOptionalStringArg, defaultNullStringOptionalstringArg);
             v8::Handle<v8::Object> wrapper = info.Holder();