[evolution/webkit: 23/154] Don't destroy EAttachmentBar when clearing EMailDisplay.



commit c96acac4ddac50e63d666d48a728ec635b112c54
Author: Dan VrÃtil <dvratil redhat com>
Date:   Wed Sep 14 12:25:39 2011 +0200

    Don't destroy EAttachmentBar when clearing EMailDisplay.
    
    EAttachmentBar is owned by EMFormatHTMLDisplay, not by EMailDisplay.

 mail/e-mail-display.c |   43 +++++++++++++++++++++++++++++--------------
 1 files changed, 29 insertions(+), 14 deletions(-)
---
diff --git a/mail/e-mail-display.c b/mail/e-mail-display.c
index d96cacc..b71446b 100644
--- a/mail/e-mail-display.c
+++ b/mail/e-mail-display.c
@@ -52,6 +52,8 @@ struct _EMailDisplayPrivate {
 	EMFormatHTML *formatter;
 
 	EMailDisplayMode display_mode;
+
+	GList *webviews;
 };
 
 enum {
@@ -226,6 +228,11 @@ mail_display_dispose (GObject *object)
 		priv->searchbar = NULL;
 	}
 
+	if (priv->webviews) {
+		g_list_free (priv->webviews);
+		priv->webviews = NULL;
+	}
+
 	/* Chain up to parent's dispose() method. */
 	G_OBJECT_CLASS (parent_class)->dispose (object);
 }
@@ -458,7 +465,8 @@ mail_display_insert_web_view (EMailDisplay *display,
 #if 0
 	GtkWidget *scrolled_window;
 
-	display->widgets = g_list_append (display->widgets, GTK_WIDGET (web_view));
+	display->priv->webviews = g_list_append (display->priv->webviews, web_view);
+
 	scrolled_window = gtk_scrolled_window_new (NULL, NULL);
 	g_object_set (G_OBJECT (scrolled_window),
 		"hscrollbar-policy", GTK_POLICY_NEVER,
@@ -466,16 +474,13 @@ mail_display_insert_web_view (EMailDisplay *display,
 		"shadow-type", GTK_SHADOW_NONE, NULL);
 		//"min-content-height", 300, NULL);
 	gtk_container_add (GTK_CONTAINER (scrolled_window), GTK_WIDGET (web_view));
-	display->widgets = g_list_append (display->widgets, scrolled_window);
 
 	gtk_box_pack_start (GTK_BOX (display->priv->vbox), scrolled_window, expandable, TRUE, 0);
 	gtk_widget_show_all (scrolled_window);
 #endif
 
 	gtk_box_pack_start (GTK_BOX (display->priv->vbox), GTK_WIDGET (web_view), expandable, TRUE, 0);
-	display->widgets = g_list_append (display->widgets, GTK_WIDGET (web_view));
 	gtk_widget_show_all (GTK_WIDGET (web_view));
-
 }
 
 static void
@@ -534,6 +539,7 @@ e_mail_display_init (EMailDisplay *display)
 	g_value_set_int (&margin, 10);
 	g_object_set_property (G_OBJECT (display->priv->vbox), "margin", &margin);
 
+	display->priv->webviews = NULL;
 
 	/* WEBKIT TODO: ESearchBar */
 
@@ -660,8 +666,7 @@ e_mail_display_load (EMailDisplay *display,
 	attachment_store = NULL;
 	puri = g_hash_table_lookup (emf->mail_part_table, "attachment-bar:");
 	if (puri && puri->widget_func) {
-		attachment_bar = puri->widget_func (emf, puri, NULL);
-		display->widgets = g_list_append (display->widgets, attachment_bar);
+		attachment_bar = g_object_ref (puri->widget_func (emf, puri, NULL));
 		gtk_box_pack_start (box, attachment_bar, TRUE, TRUE, 0);
 		attachment_store = e_attachment_view_get_store (E_ATTACHMENT_VIEW (attachment_bar));
 		gtk_widget_show (attachment_bar);
@@ -680,7 +685,6 @@ e_mail_display_load (EMailDisplay *display,
 				g_message ("Part %s didn't provide a valid widget, skipping!", puri->uri);
 				continue;
 			}
-			display->widgets = g_list_append (display->widgets, widget);
 			gtk_box_pack_start (box, widget, TRUE, TRUE, 0);
 			gtk_widget_show (widget);
 
@@ -713,9 +717,8 @@ e_mail_display_reload (EMailDisplay *display)
 
 	g_return_if_fail (E_IS_MAIL_DISPLAY (display));
 
-	for (iter = display->widgets; iter; iter = iter->next) {
-		if (E_IS_WEB_VIEW (iter->data))
-			e_web_view_reload (E_WEB_VIEW (iter->data));
+	for (iter = display->priv->webviews; iter; iter = iter->next) {
+		e_web_view_reload ((EWebView *) (iter->data));
 	}
 }
 
@@ -747,8 +750,18 @@ e_mail_display_set_status (EMailDisplay *display,
 	label = gtk_label_new (status);
 	gtk_box_pack_start (GTK_BOX (display->priv->vbox), label, TRUE, TRUE, 0);
 	gtk_widget_show_all (display->priv->vbox);
+}
 
-	display->widgets = g_list_append (display->widgets, label);
+static void
+remove_widget (GtkWidget *widget, gpointer user_data)
+{
+	EMailDisplay *display = user_data;
+
+	gtk_container_remove  (GTK_CONTAINER (display->priv->vbox), widget);
+
+	/* We must not destroy attachment bar, it's owned by EMFormatHTMLDisplay */
+	if  (!E_IS_ATTACHMENT_VIEW (widget))
+		gtk_widget_destroy (widget);
 }
 
 void
@@ -758,9 +771,11 @@ e_mail_display_clear (EMailDisplay *display)
 
 	gtk_widget_hide (display->priv->vbox);
 
-	g_list_free_full (display->widgets,
-			(GDestroyNotify) gtk_widget_destroy);
-	display->widgets = NULL;
+	gtk_container_foreach (GTK_CONTAINER (display->priv->vbox),
+		(GtkCallback) remove_widget, display);
+
+	g_list_free (display->priv->webviews);
+	display->priv->webviews = NULL;
 }
 
 ESearchBar*



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]