divyang4481 / firebreath

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

throw FB::script_error() exception in plugin is caught as Error exception in browser without custom text #127

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. in plugin: throw FB::script_error("error description");
2. catch exception in browser
3.

What is the expected output? What do you see instead?
Expect to get "error description" in browser caught exception.
Instead get: "Error calling method on NPObject."

What version of FireBreath are you using? On what operating system and
browsers?
firebreath-dev-1.4-nightly-69 with system Boost 1.43
Windows 7 x64
Chrome 8.0

What version of your compiler or IDE are you using?
Visual Studio 2010

Please provide any additional information below.
Please find attached a FBTestPlugin test that exposes the issue.

Original issue reported on code.google.com by rjani...@gmail.com on 6 Jan 2011 at 12:53

Attachments:

GoogleCodeExporter commented 8 years ago
This issue seems to be a bug in Chrome, since it works in all other NPAPI 
browsers; the same issue has been reported using Chromium on Linux.  I 
recommend that you file a bug on Chrome that NPN_SetException is not working 
correctly.

see http://code.google.com/p/chromium/issues/detail?id=26764#makechanges

I put a comment on the old issue, but a new one should probably be opened. I 
don't think there is anything we can do unless we can show somehow that this is 
a FireBreath bug, which all tests seem to indicate it is not.

Original comment by rich...@firebreath.org on 7 Jan 2011 at 4:38

GoogleCodeExporter commented 8 years ago
Do you think it would be ok / useful to ask the FireBreath community to go vote 
for the Chromium issue?

Original comment by rjani...@gmail.com on 7 Jan 2011 at 6:13

GoogleCodeExporter commented 8 years ago
I don't have a problem with that.

Original comment by rich...@firebreath.org on 7 Jan 2011 at 6:17

GoogleCodeExporter commented 8 years ago
Marking this as invalid, since we can't do anything about it.

Original comment by taxilian on 14 Jan 2011 at 12:22