Closed Bug 1780325 Opened 2 years ago Closed 2 years ago

Calendar minimonth setup is inefficient

Categories

(Calendar :: Calendar Frontend, 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

(1 file)

I've been running the profiler over start-up and discovered that there's a lot of inefficiencies in the minimonth:

  • creating a new DateTimeFormatter for each day of the month
  • conversions from JS Date to calIDateTime only to convert back again
  • finding day boxes we already have reference to
  • storing day boxes in an array instead of a Map for easy reference

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/ccaf1843fdda
Make calendar minimonth more efficient. r=aleca

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

Attachment

General

Created:
Updated:
Size: