Port bug 1353542 |Switch to async/await from Task.jsm/yield| to C-C (mailnews, char, calendar)

NEW
Unassigned

Status

MailNews Core
Backend
P1
normal
a year ago
a year ago

People

(Reporter: Jorg K (GMT+2) (bustage-fix only, NI for urgent reviews), Unassigned, NeedInfo)

Tracking

({perf})

Firefox Tracking Flags

(Not tracked)

Details

+++ This bug was initially created as a clone of Bug #1353542 +++

There are a few uses in mailnews:
https://dxr.mozilla.org/comm-central/search?q=task.spawn&redirect=false
https://dxr.mozilla.org/comm-central/search?q=task.async&redirect=false

Florian, would you like to assist here?
Flags: needinfo?(florian)
See Also: → bug 1364677
I'm curious, how long has task.jsm been in use in Thunderbird?

And since this is being billed as a perf issue, I wonder what we should expect to see in Thunderbird.
Keywords: perf
(In reply to Jorg K (GMT+2) from comment #0)
> +++ This bug was initially created as a clone of Bug #1353542 +++
> 
> There are a few uses in mailnews:
> https://dxr.mozilla.org/comm-central/search?q=task.spawn&redirect=false
> https://dxr.mozilla.org/comm-central/search?q=task.async&redirect=false
> 
> Florian, would you like to assist here?

I'm happy to help with running my auto-conversion scripts, but the process I described at bug 1353542 comment 8 has a manual part (reviewing the remaining generators and whitelisting the actual generators) which I'm unlikely to have time to do for mailnews (nor devtools or mobile/ fwiw).
Flags: needinfo?(florian)
(In reply to Wayne Mery (:wsmwk, NI for questions) from comment #1)
> I'm curious, how long has task.jsm been in use in Thunderbird?

It looks like there's very little use of it in Thunderbird.

> And since this is being billed as a perf issue, I wonder what we should
> expect to see in Thunderbird.

Don't expect anything impressive in terms of performance, on Firefox it was around a 1% win (ie. not very significantly above the noise margin on Talos). A great side benefit though is that it makes the stacks we see in profiles much more readable.
Maybe we can get Aceman interested in this little clean-up project ;-)
Flags: needinfo?(acelists)
You need to log in before you can comment on or make changes to this bug.