Closed Bug 100198 Opened 23 years ago Closed 23 years ago

nsPrompt depends on wallet

Categories

(Core Graveyard :: Embedding: APIs, defect, P3)

x86
Windows 2000
defect

Tracking

(Not tracked)

VERIFIED DUPLICATE of bug 113538
mozilla0.9.8

People

(Reporter: alecf, Assigned: alecf)

References

Details

nsPrompt.cpp has us trying to create a single signon prompt, which is part of wallet... it's creating a very strange dependency between embedding components and wallet. cc'ing morse for any ideas
Blocks: 100107
Status: NEW → ASSIGNED
No ideas. This is something that Warren Harris dropped into wallet a long time ago.
steve, do you have any idea what this code even does? I'm wondering if we can just rip it out :) I'm trying to figure it out right now..
It's the mozilla mechanism for putting up prompt messages. These messages are used heavily in wallet, which is why Warren put it there. But it is used in other places as well. These are all the messages of the form Do you want to allow this site to set a cookie Enter your username and password for http authentication Do you want to save your password etc.
QA Contact: mdunn → depstein
Target Milestone: --- → mozilla0.9.5
reassigning qa contact to Dharma. Prompts is his area.
QA Contact: depstein → dsirnapalli
Target Milestone: mozilla0.9.5 → mozilla0.9.6
Conrad beat on prompts recently. cc'ing him for a glance.
nsPrompt is what hides wallet from nsIPromptService and thus embeddors. On the other end, through nsIAuthPrompt, it also hides wallet from necko. It had to be hidden somewhere. Before it was hidden here, the embeddor had to explicitly create a single-signon.
Priority: -- → P3
Target Milestone: mozilla0.9.6 → mozilla0.9.7
too much on my plate for 0.9.7
Target Milestone: mozilla0.9.7 → mozilla0.9.8
waterson has a patch *** This bug has been marked as a duplicate of 113538 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
verified.
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.