make sure to use only the RegExpProxy interface

build.gradle

@@ -59,6 +59,9 @@ sourceSets {
 
 dependencies {
     testImplementation "junit:junit:4.13.2"
+    testImplementation("com.tngtech.archunit:archunit-junit4:1.2.0") {
+        exclude group: 'junit'
+    }
     testImplementation "org.yaml:snakeyaml:1.28"
     testImplementation "javax.xml.soap:javax.xml.soap-api:1.4.0"
     jmhImplementation project

src/org/mozilla/javascript/NativeArray.java

@@ -19,7 +19,6 @@
 import java.util.List;
 import java.util.ListIterator;
 import java.util.NoSuchElementException;
-import org.mozilla.javascript.regexp.NativeRegExp;
 import org.mozilla.javascript.xml.XMLObject;
 
 /**
@@ -1152,20 +1151,24 @@ private static Function getCallbackArg(Context cx, Object callbackArg) {
         if (!(callbackArg instanceof Function)) {
             throw ScriptRuntime.notFunctionError(callbackArg);
         }
-        if (cx.getLanguageVersion() >= Context.VERSION_ES6
-                && (callbackArg instanceof NativeRegExp)) {
-            // Previously, it was allowed to pass RegExp instance as a callback (it implements
-            // Function)
-            // But according to ES2015 21.2.6 Properties of RegExp Instances:
-            // > RegExp instances are ordinary objects that inherit properties from the RegExp
-            // prototype object.
-            // > RegExp instances have internal slots [[RegExpMatcher]], [[OriginalSource]], and
-            // [[OriginalFlags]].
-            // so, no [[Call]] for RegExp-s
-            throw ScriptRuntime.notFunctionError(callbackArg);
-        }
 
         Function f = (Function) callbackArg;
+
+        if (cx.getLanguageVersion() >= Context.VERSION_ES6) {
+            RegExpProxy reProxy = ScriptRuntime.getRegExpProxy(cx);
+            if (reProxy != null && reProxy.isRegExp(f)) {
+                // Previously, it was allowed to pass RegExp instance as a callback (it implements
+                // Function)
+                // But according to ES2015 21.2.6 Properties of RegExp Instances:
+                // > RegExp instances are ordinary objects that inherit properties from the RegExp
+                // prototype object.
+                // > RegExp instances have internal slots [[RegExpMatcher]], [[OriginalSource]], and
+                // [[OriginalFlags]].
+                // so, no [[Call]] for RegExp-s
+                throw ScriptRuntime.notFunctionError(callbackArg);
+            }
+        }
+
         return f;
     }
 

src/org/mozilla/javascript/NativeString.java

@@ -13,7 +13,6 @@
 import java.text.Normalizer;
 import java.util.Locale;
 import org.mozilla.javascript.ScriptRuntime.StringIdOrIndex;
-import org.mozilla.javascript.regexp.NativeRegExp;
 
 /**
  * This class implements the String native object.
@@ -462,11 +461,16 @@ public Object execIdCall(
                 case Id_endsWith:
                     String thisString =
                             ScriptRuntime.toString(requireObjectCoercible(cx, thisObj, f));
-                    if (args.length > 0 && args[0] instanceof NativeRegExp) {
-                        throw ScriptRuntime.typeErrorById(
-                                "msg.first.arg.not.regexp",
-                                String.class.getSimpleName(),
-                                f.getFunctionName());
+                    if (args.length > 0) {
+                        RegExpProxy reProxy = ScriptRuntime.getRegExpProxy(cx);
+                        if (reProxy != null
+                                && args[0] instanceof Scriptable
+                                && reProxy.isRegExp((Scriptable) args[0])) {
+                            throw ScriptRuntime.typeErrorById(
+                                    "msg.first.arg.not.regexp",
+                                    String.class.getSimpleName(),
+                                    f.getFunctionName());
+                        }
                     }
 
                     int idx = js_indexOf(id, thisString, args);

testsrc/org/mozilla/javascript/archunit/ArchitectureTest.java

@@ -0,0 +1,40 @@
+/* -*- Mode: java; tab-width: 8; indent-tabs-mode: nil; c-basic-offset: 4 -*-
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+package org.mozilla.javascript.archunit;
+
+import static com.tngtech.archunit.lang.syntax.ArchRuleDefinition.noClasses;
+
+import com.tngtech.archunit.core.importer.ImportOption;
+import com.tngtech.archunit.junit.AnalyzeClasses;
+import com.tngtech.archunit.junit.ArchTest;
+import com.tngtech.archunit.junit.ArchUnitRunner;
+import com.tngtech.archunit.lang.ArchRule;
+import org.junit.runner.RunWith;
+
+/**
+ * Architecture tests.
+ *
+ * @author Ronald Brill
+ */
+@RunWith(ArchUnitRunner.class)
+@AnalyzeClasses(
+        packages = "org.mozilla.javascript",
+        importOptions = ImportOption.DoNotIncludeTests.class)
+public class ArchitectureTest {
+
+    /** Use only the RegExpProxy. */
+    @ArchTest
+    public static final ArchRule corejsPackageRule =
+            noClasses()
+                    .that()
+                    .resideOutsideOfPackage("org.mozilla.javascript.regexp..")
+                    .and()
+                    .doNotHaveFullyQualifiedName("org.mozilla.javascript.RegExpProxy")
+                    .should()
+                    .dependOnClassesThat()
+                    .resideInAnyPackage("org.mozilla.javascript.regexp..");
+}