Fix corner-case behavior of Object::SetSuperProperty.

test/mjsunit/es6/super.js

 // Copyright 2014 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: --allow-natives-syntax
 // Flags: --harmony-destructuring --harmony-rest-parameters --harmony-sloppy
 
 (function TestSuperNamedLoads() {
   function Base() { }
   function fBase() { }
@@ skipping 931 matching lines @@
       assertEquals(55, this.ownReadOnly);
       var descr = Object.getOwnPropertyDescriptor(this, 'ownReadOnly');
       assertEquals(55, descr.value);
       assertTrue(descr.configurable);
       assertFalse(descr.enumerable);
       assertFalse(descr.writable);
       assertFalse(Base.prototype.hasOwnProperty('ownReadOnly'));
 
       assertEquals(15, this.ownReadonlyAccessor);
       super.ownReadonlyAccessor = 25;
-      assertEquals(25, this.ownReadonlyAccessor);
-      var descr = Object.getOwnPropertyDescriptor(this, 'ownReadonlyAccessor');
-      assertEquals(25, descr.value);
-      assertTrue(descr.configurable);
-      assertFalse(descr.enumerable);
-      assertTrue(descr.writable);
+      assertSame(undefined, super.ownReadonlyAccessor);
+      assertEquals(15, this.ownReadonlyAccessor);
       assertFalse(Base.prototype.hasOwnProperty('ownReadonlyAccessor'));
 
       super.ownSetter = 35;
-      assertEquals(35, this.ownSetter);
+      assertSame(undefined, super.ownSetter);
       var descr = Object.getOwnPropertyDescriptor(this, 'ownSetter');
-      assertEquals(35, descr.value);
-      assertTrue(descr.configurable);
-      assertFalse(descr.enumerable);
-      assertTrue(descr.writable);
+      assertTrue('set' in descr);
       assertFalse(Base.prototype.hasOwnProperty('ownSetter'));
     },
     mStrict() {
       'use strict';
       assertEquals(42, this.ownReadOnly);
       super.ownReadOnly = 55;
       assertEquals(55, this.ownReadOnly);
       var descr = Object.getOwnPropertyDescriptor(this, 'ownReadOnly');
       assertEquals(55, descr.value);
       assertTrue(descr.configurable);
       assertFalse(descr.enumerable);
       assertFalse(descr.writable);
       assertFalse(Base.prototype.hasOwnProperty('ownReadOnly'));
 
       assertEquals(15, this.ownReadonlyAccessor);
-      super.ownReadonlyAccessor = 25;
-      assertEquals(25, this.ownReadonlyAccessor);
-      var descr = Object.getOwnPropertyDescriptor(this, 'ownReadonlyAccessor');
-      assertEquals(25, descr.value);
-      assertTrue(descr.configurable);
-      assertFalse(descr.enumerable);
-      assertTrue(descr.writable);
+      assertThrows(() => {super.ownReadonlyAccessor = 25}, TypeError);
+      assertSame(undefined, super.ownReadonlyAccessor);
+      assertEquals(15, this.ownReadonlyAccessor);
       assertFalse(Base.prototype.hasOwnProperty('ownReadonlyAccessor'));
 
-      super.ownSetter = 35;
-      assertEquals(35, this.ownSetter);
+      assertThrows(() => {super.ownSetter = 35}, TypeError);
+      assertSame(undefined, super.ownSetter);
       var descr = Object.getOwnPropertyDescriptor(this, 'ownSetter');
-      assertEquals(35, descr.value);
-      assertTrue(descr.configurable);
-      assertFalse(descr.enumerable);
-      assertTrue(descr.writable);
+      assertTrue('set' in descr);
       assertFalse(Base.prototype.hasOwnProperty('ownSetter'));
     },
   };
 
   var d = new Derived();
   Object.defineProperty(d, 'ownReadOnly', {
     value: 42,
     writable: false,
     configurable: true
   });
@@ skipping 37 matching lines @@
       assertEquals(42, this.ownReadOnly);
       var descr = Object.getOwnPropertyDescriptor(this, 'ownReadOnly');
       assertEquals(42, descr.value);
       assertFalse(descr.configurable);
       assertFalse(descr.enumerable);
       assertFalse(descr.writable);
       assertFalse(Base.prototype.hasOwnProperty('ownReadOnly'));
 
       assertEquals(15, this.ownReadonlyAccessor);
       super.ownReadonlyAccessor = 25;
+      assertSame(undefined, super.ownReadonlyAccessor);
       assertEquals(15, this.ownReadonlyAccessor);
       var descr = Object.getOwnPropertyDescriptor(this, 'ownReadonlyAccessor');
       assertFalse(descr.configurable);
       assertFalse(descr.enumerable);
       assertFalse(Base.prototype.hasOwnProperty('ownReadonlyAccessor'));
 
       super.ownSetter = 35;
       var descr = Object.getOwnPropertyDescriptor(this, 'ownSetter');
       assertFalse(descr.configurable);
       assertFalse(descr.enumerable);
