From cd95ad220fd4e37fba23c7f458da980f92825e4f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ole=20J=C3=B8rgen=20Br=C3=B8nner?= <olejorgenb@yahoo.no>
Date: Sun, 5 May 2019 15:21:22 +0200
Subject: [PATCH] Don't resize windows recursively (can cause window freezes)

A freeze can be triggered like this: (might have to run the resize twice)

    metaWindow.connect('size-changed', () => {
        let f = metaWindow.get_frame_rect()
        metaWindow.move_resize_frame(true, f.x, f.y, f.width, 500)
    })
    let f = metaWindow.get_frame_rect()
    metaWindow.move_resize_frame(true, f.x, f.y, f.width, f.height-50)

Fixes #73
---
 tiling.js | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/tiling.js b/tiling.js
index 87c837d..07217da 100644
--- a/tiling.js
+++ b/tiling.js
@@ -438,6 +438,17 @@ class Space extends Array {
         }
     }
 
+    queueLayout() {
+        if (this._layoutQueued)
+            return;
+
+        this._layoutQueued = true;
+        Meta.later_add(Meta.LaterType.RESIZE, () => {
+            this._layoutQueued = false;
+            this.layout();
+        });
+    }
+
     // Space.prototype.isVisible = function
     isVisible(metaWindow) {
         let clone = metaWindow.clone;
@@ -1773,6 +1784,7 @@ function resizeHandler(metaWindow) {
     let selected = metaWindow === space.selectedWindow;
 
     if (inGrab) {
+        // OK to layout directly from size-changed signal since layout wont resize the grab window
         space.layout(false);
     } else if (!space._inLayout && needLayout) {
         // Restore window position when eg. exiting fullscreen
@@ -1780,7 +1792,8 @@ function resizeHandler(metaWindow) {
             && move_to(space, metaWindow, {
                 x: metaWindow.get_frame_rect().x - space.monitor.x});
 
-        space.layout();
+        // Resizing from within a size-changed signal is troube (#73). Queue instead.
+        space.queueLayout();
     }
 }