From 3d4ba028c4cc525da78ee966ce86a848aa09d606 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marco=20Trevisan=20=28Trevi=C3=B1o=29?= <mail@3v1n0.net>
Date: Thu, 13 Jun 2019 19:40:47 +0200
Subject: [PATCH] boxpointer: Don't trigger a relayout when updating arrow side

Changing the arrow side might need to reposition the boxpointer, however
if this happens during allocation, we don't need to trigger a new relayout since
we'd set the new allocation once _updateFlip's _reposition call is terminated,
otherwise if the position has changed, changing the boxpointer coordinates
will trigger a relayout anyways.

https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/576
---
 js/ui/boxpointer.js | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/js/ui/boxpointer.js b/js/ui/boxpointer.js
index 9939a5abf..2f4975a29 100644
--- a/js/ui/boxpointer.js
+++ b/js/ui/boxpointer.js
@@ -671,10 +671,6 @@ var BoxPointer = GObject.registerClass({
         if (this._arrowSide != arrowSide) {
             this._arrowSide = arrowSide;
             this._reposition(allocationBox);
-            Meta.later_add(Meta.LaterType.BEFORE_REDRAW, () => {
-                this.queue_relayout();
-                return false;
-            });
 
             this.emit('arrow-side-changed');
         }