Switch to mailServices.js in Thunderbird js files

ASSIGNED
Assigned to

Status

Thunderbird
General
P5
enhancement
ASSIGNED
6 years ago
2 years ago

People

(Reporter: standard8, Assigned: aryx)

Tracking

(Depends on: 1 bug, {meta})

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [please do not close])

(Reporter)

Description

6 years ago
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.

Comment 1

6 years ago
Will this cover both /mail and /mailnews?
(Reporter)

Comment 2

6 years ago
Yes, but you probably want to file the mailnews parts under MailNews Core.

Updated

6 years ago
Status: NEW → ASSIGNED
Priority: -- → P5

Updated

6 years ago
Depends on: 721517

Updated

6 years ago
Depends on: 722187

Updated

6 years ago
Depends on: 726737

Updated

6 years ago
Depends on: 732811

Updated

6 years ago
Severity: normal → enhancement
See Also: → bug 720356

Updated

6 years ago
Depends on: 736870

Updated

6 years ago
Depends on: 738810

Updated

6 years ago
Depends on: 738991

Updated

6 years ago
Depends on: 739051

Updated

6 years ago
Depends on: 748965

Updated

6 years ago
Depends on: 754579

Updated

6 years ago
Depends on: 758915

Updated

6 years ago
Depends on: 758951

Updated

6 years ago
Depends on: 758979

Updated

6 years ago
Depends on: 763106

Updated

6 years ago
Depends on: 776705

Updated

5 years ago
Depends on: 804004

Updated

5 years ago
Depends on: 804008
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Comment 3

5 years ago
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.

Comment 5

5 years ago
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]

Updated

5 years ago
Status: REOPENED → ASSIGNED

Updated

5 years ago
Depends on: 819770
Depends on: 821619
Depends on: 852690

Updated

5 years ago
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.

Updated

5 years ago
Depends on: 863617

Updated

2 years ago
See Also: → bug 1189290
You need to log in before you can comment on or make changes to this bug.