While reading code, I found what I believe is an error. The code asks for the "service" nsIDialogParamBlock. But there is no such service. Each dialog must use its own parameter block. By asking for the service, I fear the application will reuse the parameter block of a different dialog. And this will mess up the parameter block of the other dialog. In order to fix this bug, the code must request a new instance.
Assigning to myself.
Assignee: ssaux → kaie
Priority: -- → P1
Target Milestone: --- → 2.2
I found another occurrence of this error in certManager.js
Javi, can you please review?
Status: NEW → ASSIGNED
Comment on attachment 59149 [details] [diff] [review] Suggested fix r=javi
Attachment #59149 - Flags: review+
Comment on attachment 59149 [details] [diff] [review] Suggested fix sr=shaver
Attachment #59149 - Flags: superreview+
checked in, fixed
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Verified per kaie's comment.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.