Support union return type for anonymous named/indexed getter

Source/bindings/tests/results/V8TestObject.cpp

 /*
     This file is part of the Blink open source project.
     This file has been auto-generated by CodeGeneratorV8.pm. DO NOT MODIFY!
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
     version 2 of the License, or (at your option) any later version.
 
     This library is distributed in the hope that it will be useful,
@@ skipping 10 matching lines @@
 #include "config.h"
 #include "V8TestObject.h"
 
 #include "HTMLNames.h"
 #include "RuntimeEnabledFeatures.h"
 #include "ScriptProfile.h"
 #include "V8DOMStringList.h"
 #include "V8Document.h"
 #include "V8Float32Array.h"
 #include "V8Node.h"
+#include "V8NodeList.h"
 #include "V8SVGDocument.h"
 #include "V8SVGPoint.h"
 #include "V8ScriptProfile.h"
 #include "V8TestCallback.h"
 #include "V8TestNode.h"
 #include "V8TestSubObj.h"
 #include "bindings/v8/BindingSecurity.h"
 #include "bindings/v8/Dictionary.h"
 #include "bindings/v8/ScriptController.h"
 #include "bindings/v8/ScriptValue.h"
@@ skipping 4253 matching lines @@
     if (ConstructorMode::current() == ConstructorMode::WrapExistingObject)
         return args.Holder();
 
     return TestObjV8Internal::constructor(args);
 }
 
 v8::Handle<v8::Value> V8TestObject::indexedPropertyGetter(uint32_t index, const v8::AccessorInfo& info)
 {
     ASSERT(V8DOMWrapper::maybeDOMWrapper(info.Holder()));
     TestObj* collection = toNative(info.Holder());
-    RefPtr<Node> element = collection->item(index);
-    if (!element)
+    WTF::UnionType2<NodeList, Node> element = collection->item(index);
+    if (element.isNull())
         return v8Undefined();
-    return toV8Fast(element.release(), info, collection);
+    if (element.enabledValue0())
+        return toV8Fast(element.enabledValue0(), info, collection);
+    if (element.enabledValue1())

[haraken, 2013/05/15 12:09:43]

Nit: if => else if (or else)

[jsbell, 2013/05/15 21:37:52]

This is generated in a loop... IMHO the if/return style is more readable here
and for the generator.

+        return toV8Fast(element.enabledValue1(), info, collection);
+    return v8Undefined();

[jsbell, 2013/05/15 21:37:52]

But maybe generate a ASSERT_NOT_REACHED() before the return?

+
 }
 
 v8::Handle<v8::Value> V8TestObject::namedPropertyGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info)
 {
     if (!info.Holder()->GetRealNamedPropertyInPrototypeChain(name).IsEmpty())
         return v8Undefined();
     if (info.Holder()->HasRealNamedCallbackProperty(name))
         return v8Undefined();
 
     ASSERT(V8DOMWrapper::maybeDOMWrapper(info.Holder()));
@@ skipping 187 matching lines @@
     installPerContextProperties(wrapper, impl.get(), isolate);
     V8DOMWrapper::associateObjectWithWrapper(impl, &info, wrapper, isolate, hasDependentLifetime ? WrapperConfiguration::Dependent : WrapperConfiguration::Independent);
     return wrapper;
 }
 void V8TestObject::derefObject(void* object)
 {
     static_cast<TestObj*>(object)->deref();
 }
 
 } // namespace WebCore