[gnome-shell/wip/fmuellner/more-notification-warnings] messageTray: Disconnect signals when resetting notification
- From: Florian Müllner <fmuellner src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/wip/fmuellner/more-notification-warnings] messageTray: Disconnect signals when resetting notification
- Date: Tue, 29 Jan 2019 14:38:36 +0000 (UTC)
commit cae2d7425d2f20e31df6e95018daed08984585be
Author: Florian Müllner <fmuellner gnome org>
Date: Tue Jan 29 15:30:40 2019 +0100
messageTray: Disconnect signals when resetting notification
Just like we did for the parent class in commit b57832716a, we should
disconnect any notification signals when the notification is reset
to null to avoid warnings later.
https://gitlab.gnome.org/GNOME/gnome-shell/issues/942
js/ui/messageTray.js | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
---
diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 0fd739834..97452275f 100644
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -497,9 +497,12 @@ class NotificationBanner extends Calendar.NotificationMessage {
});
}
- _onDestroy() {
- super._onDestroy();
- this.notification.disconnect(this._activatedId);
+ _disconnectNotificationSignals() {
+ super._disconnectNotificationSignals();
+
+ if (this._activatedId)
+ this.notification.disconnect(this._activatedId);
+ this._activatedId = 0;
}
_onUpdated(n, clear) {
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]