926859 - Calling PopupNotification.remove() from notification constructor leaves floating popup

Status: NEW

(Toolkit :: PopupNotifications and Notification Bars, defect)

Description

[Georg Fritzsche [:gfritzsche]]

Attachment: floating anchor-less popup (ignore popup contents)

Calling |PopupNotification.remove(this.notification)| from a notification constructor (e.g. [1]), removes the notification icon/anchor in the location bar but leaves the popup floating around.

[1] http://hg.mozilla.org/mozilla-central/annotate/0e26e6f12ad9/browser/base/content/urlbarBindings.xml#l1569

Comment 1

[Georg Fritzsche [:gfritzsche]]

The same problem occurs when trying to remove() the notification from the event callback, which is fired via:

PopupNotifications_fireCallback: http://hg.mozilla.org/mozilla-central/annotate/0e26e6f12ad9/toolkit/modules/PopupNotifications.jsm#l750
PopupNotifications_showPanel: http://hg.mozilla.org/mozilla-central/annotate/0e26e6f12ad9/toolkit/modules/PopupNotifications.jsm#l555

Comment 2

[Georg Fritzsche [:gfritzsche]]

This is only an issue when trying to remove() the notification from the sync "showing" callback or it's constructor. When delaying the removal using e.g. setTimeout(f,0) it works fine.

Given that, bug 927177 should resolve this if the callbacks can be made async.

Comment 3

[Matthew N. [:MattN]]

Bulk move to Toolkit::Notifications and Alerts

Filter on notifications-and-alerts-component.