"Remove All" should call nsILoginManager::removeAllLogins()

NEW
Unassigned

Status

()

Toolkit
Password Manager
7 years ago
2 years ago

People

(Reporter: philikon, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [passwords:management])

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()
(Reporter)

Updated

7 years ago
Blocks: 642726, 651668

Comment 2

3 years ago
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.
Whiteboard: [passwords:management]
You need to log in before you can comment on or make changes to this bug.