@@ skipping 118 matching lines @@
       assertEquals(55, this[ownReadOnly]);
       var descr = Object.getOwnPropertyDescriptor(this, ownReadOnly);
       assertEquals(55, descr.value);
       assertTrue(descr.configurable);
       assertFalse(descr.enumerable);
       assertFalse(descr.writable);
       assertFalse(Base.prototype.hasOwnProperty(ownReadOnly));
 
       assertEquals(15, this[ownReadonlyAccessor]);
       super[ownReadonlyAccessor] = 25;
-      assertEquals(25, this[ownReadonlyAccessor]);
-      var descr = Object.getOwnPropertyDescriptor(this, ownReadonlyAccessor);
-      assertEquals(25, descr.value);
-      assertTrue(descr.configurable);
-      assertFalse(descr.enumerable);
-      assertTrue(descr.writable);
+      assertSame(undefined, super[ownReadonlyAccessor]);
+      assertEquals(15, this[ownReadonlyAccessor]);
       assertFalse(Base.prototype.hasOwnProperty(ownReadonlyAccessor));
 
       super[ownSetter] = 35;
-      assertEquals(35, this[ownSetter]);
+      assertSame(undefined, super[ownSetter]);
       var descr = Object.getOwnPropertyDescriptor(this, ownSetter);
-      assertEquals(35, descr.value);
-      assertTrue(descr.configurable);
-      assertFalse(descr.enumerable);
-      assertTrue(descr.writable);
+      assertTrue('set' in descr);
       assertFalse(Base.prototype.hasOwnProperty(ownSetter));
     },
     mStrict() {
       'use strict';
       assertEquals(42, this[ownReadOnly]);
       super[ownReadOnly] = 55;
       assertEquals(55, this[ownReadOnly]);
       var descr = Object.getOwnPropertyDescriptor(this, ownReadOnly);
       assertEquals(55, descr.value);
       assertTrue(descr.configurable);
       assertFalse(descr.enumerable);
       assertFalse(descr.writable);
       assertFalse(Base.prototype.hasOwnProperty(ownReadOnly));
 
       assertEquals(15, this[ownReadonlyAccessor]);
-      super[ownReadonlyAccessor] = 25;
-      assertEquals(25, this[ownReadonlyAccessor]);
-      var descr = Object.getOwnPropertyDescriptor(this, ownReadonlyAccessor);
-      assertEquals(25, descr.value);
-      assertTrue(descr.configurable);
-      assertFalse(descr.enumerable);
-      assertTrue(descr.writable);
+      assertThrows(() => {super[ownReadonlyAccessor] = 25}, TypeError);
+      assertSame(undefined, super[ownReadonlyAccessor]);
+      assertEquals(15, this[ownReadonlyAccessor]);
       assertFalse(Base.prototype.hasOwnProperty(ownReadonlyAccessor));
 
-      super[ownSetter] = 35;
-      assertEquals(35, this[ownSetter]);
+      assertThrows(() => {super[ownSetter] = 35}, TypeError);
+      assertSame(undefined, super[ownSetter]);
       var descr = Object.getOwnPropertyDescriptor(this, ownSetter);
-      assertEquals(35, descr.value);
-      assertTrue(descr.configurable);
-      assertFalse(descr.enumerable);
-      assertTrue(descr.writable);
+      assertTrue('set' in descr);
       assertFalse(Base.prototype.hasOwnProperty(ownSetter));
     },
   };
 
   var d = new Derived();
   Object.defineProperty(d, ownReadOnly, {
     value: 42,
     writable: false,
     configurable: true
   });
@@ skipping 42 matching lines @@
       assertEquals(42, this[ownReadOnly]);
       var descr = Object.getOwnPropertyDescriptor(this, ownReadOnly);
       assertEquals(42, descr.value);
       assertFalse(descr.configurable);
       assertFalse(descr.enumerable);
       assertFalse(descr.writable);
       assertFalse(Base.prototype.hasOwnProperty(ownReadOnly));
 
       assertEquals(15, this[ownReadonlyAccessor]);
       super[ownReadonlyAccessor] = 25;
+      assertSame(undefined, super[ownReadonlyAccessor]);
       assertEquals(15, this[ownReadonlyAccessor]);
       var descr = Object.getOwnPropertyDescriptor(this, ownReadonlyAccessor);
       assertFalse(descr.configurable);
       assertFalse(descr.enumerable);
       assertFalse(Base.prototype.hasOwnProperty(ownReadonlyAccessor));
 
       super[ownSetter] = 35;
       var descr = Object.getOwnPropertyDescriptor(this, ownSetter);
       assertFalse(descr.configurable);
       assertFalse(descr.enumerable);
@@ skipping 938 matching lines @@
   }
   class Derived extends Base {
     constructor(x) {
       let r = (() => super(...[x]))();
       assertEquals(this, r);
     }
   }
   let d = new Derived(42);
   assertSame(42, d.x);
 })();