Good catch, Rich. But you proposed change allows the JSREPORT_WARNING to be thrown to java as an exception. I think we should only check the JSREPORT_WARNING flag.
Assignee: live-connect → kyle.yuan
Status: UNCONFIRMED → NEW
Ever confirmed: true
Created attachment 180448 [details] [diff] [review] patch
mass reassign my bugs to Pete Zha.
Assignee: kyle.yuan → pete.zha
mass reassign to Alfred
Assignee: zhayupeng → alfred.peng
Rich, could you attach one specific test case? With the proposed change, there is still "foo has no properties" in the Error Console and no Exception throw in the Java code.
Component: Java: Live Connect → Java: Live Connect
Product: Core → Core Graveyard
Firefox code moved from custom Liveconnect code to the NPAPI/NPRuntime bridge a while back. Mass-closing the bugs in the liveconnect component which are likely invalid. If you believe that this bug is still relevant to modern versions of Firefox, please reopen it and move it the "Core" product, component "Plug-Ins".
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → INCOMPLETE
Comment on attachment 180448 [details] [diff] [review] patch Remove old review request which is not needed any more.
You need to log in before you can comment on or make changes to this bug.