D5321: [Notifications Applet] Don't crash when containment() disappears

[Kai Uwe Broulik]

broulik created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  When deleting the panel the notifications applet is in, the containment is removed but the corona is still there and rightfully emits availableScreenRectChanged.
  This will cause us to crash when we try to access the no-longer existing containment.
  
  BUG: 378508
  FIXED-IN: 5.8.7

TEST PLAN
  Even though we no longer emit the signal when tearing down, we still do (correctly) when manually removing the panel. The containment goes away nonetheless.
  
  No longer crashes when I remove the panel. Resizing the panel still repositions notification popups.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D5321

AFFECTED FILES
  applets/notifications/lib/notificationsapplet.cpp

To: broulik, #plasma
Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170406/7eb40c05/attachment-0001.html>