[rfc][icedtea-web] Fix javascript url error in JSToJSet reproducer

[Jie Kang]

Hello,

The reproducers for JSToJSet are broken due to errors in the javascript code. The error causing the code to fail has been fixed by decoding the URL before using it in arguments.

Previously a URL containing characters like [ was not decoded and so the argument would end up including the characters "%5B" as opposed to "[".


Okay to push?



There are still some parts of the javascript that result in failing tests but it is not 100% clear whether the behaviour is expected when compared to Oracle's implementation of live-connect.

Example:

javascript code:

Case 1:
var value = "1";
applet.integer = value;

versus

Case 2:
var value = 1;
applet.integer = value;


When applet.integer is declared as an Integer in java code, case 1 fails while case 2 passes. The javascript code in JSToJSet tests for case 1 at the moment.

However, when applet.integer is declared as an int (primitive as opposed to Object) in java code, both cases pass.

This affects most boxed primitives such as Double, Float, Byte, etc.

I'm assuming it should pass for Case 1 when using Integer, so there are still some issues with our implementation of LiveConnect :\


Regards,


-- 

Jie Kang

OpenJDK Team - Software Engineering Intern
-------------- next part --------------
A non-text attachment was scrubbed...
Name: itw-fix-jstojset-reproducer-1.patch
Type: text/x-patch
Size: 2397 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20150130/c97cc71c/itw-fix-jstojset-reproducer-1.patch>