[gnome-shell] workspace: Don't do checks in animateAllocation()



commit ec4b9b8894ceb3a81a25245584f1ae236c6dbf2e
Author: Jonas Dreßler <verdre v0yd nl>
Date:   Thu Feb 25 11:57:22 2021 +0100

    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 3337b2ed06..e78630df49 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);


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