Fix another corner-case behavior of Object::SetSuperProperty.

test/mjsunit/harmony/reflect.js

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Flags: --harmony-reflect
 
 // TODO(neis): Test with proxies.
 
 
 
@@ skipping 226 matching lines @@
     assertEquals(44, target.nowrite);
 
     assertFalse(Reflect.set(target, "nowrite", value, target));
     assertEquals(44, target.nowrite);
 
     assertFalse(Reflect.set(target, "nowrite", value, receiver));
     assertEquals(44, target.nowrite);
     assertFalse("nowrite" in receiver);
 
     // Data vs Non-Writable
-    // TODO(neis): This must return false but currently doesn't.
-    // assertFalse(Reflect.set({}, "nowrite", value, target));
+    assertFalse(Reflect.set({}, "nowrite", value, target));
 
     // Data vs Accessor
     assertFalse(Reflect.set({}, "unknown", 0, {set unknown(x) {}}));
     assertFalse(Reflect.set(target, "unknown", value, {set unknown(x) {}}));
     assertFalse(Reflect.set(target, "bla", value, {set bla(x) {}}));
     assertFalse(Reflect.set(target, "bla", value, {get bla() {}}));
 
     // Accessor vs Data
     assertTrue(Reflect.set({set bla(x) {}}), "bla", value, target);
     assertFalse(Reflect.set({get bla() {}}, "bla", value, target));
 
     // Data vs Non-Object
     assertFalse(Reflect.set({}, "bla", value, null));
     assertFalse(Reflect.set({bla: 42}, "bla", value, null));
 
     // Accessor vs Non-Object
     assertTrue(Reflect.set(target, "setter2", value, null));
     assertFalse(Reflect.set(target, "getter", value, null));
+
+    let receiver2 = {bla: true, not_in_target: true};
+    Object.preventExtensions(receiver2);
+    assertFalse(Reflect.set(target, "bla", value, receiver2));
+    assertFalse(Reflect.set(target, "bla", true, receiver2));
+    assertFalse(Reflect.set(target, "not_in_target", value, receiver2));
+    assertFalse(Reflect.set(target, "not_in_target", true, receiver2));
+
+    let receiver3 = {};
+    Object.defineProperty(receiver3, "bla",
+        {configurable: false, writable: true, value: true});
+    Object.defineProperty(receiver3, "not_in_target",
+        {configurable: false, writable: true, value: true});
+    assertTrue(Reflect.set(target, "bla", value, receiver3));
+    assertTrue(Reflect.set(target, "not_in_target", value, receiver3));
   }
 })();
 
 
 
 ////////////////////////////////////////////////////////////////////////////////
 // Reflect.has
 
 
 (function testReflectHasArity() {
@@ skipping 268 matching lines @@
   assertThrows(function() { Reflect.preventExtensions(); }, TypeError);
   assertThrows(function() { Reflect.preventExtensions(42); }, TypeError);
   assertThrows(function() { Reflect.preventExtensions(null); }, TypeError);
 })();
 
 
 // See reflect-prevent-extensions.js for further tests.
 
 // TODO(neis): Need proxies to test the situation where
 // [[preventExtensions]] returns false.