Closed
Bug 2714
Opened 27 years ago
Closed 26 years ago
js mailfilters and newsfilters get slow
Categories
(SeaMonkey :: MailNews: Message Display, defect, P2)
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.
Comment 1•26 years ago
|
||
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?
Comment 2•26 years ago
|
||
Since this aint a regression I am marking this TFV -> 5.0
reassign to warren since ducarroz doesn't have a bugzilla account yet.
Reporter | ||
Updated•26 years ago
|
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 4•26 years ago
|
||
This old thing? This is long fixed..
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•