Closed Bug 720358 Opened 12 years ago Closed 5 years ago

Switch to mailServices.js in Thunderbird js files

Categories

(Thunderbird :: General, task, P5)

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.
Will this cover both /mail and /mailnews?
Yes, but you probably want to file the mailnews parts under MailNews Core.
Status: NEW → ASSIGNED
Priority: -- → P5
Depends on: 721517
Depends on: 722187
Depends on: 726737
Depends on: 732811
Severity: normal → enhancement
See Also: → 720356
Depends on: 736870
Depends on: 738810
Depends on: 738991
Depends on: 739051
Depends on: 748965
Depends on: 754579
Depends on: 758915
Depends on: 758951
Depends on: 758979
Depends on: 763106
Depends on: 776705
Depends on: 804004
Depends on: 804008
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.
(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]
Status: REOPENED → ASSIGNED
Depends on: 819770
Assignee: acelists → archaeopteryx
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
(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.
Depends on: 863617
See Also: → 1189290
Blocks: 1508415
No longer depends on: 819770
No longer blocks: 1508415
Depends on: 1508415
Status: ASSIGNED → RESOLVED
Closed: 12 years ago5 years ago
Resolution: --- → FIXED
Type: enhancement → task
You need to log in before you can comment on or make changes to this bug.