add context parameter to the new AbstractEcmaObjectOperations methods

Author: rbri

rhino/src/main/java/org/mozilla/javascript/AbstractEcmaObjectOperations.java

@@ -358,8 +358,9 @@ static long lengthOfArrayLike(Context cx, Scriptable o) {
      * <p>https://262.ecma-international.org/12.0/#sec-iscompatiblepropertydescriptor
      */
     static boolean isCompatiblePropertyDescriptor(
-            boolean extensible, ScriptableObject desc, ScriptableObject current) {
+            Context cx, boolean extensible, ScriptableObject desc, ScriptableObject current) {
         return validateAndApplyPropertyDescriptor(
+                cx,
                 Undefined.SCRIPTABLE_UNDEFINED,
                 Undefined.SCRIPTABLE_UNDEFINED,
                 extensible,
@@ -373,6 +374,7 @@ static boolean isCompatiblePropertyDescriptor(
      * <p>https://262.ecma-international.org/12.0/#sec-validateandapplypropertydescriptor
      */
     static boolean validateAndApplyPropertyDescriptor(
+            Context cx,
             Scriptable o,
             Scriptable p,
             boolean extensible,
@@ -469,7 +471,7 @@ static boolean validateAndApplyPropertyDescriptor(
      *
      * <p>https://262.ecma-international.org/12.0/#sec-isconstructor
      */
-    static boolean isConstructor(Object argument) {
+    static boolean isConstructor(Context cx, Object argument) {
         /*
            The abstract operation IsConstructor takes argument argument (an ECMAScript language value).
            It determines if argument is a function object with a [[Construct]] internal method.

rhino/src/main/java/org/mozilla/javascript/NativeProxy.java

@@ -1036,7 +1036,7 @@ public boolean defineOwnProperty(Context cx, Object id, ScriptableObject desc) {
                 }
             } else {
                 if (!AbstractEcmaObjectOperations.isCompatiblePropertyDescriptor(
-                        extensibleTarget, desc, targetDesc)) {
+                        cx, extensibleTarget, desc, targetDesc)) {
                     throw ScriptRuntime.typeError(
                             "proxy can't define an incompatible property descriptor");
                 }

rhino/src/main/java/org/mozilla/javascript/NativeReflect.java

@@ -142,7 +142,7 @@ private static Scriptable construct(
                     Integer.toString(args.length));
         }
 
-        if (!AbstractEcmaObjectOperations.isConstructor(args[0])) {
+        if (!AbstractEcmaObjectOperations.isConstructor(cx, args[0])) {
             throw ScriptRuntime.typeErrorById("msg.not.ctor", ScriptRuntime.typeof(args[0]));
         }
 
@@ -151,7 +151,7 @@ private static Scriptable construct(
             return ctor.construct(cx, scope, ScriptRuntime.emptyArgs);
         }
 
-        if (args.length > 2 && !AbstractEcmaObjectOperations.isConstructor(args[2])) {
+        if (args.length > 2 && !AbstractEcmaObjectOperations.isConstructor(cx, args[2])) {
             throw ScriptRuntime.typeErrorById("msg.not.ctor", ScriptRuntime.typeof(args[2]));
         }