Fenix: Add ability to delete all logins

[s-ankur]

What is the user problem or growth opportunity you want to see solved?

Currently deleting all browsing history won't delete logins. Since logins do have website information, this might leak private data. It would be great if we had a logins option in delete browsing data

How do you know that this problem exists today? Why is this important?

As a user, I want to be able to delete all user data from within the app, or on a granular scale

Who will benefit from it?

People who frequently delete browsing data

[yoasif]

This would be an extremely destructive operation, so I suggest putting an option into the "saved logins" UI instead. I think it makes sense to make this a little harder to find, as I think the negative consequences of losing logins (that may not be saved elsewhere!) are dire.

[s-ankur]

My issue is that logins are a privacy violation. I kinda expect all user data to be deleted if I tell it to delete all user data, especially websites I visit. Another option is include it in the delete browsing history but include an additional warning as you said about being very destructive

[s-ankur]

Oops

[yoasif]

My issue is that logins are a privacy violation. I kinda expect all user data to be deleted if I tell it to delete all user data, especially websites I visit. Another option is include it in the delete browsing history but include an additional warning as you said about being very destructive

I could be wrong, but Fenix doesn't save logins without asking - each login must explicitly be saved by the user. I don't see that as a "privacy violation", if users are explicitly saving something.

[s-ankur]

No no, my issue is that logins include urls which I think counts as browsing data. So when I press on delete browsing data, I expect that all of it should be deleted, including logins. But you are right it is a destructive option

[cadeyrn]

Then you should also be able to remove all bookmarks and collections via the same interface. For the record: Collections were already part of this screen and it was intentionally changed in #7421. The same reason why it was changed applies to passwords as well.

[s-ankur]

Ohh I didn't know that. Then yes, it makes sense not to add it there

Edit: if this is implemented, please also remove the logins exceptions when deleting the logins data