Closed Bug 1660705 Opened 4 years ago Closed 4 years ago

OfflineStartup.jsm can be loaded later

Categories

(MailNews Core :: General, defect)

Product:
MailNews Core
Component:
General
Type:
defect

Tracking

(thunderbird_esr78 wontfix, thunderbird81 wontfix)

Status:
RESOLVED FIXED
Milestone:
82 Branch
Tracking Flags:
Tracking Status
thunderbird_esr78 --- wontfix
thunderbird81 --- wontfix

People

(Reporter: standard8, Assigned: standard8)

References

Details

(Keywords: perf)

Attachments

(1 file)

Bug 1660705 - Change offline startup to trigger from profile-after-change rather than app-startup, since that's what it does anyway. r?mkmelin!
47 bytes, text/x-phabricator-request
Details | Review

Currently the components.conf file for OfflineStartup.jsm specifies that it should be loaded at app-startup.

However, all it does at that point is to add a listener to for profile-after-change. Hence it would be more efficient if it was just loaded on profile-after-change.

Blocks: 1660706
Status: NEW → ASSIGNED
status-thunderbird_esr78: --- → affected
Target Milestone: --- → 81 Branch

Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/comm-central/rev/230adf5285d3
Change offline startup to trigger from profile-after-change rather than app-startup, since that's what it does anyway. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: 81 Branch → 82 Branch

Seems like just code cleanup so I guess no uplift needs.

status-thunderbird81: --- → wontfix
status-thunderbird_esr78: affected → wontfix
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: