Implement data property creation for assignments to super.x.

src/objects.cc

Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index 875c20e6d61a22139f40e9ab805057cbd029ad79..48c87496a2e620e91b50af834dcd95e54097e763 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -2906,13 +2906,11 @@ MaybeHandle<Object> Object::SetProperty(LookupIterator* it,
   }
 
   if (data_store_mode == SUPER_PROPERTY) {
-    if (strict_mode == STRICT) {
-      Handle<Object> args[1] = {it->name()};
-      THROW_NEW_ERROR(it->isolate(),
-                      NewReferenceError("not_defined", HandleVector(args, 1)),
-                      Object);
-    }
-    return value;
+    LookupIterator own_lookup(it->GetReceiver(), it->name(),
+                              LookupIterator::OWN);
+
+    return JSObject::SetProperty(&own_lookup, value, strict_mode, store_mode,

[Toon Verwaest, 2015/02/16 09:33:11]

This is not correct as far as I can tell. Your implementation will perform the
set based on the state of the receiver.

As far as I read the spec, it says that if no descriptor is found on the
prototype chain, you have to set "OwnDesc" to a writable, enumerable,
configurable data descriptor. If there's an existing previous "valueDesc" on the
receiver, you [[DefineOwnProperty]] to redefine the accessor. Otherwise you
CreateDataProperty(Receiver, P, V).

+                                 NORMAL_PROPERTY);
   }
 
   return AddDataProperty(it, value, NONE, strict_mode, store_mode);