Closed Bug 13405 Opened 21 years ago Closed 21 years ago
should cache local string bundles
I believe we can get a performance win by caching string bundles. Here is the call that creates a local string bundle: E:\raptor\mozilla\mailnews\local\src\nsLocalStringBundle.cpp(48): res = sBundleService->CreateBundle(propertyURL, locale, &sBundle);
Putting on [Perf] radar.
Pls ignore my previous comment. I am pasting the wrong things into bug reports.
Bulk add of "perf" to new keyword field. This will replace the [PERF] we were using in the Status Summary field.
Adding alecf to the cc list 'cause this was showing up as a hot spot for local msg display and pop on his quantify work. I've implemented caching for the local component in my tree. I'll probably land this tomorrow!
Changing the QA contact to suresh in lieu of lisa. I checked in a fix for this. The following operations should be faster the next time Suresh runs his performance data 1) downloading a lot of pop mail. Although there is still a big performance here we can do. We update the status bar after each message downloaded instead of after every 10 or so.. 2) delete a local message and loading the next message after it should be faster 3) parsing local folders. i.e. delete the summary file for a local mail folder then select the folder so we re-parse it.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
QA Contact: lchiang → suresh
Resolution: --- → FIXED
Suresh - check the items that mscott mentions in your next performance tests after 1/28. Compare before and after. Thanks. Scott - all platforms?
QA Contact: suresh → stephend
I'm sure this got faster with outliner, and since I don't have suresh's data before me, I'm going to mark this verified. We have outstanding bugs on message display speed and folder loading speed, both of which are actively assigned.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.