Closed
Bug 1705560
Opened 3 years ago
Closed 3 years ago
Cache an instance of CalItipEmailTransport instead of loading via XPCOM in getImipTransport()
Categories
(Calendar :: E-mail based Scheduling (iTIP/iMIP), task)
Calendar
E-mail based Scheduling (iTIP/iMIP)
Tracking
(thunderbird_esr78 wontfix)
RESOLVED
FIXED
90 Branch
Tracking | Status | |
---|---|---|
thunderbird_esr78 | --- | wontfix |
People
(Reporter: lasana, Assigned: lasana)
References
Details
Attachments
(1 file)
I noticed the service isn't really loaded anywhere else. Seems like the intention may have been at some point to create a new transport based on the calendar passed but really it just returns the same instance.
https://searchfox.org/comm-central/source/calendar/base/modules/utils/calProviderUtils.jsm#273
Removing the xpcom reliance will free up some other changes I need to make related to bug 1562896. Specifically I want to make CalItipEmailTransport subclassable.
Assignee | ||
Updated•3 years ago
|
Summary: In getImipTransport() cache a new instance of CalItipEmailTransport instead of loading via XPCOM → Cache an instance of CalItipEmailTransport instead of loading via XPCOM in getImipTransport()
Assignee | ||
Updated•3 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•3 years ago
|
||
Depends on D112254
Assignee | ||
Updated•3 years ago
|
Keywords: checkin-needed-tb
Target Milestone: --- → 90 Branch
Updated•3 years ago
|
status-thunderbird_esr78:
--- → wontfix
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/848ec77f6dad
Lazy load CalItipEmailTransport via js instead of xpcom. r=darktrojan
You need to log in
before you can comment on or make changes to this bug.
Description
•