[gnome-shell/wip/carlosg/xwayland-startup-side-channel] windowManager: Implement MetaDisplay::init-xserver hook



commit 03f2deae91b11c511f09190857e2e5d13cb25a27
Author: Carlos Garnacho <carlosg gnome org>
Date:   Thu Nov 14 23:56:42 2019 +0100

    windowManager: Implement MetaDisplay::init-xserver hook
    
    We do spawn gsd-xsettings, and watch its name before notifying on the
    given task, so the mutter bits can proceed with X11 startup.
    
    One notable change is that we only start gsd-xsettings, instead of the
    generic gnome-session-x11-services target. We do so as we have to wait
    on a dbus name to appear in order to deem the initialization done, and
    making it all depend on gsd-xsettings seems tidier.
    
    https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/836

 js/ui/windowManager.js | 27 ++++++++++++++++++++++++---
 1 file changed, 24 insertions(+), 3 deletions(-)
---
diff --git a/js/ui/windowManager.js b/js/ui/windowManager.js
index a39f5baacd..1dfa6c7193 100644
--- a/js/ui/windowManager.js
+++ b/js/ui/windowManager.js
@@ -893,11 +893,32 @@ var WindowManager = class {
             }
         });
 
-        global.display.connect('x11-display-opened', () => {
-            Shell.util_start_systemd_unit('gnome-session-x11-services.target', 'fail');
+        global.display.connect('init-xserver', (display, task) => {
+            Shell.util_start_systemd_unit('gsd-xsettings.target', 'fail');
+            let timeoutId = -1;
+
+            /* When gsd-xsettings daemon is started, we are good to resume */
+            let watchId = Gio.DBus.session.watch_name(
+                'org.gnome.SettingsDaemon.XSettings',
+                Gio.BusNameWatcherFlags.NONE,
+                () => {
+                    task.return_boolean(true);
+                    Gio.DBus.session.unwatch_name(watchId);
+                    if (timeoutId > 0)
+                        GLib.source_remove(timeoutId);
+                },
+                null);
+
+            /* Anyway, leave this watchdog timeout so don't block indefinitely here */
+            timeoutId = GLib.timeout_add_seconds(GLib.PRIORITY_DEFAULT, 5, () => {
+                Gio.DBus.session.unwatch_name(watchId);
+                task.return_boolean(false);
+                return GLib.SOURCE_REMOVE;
+            });
+            return true;
         });
         global.display.connect('x11-display-closing', () => {
-            Shell.util_stop_systemd_unit('gnome-session-x11-services.target', 'fail');
+            Shell.util_stop_systemd_unit('gsd-xsettings.target', 'fail');
         });
 
         Main.overview.connect('showing', () => {


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