Closed Bug 1780323 Opened 2 years ago Closed 2 years ago

Address book manager initialised at start-up unnecessarily

Categories

(Thunderbird :: General, defect)

defect

Tracking

(thunderbird_esr102 wontfix)

RESOLVED FIXED
104 Branch
Tracking Status
thunderbird_esr102 --- wontfix

People

(Reporter: darktrojan, Assigned: darktrojan)

References

(Blocks 1 open bug)

Details

(Keywords: perf, perf:startup)

Attachments

(2 files)

There's no reason for the address book manager to be initialised during normal start-up, but currently it does, for two reasons:

  • the message header "edit contact" pop-up contains an address book menu list, which needs the list of address books
  • telemetry data collection happens too early

In this bug I'll fix both of these and add AddrBookManager.jsm to the list of prohibited modules in browser_startup.js to ensure it doesn't happen again.

This panel contains an address books menulist, which requires the address book manager to start when the menulist is added to the DOM. This unnecessarily happens at start-up and impacts performance. Instead, add the panel to the DOM when (if) it is first used.

This doesn't need to run at start-up. It requires some things to load that otherwise may not load (such as the address book manager), and that adversely affects start-up performance.

Depends on D152238

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/01792dda9dbd
Load message header edit contact panel on demand from a template. r=aleca
https://hg.mozilla.org/comm-central/rev/8241573875a5
Move Telemetry data collection to an idle task. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 104 Branch
See Also: → 1780993
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: