Re: RFR:JDK-8186807:JSObject gets ScriptFunction when ScriptObjectMirror is expected

2017-11-03 Thread Hannes Wallnöfer
+1 Hannes > Am 03.11.2017 um 13:27 schrieb Sundararajan Athijegannathan > : > > +1 > > -Sundar > > On 03/11/17, 5:15 PM, Priya Lakshmi Muthuswamy wrote: >> Updated the webrev with modified testcase. >> >> webrev: http://cr.openjdk.java.net/~pmuthuswamy/8186807/webrev.01/ >> >> Thanks, >> Pr

Re: RFR:JDK-8186807:JSObject gets ScriptFunction when ScriptObjectMirror is expected

2017-11-03 Thread Sundararajan Athijegannathan
+1 -Sundar On 03/11/17, 5:15 PM, Priya Lakshmi Muthuswamy wrote: Updated the webrev with modified testcase. webrev: http://cr.openjdk.java.net/~pmuthuswamy/8186807/webrev.01/ Thanks, Priya On 11/3/2017 3:11 PM, Sundararajan Athijegannathan wrote: Two suggestions: * You may want to add a che

Re: RFR:JDK-8186807:JSObject gets ScriptFunction when ScriptObjectMirror is expected

2017-11-03 Thread Priya Lakshmi Muthuswamy
Updated the webrev with modified testcase. webrev: http://cr.openjdk.java.net/~pmuthuswamy/8186807/webrev.01/ Thanks, Priya On 11/3/2017 3:11 PM, Sundararajan Athijegannathan wrote: Two suggestions: * You may want to add a check to make sure that the control reached func.call method - there i

Re: RFR:JDK-8186807:JSObject gets ScriptFunction when ScriptObjectMirror is expected

2017-11-03 Thread Sundararajan Athijegannathan
Two suggestions: * You may want to add a check to make sure that the control reached func.call method - there is an assert there but the test does not assert the control reached there * class name "func" be changed to "Func" or some such (capitalization of class names) -Sundar On 03/11/17