RESOLVED WONTFIX Bug 18234
JS exception thrown from NPN_InvokeDefault not shown in error console.
https://bugs.webkit.org/show_bug.cgi?id=18234
Summary JS exception thrown from NPN_InvokeDefault not shown in error console.
Jeremy Moskovich
Reported 2008-03-31 01:57:53 PDT
Invoking the following JS code from an NPAPI plug-in via the NPN_InvokeDefault() function doesn't show the exception in the error console: function() { throw "An Exception"; } Stepping into _NPN_InvokeDefault() in NP_jsobject.cpp shows that the exception appears to have been set correctly in the ExecState but this apparently isn't bubbled up when the plugin code returns control to the browser.
Attachments
Alexey Proskuryakov
Comment 1 2008-03-31 22:58:26 PDT
See also: bug 17470.
Eric Seidel (no email)
Comment 2 2008-04-01 12:16:55 PDT
This bug blocks Google Gears support in Safari.
Eric Seidel (no email)
Comment 3 2008-04-08 02:05:25 PDT
Working on a real fix as we speak.
Eric Seidel (no email)
Comment 4 2008-04-16 07:47:56 PDT
Updating the bug, finally. This is very much tied to the NPN_SetException bug. The fix is to save off the current ExecState in a static (or per-thread) static, and then throw the exception in that ExecState. I tried saving the ExecState on the Instance, however that doesn't work for Obj-C (since it's a static method), and doesn't work well for JS/C++ either since you have to be thrown an JavaScriptObject in order to be able to get back to an Instance. I may find time to finish up my patch, but right now real work is distracting...
David Kilzer (:ddkilzer)
Comment 5 2008-07-20 10:41:54 PDT
David Kilzer (:ddkilzer)
Comment 6 2008-07-20 15:26:10 PDT
Related to Bug 19936?
Eric Seidel (no email)
Comment 7 2011-05-17 10:48:38 PDT
This is stll likely a bug, but Gears no longer cares since Gears is dead.
Alexey Proskuryakov
Comment 8 2022-06-20 17:03:45 PDT
NPAPI is gone.
Note You need to log in before you can comment on or make changes to this bug.