brl/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

dash: Make sure item labels are only destroyed once

Browse Source 
Labels are currently destroyed from both animateOutAndDestroy()
and destroy(), which now (rightfully) triggers a gjs warning. As
the label is created unconditionally since commit 36e5ae4a25,
mirror that and always release it in destroy() and hide it
elsewhere.

https://bugzilla.gnome.org/show_bug.cgi?id=791233
This commit is contained in:
Marco Trevisan (Treviño) 2017-12-04 20:45:35 -05:00
parent 3b4be770a0
commit d5805f9f0d
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
js/ui/dash.js
View File

@ -178,15 +178,12 @@ var DashItemContainer = new Lang.Class({
}, },
destroy: function() { destroy: function() {
if (this.label)
this.label.destroy(); this.label.destroy();
this.parent(); this.parent();
}, },
animateOutAndDestroy: function() { animateOutAndDestroy: function() {
if (this.label) this.label.hide();
this.label.destroy();
if (this.child == null) { if (this.child == null) {
this.destroy(); this.destroy();