Node.appendChild should throw TypeError when newChild is not an object

Source/bindings/core/v8/custom/V8NodeCustom.cpp

 /*
  * Copyright (C) 2007-2009 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
@@ skipping 89 matching lines @@
     ExceptionState exceptionState(ExceptionState::ExecutionContext, "removeChild", "Node", info.Holder(), info.GetIsolate());
     Node* oldChild = V8Node::toNativeWithTypeCheck(info.GetIsolate(), info[0]);
     impl->removeChild(oldChild, exceptionState);
     if (exceptionState.throwIfNeeded())
         return;
     v8SetReturnValue(info, info[0]);
 }
 
 void V8Node::appendChildMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
 {
+    ExceptionState exceptionState(ExceptionState::ExecutionContext, "appendChild", "Node", info.Holder(), info.GetIsolate());
+    if (info.Length() > 0 && !info[0]->IsObject()) {
+        exceptionState.throwTypeError("parameter 1 ('newChild') is not an object.");
+        exceptionState.throwIfNeeded();
+        return;

[haraken, 2014/07/08 14:59:01]

Can you refer to the generated code for [TypeChecking=Interface] and write the
same code in this custom file?

See:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

[kangil_, 2014/07/09 01:34:17]

Done.

+    }
+
     v8::Handle<v8::Object> holder = info.Holder();
     Node* impl = V8Node::toNative(holder);
 
     CustomElementCallbackDispatcher::CallbackDeliveryScope deliveryScope;
 
-    ExceptionState exceptionState(ExceptionState::ExecutionContext, "appendChild", "Node", info.Holder(), info.GetIsolate());
     Node* newChild = V8Node::toNativeWithTypeCheck(info.GetIsolate(), info[0]);
     impl->appendChild(newChild, exceptionState);
     if (exceptionState.throwIfNeeded())
         return;
     v8SetReturnValue(info, info[0]);
 }
 
 v8::Handle<v8::Object> wrap(Node* impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
 {
     ASSERT(impl);
@@ skipping 21 matching lines @@
         return wrap(toDocumentType(impl), creationContext, isolate);
     case Node::DOCUMENT_FRAGMENT_NODE:
         if (impl->isShadowRoot())
             return wrap(toShadowRoot(impl), creationContext, isolate);
         return wrap(toDocumentFragment(impl), creationContext, isolate);
     }
     ASSERT_NOT_REACHED();
     return V8Node::createWrapper(impl, creationContext, isolate);
 }
 } // namespace WebCore