Closed Bug 383209 Opened 17 years ago Closed 17 years ago
Clear Private Data fails to clear stored passwords
Steps to reproduce: 1. Visit various login forms, store passwords in password manager 2. Tools->Clear Private Data 3. Select the option to clear saved passwords 4. Clear Private Data Now 5. Open Preferences->Security->Show Passwords Note that the saved passwords are not cleared and are still in the Password Manager list. Clear Private Data does clear history and other data, so the feature as a whole seems to be working, just not the clearing of passwords. This seems pretty critical to me, as users think their stored passwords have been deleted, but we silently fail to do so.
I've narrowed down a regression range: 2007-05-15-04: Works properly 2007-05-16-04: Exhibits bug
Just did a Bonsai Query (hopefully I did this right): http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2007-05-15+04%3A00%3A00&maxdate=2007-05-16+04%3A00%3A00&cvsroot=%2Fcvsroot Could this possibly be blocking Bug 374723?
Yes, that's almost certainly the cause.
17 years ago
Flags: blocking-firefox3? → blocking-firefox3+
Target Milestone: --- → Firefox 3 alpha6
moving out, needs to make b1, please make sure this is covered in the a6 release notes (I think Deb's doing those)
Target Milestone: Firefox 3 alpha6 → Firefox 3 beta1
This is a fairly trivial fix. [I'll probably need to post an updated patch, with testcase, to get this to cleanly apply. It's on top of a couple other patches in my tree that should be landing today.]
Attachment #270041 - Flags: review?(gavin.sharp)
Attachment #270041 - Flags: review?(gavin.sharp) → review+
mozilla/toolkit/components/passwordmgr/public/nsILoginManager.idl 1.2 mozilla/toolkit/components/passwordmgr/public/nsILoginManagerStorage.idl 1.3 mozilla/toolkit/components/passwordmgr/src/nsLoginManager.js 1.8 mozilla/toolkit/components/passwordmgr/src/storage-Legacy.js 1.8
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Target Milestone: Firefox 3 beta1 → Firefox 3 alpha6
Hmm, since the implementation changed, shouldn't the iid have changed as well? http://lxr.mozilla.org/mozilla/source/toolkit/components/passwordmgr/src/nsLoginManager.js#1356
(In reply to comment #7) > Hmm, since the implementation changed, shouldn't the iid have changed as well? The iid changes when the interface changes, not the implementation. If I had changed the interface to match the implementation, then the iid would have been updated.
Verified fix on Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a6) Gecko/20070629 GranParadiso/3.0a6. Clear private data is clearing passwords now.
Status: RESOLVED → VERIFIED
Flags: in-litmus? → in-litmus+
You need to log in before you can comment on or make changes to this bug.