64947 - remove nsStatusBarBiffManager, create nsIBiffHandler, nsBiffManager, nsPlaySoundBiffHandler, nsStatusBarBiffHandler

Status: NEW

(MailNews Core :: Backend, defect)

Description

[(not reading, please use [e-mail] instead)]

We should have one folder listener for biff, say nsBiffManager.  We'll create a
nsIBiffHandler interface that has a single doBiff() method.

nsBiffManager would be based on nsStatusBarBiffManager.cpp, but the code to
update the UI would be moved out into nsStatusBarBiffHandler.

nsPlaySoundBiffHandler, nsStatusBarBiffHandler, (etc) would implement the
nsIBiffHandler interface and register themselves as members of the
"biff-handler" category.

on new mail, nsBiffManager would enumerate through all the services that
register themselves as members of a "biff-handler" category, QI them to
nsIBiffHandler, and call DoBiff().

to add future biff handlers, like a notification one, someone would just have to
drop in a component that had a service that implemented the nsIBiffHandler
interface that registered with the "biff-handler" interface.

Comment 1

[Sheela Ravindran]

change qa contact ->fenella

Comment 2

[fenella]

Assign to Sheelar

Comment 3

[Sheela Ravindran]

adding this to main biff tracking bug

Comment 4

[(not reading, please use [e-mail] instead)]

sorry for the spam.  making bugzilla reflect reality as I'm not working on these bugs.  filter on FOOBARCHEESE to remove these in bulk.

Comment 5

[Serge Gautherie (:sgautherie)]

Filter on "Nobody_NScomTLD_20080620"