[gnome-software] shell details: Instantiate a new history dialog as needed
- From: Kalev Lember <klember src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-software] shell details: Instantiate a new history dialog as needed
- Date: Fri, 26 Jun 2015 20:40:16 +0000 (UTC)
commit 5829398bda18ca74fb7c7860359fb76e8d88792a
Author: Kalev Lember <klember redhat com>
Date: Fri Jun 26 22:29:23 2015 +0200
shell details: Instantiate a new history dialog as needed
Don't keep the history dialog around as a priv field. This makes the
code a bit smaller and more self contained; likely makes gnome-software
startup slightly faster; and finally plugs a history dialog leak - we
forgot to destroy it when shutting down.
src/gs-shell-details.c | 14 +++++---------
1 files changed, 5 insertions(+), 9 deletions(-)
---
diff --git a/src/gs-shell-details.c b/src/gs-shell-details.c
index 6067369..1737acc 100644
--- a/src/gs-shell-details.c
+++ b/src/gs-shell-details.c
@@ -48,7 +48,6 @@ struct GsShellDetailsPrivate
GCancellable *cancellable;
GsApp *app;
GsShell *shell;
- GtkWidget *history_dialog;
GtkWidget *star;
SoupSession *session;
@@ -1128,11 +1127,13 @@ static void
gs_shell_details_app_history_button_cb (GtkWidget *widget, GsShellDetails *shell_details)
{
GsShellDetailsPrivate *priv = shell_details->priv;
+ GtkWidget *dialog;
- gs_history_dialog_set_app (GS_HISTORY_DIALOG (priv->history_dialog), priv->app);
+ dialog = gs_history_dialog_new ();
+ gs_history_dialog_set_app (GS_HISTORY_DIALOG (dialog), priv->app);
- gtk_window_set_transient_for (GTK_WINDOW (priv->history_dialog), gs_shell_get_window (priv->shell));
- gtk_window_present (GTK_WINDOW (priv->history_dialog));
+ gtk_window_set_transient_for (GTK_WINDOW (dialog), gs_shell_get_window (priv->shell));
+ gtk_window_present (GTK_WINDOW (dialog));
}
/**
@@ -1238,11 +1239,6 @@ gs_shell_details_setup (GsShellDetails *shell_details,
G_CALLBACK (gs_shell_details_website_cb),
shell_details);
- /* setup history window */
- priv->history_dialog = gs_history_dialog_new ();
- g_signal_connect (priv->history_dialog, "delete-event",
- G_CALLBACK (gtk_widget_hide_on_delete), shell_details);
-
adj = gtk_scrolled_window_get_vadjustment (GTK_SCROLLED_WINDOW (priv->scrolledwindow_details));
gtk_container_set_focus_vadjustment (GTK_CONTAINER (priv->box_details), adj);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]