393857 - Add an extended notificationbox to toolkit which handles notifications initiated by the containing browser internally

Status: RESOLVED WONTFIX

(Toolkit :: UI Widgets, defect)

Description

[Teune van Steeg]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.6) Gecko/20070725 Firefox/2.0.0.6
Build Identifier: 

Notifications of e.g. blocked popups or blocked extension installs could be handled internally by the notification box self. This will allow the code to be shared between multiple browser projects like SeaMonkey. 

   

Reproducible: Always




There is already code that handles popup blocking in browser.xml, I think it's a very natural step that when a browser is a child of a notificationbox element, the notificationbox can handle that code further for showing a notification.

Comment 1

[Teune van Steeg]

Attachment: patch

This patch creates an extended notificationbox which handles notifications for popup blocking and xpinstall blocking internally.

The notificationbox needs the attribute popupMenu set with the id of a <popup>, to show that menu on the button of the notification, when a popup is blocked.
  
This patch does not contain the icon to show on a notification for popup blocking. If I'm correct Firefox uses these icons currently: 
http://lxr.mozilla.org/mozilla/source/browser/themes/winstripe/browser/Info.png
http://lxr.mozilla.org/mozilla/source/browser/themes/pinstripe/browser/Info.png
So these should probably be moved to toolkit. (toolkit/themes/(p|w)instripe/global/icons/).

Comment 2

[Teune van Steeg]

Attachment: patch (v2)

This patch is the same as the previous patch updated with the addition of the new files to the corresponding jar.mn files (assuming the images mentioned in comment 1 are copied over). And it adds a preference used in this patch to the defaults.

Comment 3

[Teune van Steeg]

Attachment: complete patch

This patch adds internal handling of the following events to the notificationbox element:
popup blocking
xpinstall install blocking
finding missing plugins

This patch contains (minor) changes to Firefox' browser.js and toolkits pluginInstallerWizard.js. Should there also be a review from reviewers in those areas?

This patch still assumes that the images mentioned in comment 1 are copied over.

The attribute mentioned in comment 1 that needs to be set is now changed to popupmenu. 

The code concerning xpinstall blocking needs to be updated when bug 252830 gets fixed.

Comment 4

[[e-mail]]

Comment on attachment 278663 [details] [diff] [review]
complete patch

When I mentioned this to him on IRC he wasn't convinced that we could share this but let's see...

Comment 5

[Teune van Steeg]

Attachment: complete patch (v2)

The previous patch contained some errors I forgot to fix.

BTW Would it be disadvantageous to change the observer notification for "xpinstall-install-blocked" in an event dispatched from the originating document, with installInfo as a property (installInfo is introduced in bug 252830)?

In that way only one notificationbox would receive a notification to its event listener, while now every notificationbox gets a notification so you must check if the originating document is in the browser that lives in the notificationbox. 

And when this bug gets approval and Firefox decides to use this notificationbox, the only place where code is observing for "xpinstall-install-blocked" is here. (according to lxr)

Comment 6

[Teune van Steeg]

Attachment: complete patch (v3)

Update patch with changes from bug 252830.

Comment 7

[Teune van Steeg]

Attachment: complete patch (v4)

Remove some mistakes from the previous patch.

Comment 8

[Teune van Steeg]

According to gavin on IRC this is not wanted for Firefox, so I'm marking this bug as WONTFIX.

The changes needed in toolkit to be able to use the extended notificationbox in SeaMonkey will be handled in bug 400764