diff options
| author | Ray Strode <rstrode@redhat.com> | 2014-05-16 09:38:08 -0400 |
|---|---|---|
| committer | Ray Strode <rstrode@redhat.com> | 2014-05-16 09:56:14 -0400 |
| commit | 5a96e7f97dcc33d5dbca71652994183eeef0df4f (patch) | |
| tree | b1762dcfabed985bf3922d5ee997816f19874c6b | |
| parent | 98e7f1e683b635dbaf2679583e040001c93d40ec (diff) | |
| download | gdm-5a96e7f97dcc33d5dbca71652994183eeef0df4f.tar.gz | |
transient-display: don't call unmanage from finish
GdmTransientDisplay has this code:
/* we don't restart/remanage transient displays */
gdm_display_unmanage (display);
But the display factory already makes sure transient displays aren't
restarted. Furthermore, calling unmanage from finish changes the status
of the display to UNMANAGED instead of FINISHED which causes an
assertion to get blown when the display is disposed if the user logouts
out when user switching.
This commit removes the code since it's unnecessary and actively
harmful.
https://bugzilla.gnome.org/show_bug.cgi?id=728488
| -rw-r--r-- | daemon/gdm-transient-display.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/daemon/gdm-transient-display.c b/daemon/gdm-transient-display.c index c873688d..661256e1 100644 --- a/daemon/gdm-transient-display.c +++ b/daemon/gdm-transient-display.c @@ -98,9 +98,6 @@ gdm_transient_display_finish (GdmDisplay *display) GDM_DISPLAY_CLASS (gdm_transient_display_parent_class)->finish (display); - /* we don't restart/remanage transient displays */ - gdm_display_unmanage (display); - return TRUE; } |