Closed Bug 2714 Opened 27 years ago Closed 26 years ago

js mailfilters and newsfilters get slow

Categories

(SeaMonkey :: MailNews: Message Display, defect, P2)

All
IRIX
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: bugzilla, Assigned: warrensomebody)

Details

(This bug imported from BugSplat, Netscape's internal bugsystem. It was known there as bug #113491 http://scopus.netscape.com/bugsplat/show_bug.cgi?id=113491 Imported into Bugzilla on 01/27/99 13:03) There is a problem somewhere in the js mailfilter code (or the js code itself), in which some sort of memory leak causes the jsgc to take longer and longer and longer and time goes by. In 4.0x, after a few days this gc problem hangs the mozilla thread for several minutes each time the filters run. It turns out that it scales with 1) the number of times you download mail, and 2) the number of times you open a newsgroup. Since 4.5 downloads mail every couple minutes, the slowdown that used to take days now takes hours. Also, related to this, every time you download mail or open a newsgroup, the filters.js file is re-read, whether it needs it or not. This is also slow. I have a fix in the 4.5 tree, which moves the js mailfilter code out of the preferences context and into its own context. This fixes both of the above problems. I'd like to get it into 4.1 (which will take a little merging). I'm assigning this to toshok, since I guess he owns js mailfilters. I can do the work, just reassign it to me if that's okay.
In the absence of an escalation, I don't think we should take a change for 4.06/4.1. I'd be happy to review your fix and take it into Nova, though. Does moving JS mail filters into their own context really fix the leak, or does it just move it around?
Since this aint a regression I am marking this TFV -> 5.0
Assignee: toshok → warren
Hardware: All
reassign to warren since ducarroz doesn't have a bugzilla account yet.
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
This old thing? This is long fixed..
Status: RESOLVED → VERIFIED
super. Marking verified then.
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.