All users were logged out of Bugzilla on October 13th, 2018

folder pane should use nsIMsgFolderListener instead of nsIFolderListener where possible

NEW
Unassigned

Status

--
minor
9 years ago
5 years ago

People

(Reporter: asuth, Unassigned)

Tracking

({perf})

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

9 years ago
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?
(Reporter)

Comment 1

9 years ago
Whoops.  This should not block.
Flags: blocking-thunderbird3?

Comment 2

9 years ago
I have a patch for this somewhere, and I think a bug for it...

Comment 3

6 years ago
(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

Comment 4

5 years ago
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)
You need to log in before you can comment on or make changes to this bug.