Closed Bug 508980 Opened 16 years ago Closed 1 year ago

folder pane should use nsIMsgFolderListener instead of nsIFolderListener where possible

Categories

(Thunderbird :: Folder and Message Lists, defect)

defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: asuth, Unassigned)

Details

(Keywords: perf)

Because the folder pane uses nsIFolderListener, it receives notifications about messages when it does not care about messages at all. While that works for OnItemAdded/OnItemRemoved, we will probably need to keep OnItemIntPropertyChanged and I guess OnItemEvent around (although it's not immediately clear what the latter is used for). However, we can mask down to those notifications.
Flags: blocking-thunderbird3?
Whoops. This should not block.
Flags: blocking-thunderbird3?
I have a patch for this somewhere, and I think a bug for it...
(In reply to David :Bienvenu from comment #2) > I have a patch for this somewhere, and I think a bug for it... David, I searched for such a bug, and failed. Is this already fixed?
Flags: needinfo?(mozilla)
Keywords: perf
presumably this is neither significant nor important performance-wise
Severity: normal → minor
(In reply to Wayne Mery (:wsmwk) from comment #3) > (In reply to David :Bienvenu from comment #2) > > I have a patch for this somewhere, and I think a bug for it... > > David, I searched for such a bug, and failed. Is this already fixed? This isn't fixed, the code is in the same state. There would potentially be a small impact on performance but it'd probably wouldn't be noticeable at a guess.
Flags: needinfo?(mozilla)
Severity: minor → S3

Made obsolete by 115?

Severity: S3 → S4
Flags: needinfo?(geoff)

Not really, but I sure don't care enough to keep a 15 year old bug about it.

Status: NEW → RESOLVED
Closed: 1 year ago
Flags: needinfo?(geoff)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.