Boris Zbarsky <email@example.com> has granted Darin Fisher <firstname.lastname@example.org>'s request for superreview: Bug 293350: Allow dynamic changes of notification callbacks (was: Don't show download status in status bar (since it shows in the DL manager)) https://bugzilla.mozilla.org/show_bug.cgi?id=293350 Attachment 194557: v1 patch https://bugzilla.mozilla.org/attachment.cgi?id=194557&action=edit ------- Additional Comments from Boris Zbarsky <email@example.com> >Index: netwerk/base/public/nsIChannel.idl >+ * listener as it becomes available. The stream listener's methods are >+ * called via an event dispatched through the event queue service on the >+ * same thread that called asyncOpen. How about just: The stream listener's methods are called on the thread that asyncOpen was called on. because if a channel runs entirely on one thread there is no pressing need for it to call these methods via an event, really... Unless you're trying to emphasize that asyncOpen will return before OnStartRequest is called, in which case we should perhaps just state that explicitly. The rest looks great. sr=bzbarsky.