Open Bug 685884 Opened 13 years ago Updated 9 years ago

Lazily initialize calendar items from their components

Categories

(Calendar :: Internal Components, defect, P3)

x86_64
All

Tracking

(Not tracked)

People

(Reporter: Fallen, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: perf, Whiteboard: [patchlove])

Attachments

(1 file)

While working on bug 678343, I found that parsing a calIIcalComponent into the javascript objects we use in calEvent/calTodo costs around 30 ms on my machine. This of course adds up with a lot of events and causes UI freeze. I have some work in progress code that takes care, but its not quite ready yet. Using this bug for tracking.
Keywords: perf
Severity: normal → major
Attached patch WiP - v1 (deleted) β€” β€” Splinter Review
I *think* this is the patch I referenced. Not quite sure. Its far from ready yet. Attaching so it doesn't get lost. Anyone is invited to pick up this patch!
Priority: -- → P2
Priority: P2 → P3
This would help startup performance, eg bug 487832 ?
Flags: needinfo?(philipp)
OS: Linux → All
Whiteboard: [patchlove]
I'm sure it will, constructing items now takes 30-40ms, with this patch we could cut it down to the minimum, e.g. 10-20ms.
Flags: needinfo?(philipp)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: