Closed
Bug 451040
Opened 16 years ago
Closed 16 years ago
Passwords Manager Empty after convert to MozStorage
Categories
(Toolkit :: Password Manager, defect)
Toolkit
Password Manager
Tracking
()
VERIFIED
FIXED
mozilla1.9.1a2
People
(Reporter: jmjjeffery, Assigned: Dolske)
References
Details
(Keywords: dataloss, dogfood, regression)
Attachments
(1 file)
1.67 KB,
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
Starting with today's nightly build the Password Manager is Empty. This appears in the Error Console: Error: Cc[contractID] is undefined Source file: file:///C:/Program%20Files/Minefield/components/nsLoginManager.js Line: 112
Updated•16 years ago
|
Flags: blocking1.9.1?
OS: Windows Vista → All
Reporter | ||
Comment 1•16 years ago
|
||
There is also no 'Notification Bar' shown when adding a site that requires a login, and no signons3.txt is created at all, nor any password files that I see other than key3.db Above noted with a new profile, no addons.
Comment 2•16 years ago
|
||
Pretty sure this breaks signins to Weave, too.
Comment 3•16 years ago
|
||
(In reply to comment #2) > Pretty sure this breaks signins to Weave, too. > I can confirm this. Weave would not sign in. Even the dialog box was not populated with password nor passphrase. Weave just sat there when I tried inputting them and then trying to log in. I have reverted to yesterday's nightly and am now signed in. Also, FYI-Logins and passwords are accessible in yesterday's nightly under Tools > Options > Security > Passwords > Saved Passwords...
Comment 5•16 years ago
|
||
(In reply to comment #0) > This appears in the Error Console: > Error: Cc[contractID] is undefined > Source file: file:///C:/Program%20Files/Minefield/components/nsLoginManager.js > Line: 112 That's strange. The tests should not be passing if that fails :(
Comment 6•16 years ago
|
||
Maybe this file needs to be added to packages-static?
Assignee | ||
Comment 7•16 years ago
|
||
Ugh, yes, packages-static is almost certainly the problem. *sigh*
Assignee | ||
Comment 8•16 years ago
|
||
This bit us in the original password manager landing (bug 374723 comment 29), too. You'd think we'd learn. :)
Updated•16 years ago
|
Attachment #334300 -
Flags: review?(gavin.sharp) → review+
Comment 9•16 years ago
|
||
What's strange is that unit test boxes used to not work with the component either when it wasn't in packages-static. So, why did they not go orange this time?
Assignee | ||
Comment 10•16 years ago
|
||
Pushed changeset c37a5b4ed1af. Filed bug 451059 to have the tinderboxen clobbered, so that nightly users will get a fixed update ASAP.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: blocking1.9.1?
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1a2
Comment 11•16 years ago
|
||
You can clobber yourself by modifying the CLOBBER file: http://mxr.mozilla.org/mozilla/source/tools/tinderbox-configs/firefox/win32/ (and macosx/ and linux/)
Target Milestone: mozilla1.9.1a2 → ---
Comment 12•16 years ago
|
||
(In reply to comment #11) > You can clobber yourself by modifying the CLOBBER file: > http://mxr.mozilla.org/mozilla/source/tools/tinderbox-configs/firefox/win32/ > (and macosx/ and linux/) That only works for CVS, FYI.
Assignee | ||
Comment 13•16 years ago
|
||
Confirmed that "Check for Updates" on Windows is pulling Gecko/20080818113442, and it created signons.sqlite as expected.
Status: RESOLVED → VERIFIED
Updated•16 years ago
|
Hardware: PC → All
Target Milestone: --- → mozilla1.9.1a2
Comment 14•16 years ago
|
||
This bug re-appears with hightly 20090116
Assignee | ||
Comment 15•16 years ago
|
||
Nope, current nightly works fine for me. This bug has been closed for 5 months. If you can reproduce your problem, open a new bug with logging from https://wiki.mozilla.org/Firefox:Password_Manager_Debugging
Comment 16•16 years ago
|
||
I think he is using SeaMonkey trunk builds. 20090116 is when the toolkit login manager landed in comm-central (SeaMonkey+Thunderbird). In any case Igor, please file a new bug if you can come up with a consistent test case.
You need to log in
before you can comment on or make changes to this bug.
Description
•