8186807: JSObject gets ScriptFunction when ScriptObjectMirror is expected
Reviewed-by: sundar, hannesw
Contributed-by: priya.lakshmi.muthuswamy@oracle.com
--- a/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/objects/NativeFunction.java Mon Nov 06 17:48:00 2017 -0800
+++ b/src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/objects/NativeFunction.java Tue Nov 07 12:18:29 2017 +0100
@@ -35,6 +35,7 @@
import java.util.List;
import jdk.dynalink.linker.support.Lookup;
import jdk.nashorn.api.scripting.JSObject;
+import jdk.nashorn.api.scripting.ScriptObjectMirror;
import jdk.nashorn.internal.objects.annotations.Attribute;
import jdk.nashorn.internal.objects.annotations.Constructor;
import jdk.nashorn.internal.objects.annotations.Function;
@@ -101,8 +102,13 @@
if (self instanceof ScriptFunction) {
return ScriptRuntime.apply((ScriptFunction)self, thiz, args);
+ } else if (self instanceof ScriptObjectMirror) {
+ return ((JSObject)self).call(thiz, args);
} else if (self instanceof JSObject) {
- return ((JSObject)self).call(thiz, args);
+ final Global global = Global.instance();
+ final Object result = ((JSObject) self).call(ScriptObjectMirror.wrap(thiz, global),
+ ScriptObjectMirror.wrapArray(args, global));
+ return ScriptObjectMirror.unwrap(result, global);
}
throw new AssertionError("Should not reach here");
}
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/test/nashorn/src/jdk/nashorn/internal/runtime/test/JDK_8186807_Test.java Tue Nov 07 12:18:29 2017 +0100
@@ -0,0 +1,76 @@
+/*
+ * Copyright (c) 2017, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation. Oracle designates this
+ * particular file as subject to the "Classpath" exception as provided
+ * by Oracle in the LICENSE file that accompanied this code.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+package jdk.nashorn.internal.runtime.test;
+
+import jdk.nashorn.api.scripting.AbstractJSObject;
+import jdk.nashorn.api.scripting.NashornScriptEngine;
+import jdk.nashorn.api.scripting.NashornScriptEngineFactory;
+import jdk.nashorn.api.scripting.ScriptObjectMirror;
+import static org.testng.Assert.assertEquals;
+import org.testng.annotations.Test;
+import javax.script.ScriptException;
+
+
+/**
+ * @test
+ * @bug 8186807
+ * @summary JSObject gets ScriptFunction when ScriptObjectMirror is expected
+ * @modules jdk.scripting.nashorn/jdk.nashorn.internal.runtime
+ * @run testng/othervm -Dnashorn.unstable.relink.threshold=1 jdk.nashorn.internal.runtime.test.JDK_8186807_Test
+ */
+
+public class JDK_8186807_Test {
+
+ @Test
+ public void testScript() throws ScriptException {
+ NashornScriptEngine engine = (NashornScriptEngine) new NashornScriptEngineFactory().getScriptEngine();
+ engine.put("a", new Func());
+ engine.eval("var Assert = Java.type('org.testng.Assert')\n" +
+ "var fn=function() {return 2;}\n" +
+ "var arr = [ a, fn, a ];\n" +
+ "var result;\n" +
+ "for (i in arr) {\n" +
+ " result = Function.prototype.apply.apply(arr[i],[null,[fn]]);\n" +
+ " if(i==0 || i==2)\n" +
+ " Assert.assertTrue(result==1);\n" +
+ " if(i==1)\n" +
+ " Assert.assertTrue(result==2);\n" +
+ "}");
+ }
+
+ static class Func extends AbstractJSObject {
+ @Override
+ public boolean isFunction() {
+ return true;
+ }
+
+ @Override
+ public Object call(Object thiz, Object... args) {
+ assertEquals(args[0].getClass(), ScriptObjectMirror.class);
+ return 1;
+ }
+ }
+}