diff options
author | Ray Strode <rstrode@redhat.com> | 2015-09-09 15:24:31 -0400 |
---|---|---|
committer | Ray Strode <rstrode@redhat.com> | 2015-09-10 09:57:23 -0400 |
commit | abb46853f824a004e0d7f58b26e068589b121d6b (patch) | |
tree | dc218e81d8a010a9999db515dfcbf9f774b1867d /daemon/gdm-local-display-factory.c | |
parent | 3499aab05e60bf4fe2b9694b9ce9bde7bf2aa54a (diff) | |
download | gdm-abb46853f824a004e0d7f58b26e068589b121d6b.tar.gz |
local-display-factory: track autologin displays
At the moment the display-factory only tracks displays it adds itself.
autologin displays aren't added by the display-factory, but by the
manager.
This commit makes sure the display-factory is notified about the
autologin display, so when it finishes a login screen can be put in
its place.
https://bugzilla.gnome.org/show_bug.cgi?id=749418
Diffstat (limited to 'daemon/gdm-local-display-factory.c')
-rw-r--r-- | daemon/gdm-local-display-factory.c | 61 |
1 files changed, 56 insertions, 5 deletions
diff --git a/daemon/gdm-local-display-factory.c b/daemon/gdm-local-display-factory.c index 676303cb..c7943131 100644 --- a/daemon/gdm-local-display-factory.c +++ b/daemon/gdm-local-display-factory.c @@ -183,11 +183,6 @@ store_display (GdmLocalDisplayFactory *factory, { GdmDisplayStore *store; - g_signal_connect (display, "notify::status", - G_CALLBACK (on_display_status_changed), factory); - - g_object_weak_ref (G_OBJECT (display), (GWeakNotify)on_display_disposed, factory); - store = gdm_display_factory_get_display_store (GDM_DISPLAY_FACTORY (factory)); gdm_display_store_add (store, display); } @@ -544,13 +539,59 @@ gdm_local_display_factory_stop_monitor (GdmLocalDisplayFactory *factory) } } +static void +on_display_added (GdmDisplayStore *display_store, + const char *id, + GdmLocalDisplayFactory *factory) +{ + GdmDisplay *display; + + display = gdm_display_store_lookup (display_store, id); + + if (display != NULL) { + g_signal_connect (display, "notify::status", + G_CALLBACK (on_display_status_changed), factory); + + g_object_weak_ref (G_OBJECT (display), (GWeakNotify)on_display_disposed, factory); + } +} + +static void +on_display_removed (GdmDisplayStore *display_store, + const char *id, + GdmLocalDisplayFactory *factory) +{ + GdmDisplay *display; + + display = gdm_display_store_lookup (display_store, id); + + if (display != NULL) { + g_signal_handlers_disconnect_by_func (display, G_CALLBACK (on_display_status_changed), factory); + g_object_weak_unref (G_OBJECT (display), (GWeakNotify)on_display_disposed, factory); + + } +} + static gboolean gdm_local_display_factory_start (GdmDisplayFactory *base_factory) { GdmLocalDisplayFactory *factory = GDM_LOCAL_DISPLAY_FACTORY (base_factory); + GdmDisplayStore *store; g_return_val_if_fail (GDM_IS_LOCAL_DISPLAY_FACTORY (factory), FALSE); + store = gdm_display_factory_get_display_store (GDM_DISPLAY_FACTORY (factory)); + + g_signal_connect (G_OBJECT (store), + "display-added", + G_CALLBACK (on_display_added), + factory); + + g_signal_connect (G_OBJECT (store), + "display-removed", + G_CALLBACK (on_display_removed), + factory); + gdm_local_display_factory_start_monitor (factory); return gdm_local_display_factory_sync_seats (factory); } @@ -559,11 +600,21 @@ static gboolean gdm_local_display_factory_stop (GdmDisplayFactory *base_factory) { GdmLocalDisplayFactory *factory = GDM_LOCAL_DISPLAY_FACTORY (base_factory); + GdmDisplayStore *store; g_return_val_if_fail (GDM_IS_LOCAL_DISPLAY_FACTORY (factory), FALSE); gdm_local_display_factory_stop_monitor (factory); + store = gdm_display_factory_get_display_store (GDM_DISPLAY_FACTORY (factory)); + + g_signal_handlers_disconnect_by_func (G_OBJECT (store), + G_CALLBACK (on_display_added), + factory); + g_signal_handlers_disconnect_by_func (G_OBJECT (store), + G_CALLBACK (on_display_removed), + factory); + return TRUE; } |