some fixes based on HtmlUnit#9

Author: rbri

src/org/mozilla/javascript/NativeReflect.java

@@ -181,7 +181,7 @@ private static Scriptable construct(
             if (result != null) {
                 result.setPrototype((Scriptable) newTargetPrototype);
 
-                Object val = ctorBaseFunction.call(cx, scope, result, args);
+                Object val = ctorBaseFunction.call(cx, scope, result, callArgs);
                 if (val instanceof Scriptable) {
                     return (Scriptable) val;
                 }
@@ -325,21 +325,35 @@ private static Object preventExtensions(
 
     private static Object set(Context cx, Scriptable scope, Scriptable thisObj, Object[] args) {
         ScriptableObject target = checkTarget(args);
+        if (args.length < 2) {
+            return true;
+        }
 
-        if (args.length > 1) {
-            if (ScriptRuntime.isSymbol(args[1])) {
-                target.put((Symbol) args[1], target, args[2]);
-                return true;
-            }
-            if (args[1] instanceof Double) {
-                target.put(ScriptRuntime.toIndex(args[1]), target, args[2]);
-                return true;
+        ScriptableObject receiver = args.length > 3 ? ScriptableObject.ensureScriptableObject(args[3]) : target;
+        if (receiver != target) {
+            ScriptableObject descriptor = target.getOwnPropertyDescriptor(cx, args[1]);
+            if (descriptor != null) {
+                Object setter = descriptor.get("set");
+                if (setter != null && setter != NOT_FOUND) {
+                    ((Function) setter).call(cx, scope, receiver, new Object[] {args[2]});
+                    return true;
+                }
+
+                if (descriptor.get("configurable") == Boolean.FALSE) {
+                    return false;
+                }
             }
+        }
 
-            target.put(ScriptRuntime.toString(args[1]), target, args[2]);
-            return true;
+        if (ScriptRuntime.isSymbol(args[1])) {
+            receiver.put((Symbol) args[1], receiver, args[2]);
+        } else if (args[1] instanceof Double) {
+            receiver.put(ScriptRuntime.toIndex(args[1]), receiver, args[2]);
+        } else {
+            receiver.put(ScriptRuntime.toString(args[1]), receiver, args[2]);
         }
-        return false;
+
+        return true;
     }
 
     private static Object setPrototypeOf(

testsrc/test262.properties

@@ -1324,7 +1324,7 @@ built-ins/Proxy 74/306 (24.18%)
     get-fn-realm.js
     get-fn-realm-recursive.js
 
-built-ins/Reflect 17/139 (12.23%)
+built-ins/Reflect 13/139 (9.35%)
     construct/newtarget-is-not-constructor-throws.js
     defineProperty/return-abrupt-from-property-key.js
     deleteProperty/return-abrupt-from-result.js
@@ -1333,15 +1333,11 @@ built-ins/Reflect 17/139 (12.23%)
     ownKeys/order-after-define-property.js
     ownKeys/return-on-corresponding-order-large-index.js {unsupported: [computed-property-names]}
     set/call-prototype-property-set.js
-    set/creates-a-data-descriptor.js
     set/different-property-descriptors.js
     set/receiver-is-not-object.js
     set/return-abrupt-from-result.js
     set/return-false-if-receiver-is-not-writable.js
     set/return-false-if-target-is-not-writable.js
-    set/set-value-on-accessor-descriptor-with-receiver.js
-    set/set-value-on-data-descriptor.js
-    set/symbol-property.js
 
 built-ins/RegExp 897/1464 (61.27%)
     CharacterClassEscapes 24/24 (100.0%)