ShellClients: Wait and reveal in sync on session start

Author: leolost2605

src/ShellClients/HideTracker.vala

@@ -147,7 +147,13 @@ public class Gala.HideTracker : Object {
         });
     }
 
-    private void update_overlap () {
+    /**
+     * Immediately updates the overlap. Usually {@link schedule_update} should be used instead which
+     * internally calls this but throttles it since it's a somewhat expensive operation.
+     * On very rare use cases however it is required to update immediately. In that case you can call
+     * this directly.
+     */
+    public void update_overlap () {
         overlap = false;
         focus_overlap = false;
         focus_maximized_overlap = false;

src/ShellClients/PanelClone.vala

@@ -70,13 +70,13 @@ public class Gala.PanelClone : Object {
         update_visible ();
         update_clone_position ();
 
-        Idle.add_once (() => {
-            if (hide_mode == NEVER) {
-                show ();
-            } else {
-                hide_tracker.schedule_update ();
-            }
-        });
+        var manager = ShellClientsManager.get_instance ();
+
+        if (manager.is_waiting) {
+            manager.notify["is-waiting"].connect (on_ready);
+        } else {
+            Idle.add_once (on_ready);
+        }
     }
 
     private void update_visible () {
@@ -142,7 +142,7 @@ public class Gala.PanelClone : Object {
     }
 
     private void show () {
-        if (!panel_hidden) {
+        if (!panel_hidden || ShellClientsManager.get_instance ().is_waiting) {
             return;
         }
 
@@ -160,4 +160,12 @@ public class Gala.PanelClone : Object {
             return Source.REMOVE;
         });
     }
+
+    private void on_ready () {
+        if (hide_mode == NEVER) {
+            show ();
+        } else {
+            hide_tracker.update_overlap ();
+        }
+    }
 }

src/ShellClients/ShellClientsManager.vala

@@ -20,6 +20,8 @@ public class Gala.ShellClientsManager : Object {
         return instance;
     }
 
+    public bool is_waiting { get; private set; default = true; }
+
     public WindowManager wm { get; construct; }
 
     private NotificationsClient notifications_client;
@@ -105,6 +107,10 @@ public class Gala.ShellClientsManager : Object {
         }
     }
 
+    public void notify_ready () {
+        Timeout.add_once (1500, () => is_waiting = false);
+    }
+
     public void make_dock (Meta.Window window) {
         if (Meta.Util.is_wayland_compositor ()) {
             make_dock_wayland (window);

src/WindowManager.vala

@@ -367,13 +367,13 @@ namespace Gala {
 
             stage.show ();
 
-            Idle.add (() => {
+            Idle.add_once (() => {
                 // let the session manager move to the next phase
 #if WITH_SYSTEMD
                 Systemd.Daemon.notify (true, "READY=1");
 #endif
-                display.get_context ().notify_ready ();
-                return GLib.Source.REMOVE;
+                get_display ().get_context ().notify_ready ();
+                ShellClientsManager.get_instance ().notify_ready ();
             });
         }