Closed Bug 1783770 Opened 2 years ago Closed 2 years ago

Miscellaneous start-up performance improvements

Categories

(Thunderbird :: General, enhancement)

enhancement

Tracking

(thunderbird_esr102 wontfix)

RESOLVED FIXED
105 Branch
Tracking Status
thunderbird_esr102 --- wontfix

People

(Reporter: darktrojan, Assigned: darktrojan)

References

Details

(Keywords: perf, perf:startup)

Attachments

(7 files)

I've got a number of patches for improving start-up performance. I don't think it's worth filing a bug for each so I'll stick them all in one bug, this one.

These two modules are loaded at profile-after-change to add some listeners/observers. AFAICT, none of the things they are listening for will happen this early, so we can initialise them later.

By delaying the start of the chat logger, we can prevent a number of modules from loading before the first window does.

Depends on D154026

These modules don't need to be loaded with the module that imports them. Some don't need loading at all.

Depends on D154027

I see no reason for this to start so early. Making this change plus some others means we can avoid loading all of the Gloda code until much later. Plus if chat is disabled we can avoid it altogether.

Depends on D154028

Attachment #9288940 - Attachment description: WIP: Bug 1783770 - Stop loading mail notification modules at profile-after-change. r=mkmelin → Bug 1783770 - Stop loading mail notification modules at profile-after-change. r=mkmelin
Attachment #9288941 - Attachment description: WIP: Bug 1783770 - Start the chat logger after the first window loads. r=freaktechnik → Bug 1783770 - Start the chat logger after the first window loads. r=freaktechnik
Attachment #9288942 - Attachment description: WIP: Bug 1783770 - Convert some module imports to lazy imports. r=#thunderbird-reviewers → Bug 1783770 - Convert some module imports to lazy imports. r=#thunderbird-reviewers
Attachment #9288943 - Attachment description: WIP: Bug 1783770 - Delay starting Gloda IM indexing until after the first window loads. r=freaktechnik → Bug 1783770 - Delay starting Gloda IM indexing until after the first window loads. r=freaktechnik
Attachment #9288944 - Attachment description: WIP: Bug 1783770 - Move Gloda constants to a separate module, part 1. r=#thunderbird-reviewers → Bug 1783770 - Move Gloda constants to a separate module, part 1. r=#thunderbird-reviewers
Attachment #9288945 - Attachment description: WIP: Bug 1783770 - Move Gloda constants to a separate module, part 2. r=#thunderbird-reviewers → Bug 1783770 - Move Gloda constants to a separate module, part 2. r=#thunderbird-reviewers
Attachment #9288946 - Attachment description: WIP: Bug 1783770 - Move Gloda constants to a separate module, part 3. r=#thunderbird-reviewers → Bug 1783770 - Move Gloda constants to a separate module, part 3. r=#thunderbird-reviewers
Keywords: leave-open
Target Milestone: --- → 105 Branch
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/17ad6ea84651
Stop loading mail notification modules at profile-after-change. r=mkmelin
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/fdf6a586175e
Start the chat logger after the first window loads. r=freaktechnik
https://hg.mozilla.org/comm-central/rev/8215c04fbfe3
Convert some module imports to lazy imports. r=mkmelin
https://hg.mozilla.org/comm-central/rev/52e1ef6ec3f3
Delay starting Gloda IM indexing until after the first window loads. r=freaktechnik
https://hg.mozilla.org/comm-central/rev/d8166ba9e8b9
Move Gloda constants to a separate module, part 1. r=mkmelin
https://hg.mozilla.org/comm-central/rev/e697b1403810
Move Gloda constants to a separate module, part 2. r=mkmelin
https://hg.mozilla.org/comm-central/rev/cbeeafe798f3
Move Gloda constants to a separate module, part 3. r=mkmelin

That's enough for now. I have some other things in mind but they can have another bug.

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Keywords: leave-open
Resolution: --- → FIXED
Regressions: 1832395
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: