diff --git a/src/ecmascript/spidermonkey/element.cpp b/src/ecmascript/spidermonkey/element.cpp
index 2159a4421..5a3838011 100644
--- a/src/ecmascript/spidermonkey/element.cpp
+++ b/src/ecmascript/spidermonkey/element.cpp
@@ -3973,7 +3973,6 @@ element_appendChild(JSContext *ctx, unsigned int argc, JS::Value *rval)
 #endif
 		return false;
 	}
-
 	dom_node *el = (dom_node *)JS::GetMaybePtrFromReservedSlot<dom_node>(hobj, 0);
 	dom_node *res = NULL;
 	dom_exception exc;
@@ -4766,34 +4765,31 @@ element_insertBefore(JSContext *ctx, unsigned int argc, JS::Value *rval)
 	JS::RootedObject hobj(ctx, &args.thisv().toObject());
 
 	if (!JS_InstanceOf(ctx, hobj, &element_class, NULL)) {
-		args.rval().setBoolean(false);
-		return true;
+		return false;
 	}
 	dom_node *el = (dom_node *)JS::GetMaybePtrFromReservedSlot<dom_node>(hobj, 0);
 
 	if (!el) {
-		args.rval().setUndefined();
-		return true;
+		return false;
 	}
-	JS::RootedObject next_sibling1(ctx, &args[1].toObject());
-	JS::RootedObject child1(ctx, &args[0].toObject());
+	dom_node *next_sibling = NULL;
+	dom_node *child = NULL;
 
-	dom_node *next_sibling = (dom_node *)JS::GetMaybePtrFromReservedSlot<dom_node>(next_sibling1, 0);
-
-	if (!next_sibling) {
-		args.rval().setNull();
-		return true;
+	if (!args[1].isNull()) {
+		JS::RootedObject next_sibling1(ctx, &args[1].toObject());
+		next_sibling = (dom_node *)JS::GetMaybePtrFromReservedSlot<dom_node>(next_sibling1, 0);
 	}
 
-	dom_node *child = (dom_node *)JS::GetMaybePtrFromReservedSlot<dom_node>(child1, 0);
-
+	if (!args[0].isNull()) {
+		JS::RootedObject child1(ctx, &args[0].toObject());
+		child = (dom_node *)JS::GetMaybePtrFromReservedSlot<dom_node>(child1, 0);
+	}
 	dom_exception err;
-	dom_node *spare;
+	dom_node *spare = NULL;
 
 	err = dom_node_insert_before(el, child, next_sibling, &spare);
-	if (err != DOM_NO_ERR) {
-		args.rval().setUndefined();
-		return true;
+	if (err != DOM_NO_ERR || !spare) {
+		return false;
 	}
 	JSObject *obj = getElement(ctx, spare);
 	dom_node_unref(spare);