Bug 700923 – messageTray: Don't destroy a notification from within its destroy handler

After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.

Bug 700923 - messageTray: Don't destroy a notification from within its destroy handler


Summary:	messageTray: Don't destroy a notification from within its destroy handler

Reported:	2013-05-23 21:43 UTC by Jasper St. Pierre (not reading bugmail)
Modified:	2013-05-23 22:26 UTC

See Also:
GNOME target:	---
GNOME version:	---

Attachments
messageTray: Don't destroy a notification from within its destroy handler (817 bytes, patch) 2013-05-23 21:43 UTC, Jasper St. Pierre (not reading bugmail)	committed	Details \| Review

Description Jasper St. Pierre (not reading bugmail) 2013-05-23 21:43:04 UTC

Nothing but bad can come from this.

Comment 1 Jasper St. Pierre (not reading bugmail) 2013-05-23 21:43:06 UTC

Created attachment 245187 [details] [review]
messageTray: Don't destroy a notification from within its destroy handler

Comment 2 Giovanni Campagna 2013-05-23 22:23:37 UTC

Review of attachment 245187 [details] [review]:

Indeed.

Comment 3 Giovanni Campagna 2013-05-23 22:24:10 UTC

Ah, see also bug 686855 about destroying stuff from destroy handlers...

Comment 4 Jasper St. Pierre (not reading bugmail) 2013-05-23 22:26:51 UTC

Attachment 245187 [details] pushed as f73a012 - messageTray: Don't destroy a notification from within its destroy handler