Closed
Bug 720358
Opened 13 years ago
Closed 6 years ago
Switch to mailServices.js in Thunderbird js files
Categories
(Thunderbird :: General, task, P5)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: standard8, Assigned: aryx)
References
Details
(Keywords: meta, Whiteboard: [please do not close])
Using mailServices.js will clean up the readability of our code a bit, and should avoid the costs the getService call each time we want a service.
Reporter | ||
Comment 2•13 years ago
|
||
Yes, but you probably want to file the mailnews parts under MailNews Core.
Severity: normal → enhancement
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Usul, why do you think this is fixed already? Are all files already converted? I have not checked yet.
Comment 4•12 years ago
|
||
(In reply to :aceman from comment #3)
> Usul, why do you think this is fixed already? Are all files already
> converted? I have not checked yet.
All the bugs linked in this meta are fixed , ergo the meta is fixed.
Yeah, but bugs are created and linked here all the time. So the list of 'depends on' is not yet final.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [please do not close]
Assignee | ||
Comment 6•12 years ago
|
||
Can this bug be closed?
/calendar/: should be converted: bug 795102
/chat/: has no such interfaces
/editor/ has no such interfaces (checked two months ago or so)
/ldap/: Didn't touch this because there was discussion about replacing it with a different implementation
/mail/ and /mailnews/: should be converted: bug 852690
/suite/: Leaving the work to the SeaMonkey people
Comment 7•12 years ago
|
||
(In reply to Archaeopteryx [:aryx] from comment #6)
> /ldap/: Didn't touch this because there was discussion about replacing it
> with a different implementation
ldap/xpcom is fair game for these kinds of changes, ldap/sdks is not.
FWIW, our thoughts about replacing it really mean ldap/sdks, not ldap altogether, although ensemble may eventually eliminate the need for ldap/xpcom altogether.
Assignee | ||
Updated•6 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago → 6 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Type: enhancement → task
You need to log in
before you can comment on or make changes to this bug.
Description
•