Instead of using nsBrowserStatusHandler.js, we now need to notify PopupNotifications from each <notificationbox type="popup">.
Created attachment 716756 [details] [diff] [review] Possible patch Unfortunately removing popupNotifications from nsBrowserStatusHandler.js means that I don't have anywhere convenient to remember whether I created the notifications lazily or not, so this version always creates the notifications eagerly, since they're going to be notified in the next location change anyway. XBL isn't good at calling inherited methods so I moved as much as I could of the onLocationChange logic out to a separate method and duplicated the rest.
From Bug 842412: firstname.lastname@example.org 2013-02-18 16:36:25 PST > As per bug 839445 we need to notify the PopupNotifications object when any relevant > onLocationChange fires, not just for the current tab. [reply] [-] Comment 1 Philip Chee 2013-02-22 11:33:58 PST > Neil pushed a wallpaper bustage fix (fallout from bug 839445). > http://hg.mozilla.org/comm-central/rev/26093ce6e30c
Comment on attachment 716756 [details] [diff] [review] Possible patch [Approval Request Comment] Regression caused by (bug #): 839445 User impact if declined: PopupNotifications in background tabs linger Testing completed (on m-c, etc.): Risk to taking this patch (and alternatives if risky): String changes made by this patch: None
Comment on attachment 716756 [details] [diff] [review] Possible patch a=me