[gnome-shell/wip/rstrode/rhel-8.0.0: 35/50] messageList: stop syncing if closeButton has been destroyed

[Ray Strode <halfline src gnome org>]

commit 72e2936f2085d930cffee70f957fca909e240de4
Author: Marco Trevisan (Treviño) <mail 3v1n0 net>
Date:   Wed Jul 4 15:56:25 2018 +0200

    messageList: stop syncing if closeButton has been destroyed
    
    The _sync function for Message only updates the close button visibility,
    so we can safely stop doing that if the close button get get destroyed earlier
    (as it happens when clicking on it).
    
    https://bugzilla.gnome.org/show_bug.cgi?id=791233

 js/ui/messageList.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
---
diff --git a/js/ui/messageList.js b/js/ui/messageList.js
index aff201ed6..2d397c1d7 100644
--- a/js/ui/messageList.js
+++ b/js/ui/messageList.js
@@ -362,7 +362,8 @@ var Message = new Lang.Class({
         this.setBody(body);
 
         this._closeButton.connect('clicked', this.close.bind(this));
-        this.actor.connect('notify::hover', this._sync.bind(this));
+        let actorHoverId = this.actor.connect('notify::hover', this._sync.bind(this));
+        this._closeButton.connect('destroy', this.actor.disconnect.bind(this.actor, actorHoverId));
         this.actor.connect('clicked', this._onClicked.bind(this));
         this.actor.connect('destroy', this._onDestroy.bind(this));
         this._sync();