Block popup with NULL url (javascript:window.open() for instance). but

2006-02-08  Jean-François Rameau  <jframeau@cvs.gnome.org>

	* embed/mozilla/EphyBrowser.cpp: (HandleEvent):
	* src/ephy-tab.c: (popups_manager_add),(popups_manager_show):

	Block popup with NULL url (javascript:window.open() for instance).
	but don't show them when unblocking.
	Bug #155009.

1 files changed, 10 insertions, 7 deletions

diff --git a/src/ephy-tab.c b/src/ephy-tab.c
index 8a95ff477..2fa0c31aa 100644
--- a/src/ephy-tab.c
+++ b/src/ephy-tab.c
@@ -535,7 +535,7 @@ popups_manager_add (EphyTab *tab,
 
 	popup = g_new0 (PopupInfo, 1);
 
-	popup->url = g_strdup (url);
+	popup->url = (url == NULL) ? NULL : g_strdup (url);
 	popup->name = g_strdup (name);
 	popup->features = g_strdup (features);
 
@@ -651,14 +651,17 @@ popups_manager_show (PopupInfo *popup,
 	EphyEmbed *embed;
 	EphyEmbedSingle *single;
 
-	embed = ephy_tab_get_embed (tab);
-
-	single = EPHY_EMBED_SINGLE
-		(ephy_embed_shell_get_embed_single (embed_shell));
+	/* Only show popup with non NULL url */
+	if (popup->url != NULL)
+	{
+		embed = ephy_tab_get_embed (tab);
 
-	ephy_embed_single_open_window (single, embed, popup->url,
-				       popup->name, popup->features);
+		single = EPHY_EMBED_SINGLE
+			(ephy_embed_shell_get_embed_single (embed_shell));
 
+		ephy_embed_single_open_window (single, embed, popup->url,
+				popup->name, popup->features);
+	}
 	popups_manager_free_info (popup);
 }