Closed Bug 404428 Opened 17 years ago Closed 17 years ago

unnecessary JS exceptions in exceptionDialog.js

Categories

(Core :: Security: PSM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: KaiE, Assigned: KaiE)

References

Details

Attachments

(1 obsolete file)

This happens in a debug build only.
I suspect these messages are being suppressed in an optimized build?

I use a Firefox trunk nightly build, open cert manager, and click "add exception".

I get 3 exceptions in the error console:
yellow: reference to undefined property windows.arguments
yellow: reference to undefined property windows.arguments
red: window.arguments is undefined
Attached patch Patch v1 (obsolete) — Splinter Review
This fixes the problem for me
Attachment #289422 - Flags: superreview?(gavin.sharp)
Attachment #289422 - Flags: review?(johnath)
Kai - I think this problem is also fixed by my latest patch in bug 402207 which also adds the support for pref control of the exception dialog.  This bug isn't a dup of that one directly, but I think fixing that one will fix this one too, will it not?
Ok sorry, I had forgotten about that other bug where I already had seen your fix :-)

This patch is unnecessary, if bug 402207 will be picked up.
Depends on: 402207
Fixed as a result of bug 402207 landing.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Attachment #289422 - Attachment is obsolete: true
Attachment #289422 - Flags: superreview?(gavin.sharp)
Attachment #289422 - Flags: review?(johnath)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: