From ec4b9b8894ceb3a81a25245584f1ae236c6dbf2e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jonas=20Dre=C3=9Fler?= <verdre@v0yd.nl>
Date: Thu, 25 Feb 2021 11:57:22 +0100
Subject: [PATCH] workspace: Don't do checks in animateAllocation()

These checks aren't needed since Clutter should enforce this for us
already and skip the implicit transition when possible. This gets our
time spent in vfunc_allocate() down to 2.0 ms with 20 windows

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1743>
---
 js/ui/workspace.js | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/js/ui/workspace.js b/js/ui/workspace.js
index 3337b2ed0..e78630df4 100644
--- a/js/ui/workspace.js
+++ b/js/ui/workspace.js
@@ -382,13 +382,6 @@ var UnalignedLayoutStrategy = class extends LayoutStrategy {
 };
 
 function animateAllocation(actor, box) {
-    if (actor.allocation.equal(box) ||
-        actor.allocation.get_width() === 0 ||
-        actor.allocation.get_height() === 0) {
-        actor.allocate(box);
-        return null;
-    }
-
     actor.save_easing_state();
     actor.set_easing_mode(Clutter.AnimationMode.EASE_OUT_QUAD);
     actor.set_easing_duration(200);