search

hust-marx / firebreath

Automatically exported from code.google.com/p/firebreath
0 stars 0 forks source link

Passing JS Arrays to the plugin crashes Safari #63

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Using FBTestPlugin
2. Call plugin.reverseArray(new Array(1, 2, 3, 4));
3. Observe crash

What is the expected output? What do you see instead?
Expect to see ['4', '3', '2', '1'] returned. Returns successfully in Firefox. 
Instead Safari hangs and never returns from the function. Browser becomes 
unresponsive.

Original issue reported on code.google.com by amack...@gmail.com on 12 Sep 2010 at 8:38

GoogleCodeExporter commented 9 years ago 
So aparently Safari 64 bit no longer allows us to use Int NPIdentifiers.  That 
is interesting.  Aparently, though, converting the int to a string and using 
String NPIdentifiers works fine on all browsers (at least all that I tested 
on).  So, it is fixed.

I also added a workaround to inconsistent performance of the 
pluginthreadasynccall on safari 64 bit that was causing a crash.

Original comment by taxilian on 14 Sep 2010 at 12:52