Closed Bug 1438788 Opened 2 years ago Closed 2 years ago
Remove broken and unused Passwordmanager from Sea
This one is broken: chrome://communicator/content/passwordManager.xul We don't have most locales for it in the tree. It could never have worked for a long long time as far as I see. The one from toolkit: chrome://passwordmgr/content/passwordManager.xul located in: mozilla/toolkit/components/passwordmgr/content still works fine. So we should just get rid of the bad one and only when/if the one in toolkit gets removed fork this one then.
[Approval Request Comment] Regression caused by (bug #): -- User impact if declined: just dead code Testing completed (on m-c, etc.): c-r 2.53 Risk to taking this patch (and alternatives if risky): trivial String changes made by this patch: only removals with no direct l10n impact.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/comm-central/rev/4c4a6e0505ac Remove unused and broken Password Manager in SeaMonkey. r=stefanh
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Comment on attachment 8951541 [details] [diff] [review] 1438788-removepasswordmanager.patch a=me
Already in beta. Wontfix for release. https://hg.mozilla.org/releases/comm-esr52/rev/058a2693dd9184447466f01f9717b303ab64a3cb
You need to log in before you can comment on or make changes to this bug.