Is this an issue for any existing extensions?
Yes, for Secure Login for example: https://addons.mozilla.org/firefox/4429/ https://blueimp.net/mozilla/
This was implemented temporarily on trunk, by attachment 262187 [details] [diff] [review] to bug 373253. Trunk then switched to the new Login Manager code, which replaces all the old interfaces with new ones.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Thanks, Justin for the new LoginManager in Firefox 3.0a5 - this resolves the issue here. Will there be a fix for Firefox 2 as well?
The last attachment has the relevant bits extracted from the older trunk patch I mentioned in comment #3... dveditz, is this something that can be taken on branch?
Sure, but you'll have to implement it as a new _MOZILLA_1_8_BRANCH interface instead of changing the existing iface. Sounded like Sebastian also wanted changes to nsIPasswordInternal, for password entry information, not just a change for adding passwords. A change to that interface would also have to undergo branch uglification. Ugly, but allows old extensions to keep working and updated extensions can work equally well in new and old versions by detecting whether or not the new interface exists.
I think this is essentially WONTFIX at this point. FF3 is out, and there doesn't seem to be a pressing need to get this in FF2. If someone wants to do the legwork to polish up the proof-of-concept here for the 1.8 branch, and write a set of tests for it, then I *might* be willing to take it... But that seems unlikely, and there's not much point in keeping the bug around just waiting for FF2 support to officially EOL.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.