Right now the "Remove All" button handler loops over the logins and calls nsILoginManager::removeLogin() for each. I guess this is due to the fact that "Remove All" actually removes whatever the tree is showing, so if you're filtering logins by a search term, "Remove All" would only delete those. (I find that rather surprising, I would've expected that at least the button label changed to "Remove Shown" or something. But that's for a follow-up bug.) It would be good if the "Remove All" button would call nsILoginManager::removeAll() when you're actually wiping all logins. Because that would send out the "removeAllLogins" notification which we'd like to listen to for bug 650201, to make Sharing accounts similar to stored passwords in terms of privacy.
(In reply to comment #0) > It would be good if the "Remove All" button would call > nsILoginManager::removeAll() removeAllLogins()
Summary: "Remove All" should call nsILoginManager::removeAll() → "Remove All" should call nsILoginManager::removeAllLogins()
See Also: → bug 601447
I started looking into this, and unless we change the UX around only removing the filtered logins, the best we could do would be to call removeAllLogins() only if no filter is applied.
Yeah, I agree that's best we can do.
See Also: → bug 1239657
7 months ago
No longer blocks: 1468301
You need to log in before you can comment on or make changes to this bug.