Bug 19936 - Correctly propagate exception information from NPAPI callbacks
Summary: Correctly propagate exception information from NPAPI callbacks
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2008-07-07 16:34 PDT by Cameron Zwarich (cpst)
Modified: 2022-06-20 17:03 PDT (History)
7 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Cameron Zwarich (cpst) 2008-07-07 16:34:30 PDT
It came up when fixing bug 19907 that we don't propagate any exception information in an NPAPI callback. The NPAPI reference suggests that we should return false, at least in some cases:

http://developer.mozilla.org/en/docs/Gecko_Plugin_API_Reference

Some compatibility testing will be required, because the NPAPI documentation isn't very specific.
Comment 1 David Kilzer (:ddkilzer) 2008-07-20 15:25:54 PDT
Related to Bug 18234?

Comment 2 Cameron Zwarich (cpst) 2008-07-20 20:32:02 PDT
(In reply to comment #1)
> Related to Bug 18234?

Yes, that bug should probably depend on this one, or maybe even be marked a duplicate. There is now no real support for NPN_SetException.
Comment 3 Ahmad Saleem 2022-06-20 10:42:34 PDT
From Webkit Github mirror, I am only able to find following references about NPN_SetException:

https://github.com/WebKit/WebKit/blob/25eed588cb7fcdb340b0783e2c2c7022cd48897a/Source/WebCore/bridge/testbindings.cpp#L148

I think all other references of it are removed as part of NPAPI removal, since it is not supported from Safari 14 onward and also not supported on WebkitGTK port.

I think any further enhancement to NPAPI is not in radar and this can be marked as "RESOLVED WONTFIX". Thanks!
Comment 4 Radar WebKit Bug Importer 2022-06-20 10:43:02 PDT
<rdar://problem/95544910>