Closed Bug 232111 Opened 16 years ago Closed 11 years ago

Wallet should use the newer nsIPrefService APIs instead of nsIPref

Categories

(SeaMonkey :: Passwords & Permissions, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED WONTFIX
seamonkey2.0a3

People

(Reporter: mikael, Unassigned)

References

()

Details

Attachments

(1 file)

nsIPref is deprecated
Status: NEW → ASSIGNED
Summary: Remove nsIPref from wallet → Wallet should use the newer nsIPrefService APIs instead of nsIPref
Assignee: mikael → dveditz
Status: ASSIGNED → NEW
Product: Browser → Seamonkey
I'm a bit of a noob here.
This seems like a hopefully fairly simple bug.

Jesse: I'm not sure this is worth it ...
AFAIK, the wallet module is not used by Firefox or Thunderbird, only Seamonkey.
Eventually the plan is to move Seamonkey to Toolkit, so the entire wallet
extension will be obsolete ...

But no, this is not a simple bug. The Wallet code is horrible and ugly - there
are far better bugs for noobs to take on. 

Take a look at <http://www.mozilla.org/contribute/hacking/first-bugs/>.
I'm just posting this here for the sake of trying to get a clean tree.

I do not think I'll fixup wallet any more than this (but maybe), so if dveditz
or someone wants to review this, that'd be cool.
Assignee: dveditz → nobody
(In reply to comment #3)
> Created an attachment (id=195723) [details]
> cleans up nsWalletService.cpp
> 
> I'm just posting this here for the sake of trying to get a clean tree.
> 
> I do not think I'll fixup wallet any more than this (but maybe), so if dveditz
> or someone wants to review this, that'd be cool.
Did you mean to ask for a review on this?
R.WontFix, after bug 473827.
Status: NEW → RESOLVED
Closed: 11 years ago
Depends on: 473827
QA Contact: privacy
Resolution: --- → WONTFIX
Target Milestone: --- → seamonkey2.0a3
You need to log in before you can comment on or make changes to this bug.