[gnome-shell] workspace: Fix an incorrect signal check
- From: Florian Müllner <fmuellner src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] workspace: Fix an incorrect signal check
- Date: Fri, 20 Sep 2019 10:25:31 +0000 (UTC)
commit 48adb2ef4b3fa97c59952ca45918268e1e2d0160
Author: Jonas Dreßler <verdre v0yd nl>
Date: Fri Sep 20 11:37:26 2019 +0200
workspace: Fix an incorrect signal check
While commit 3094f863 was intended to cancel the ongoing idle hide
timeout before we start a new one, a mistake slipped in there while
rebasing: Obviously we should check if the signal id is NOT 0 here.
This didn't prevent timeouts being started while old ones are still
running and did override `this._idleHideOverlayId`, which caused the old
timeouts to run indefinitely after an overlay actor was destroyed
because we fail early (and don't return TRUE) in `_idleHideOverlay()`.
https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/735
js/ui/workspace.js | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
---
diff --git a/js/ui/workspace.js b/js/ui/workspace.js
index 3177928320..a93b19add9 100644
--- a/js/ui/workspace.js
+++ b/js/ui/workspace.js
@@ -684,7 +684,7 @@ var WindowOverlay = class {
}
_onHideChrome() {
- if (this._idleHideOverlayId == 0)
+ if (this._idleHideOverlayId > 0)
GLib.source_remove(this._idleHideOverlayId);
this._idleHideOverlayId = GLib.timeout_add(GLib.PRIORITY_DEFAULT, WINDOW_OVERLAY_IDLE_HIDE_TIMEOUT,
this._idleHideOverlay.bind(this));
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]