Review Request: Asynchronously show the KSNI context menu to avoid crashes

Aurélien Gâteau agateau at kde.org
Wed Dec 1 18:06:20 CET 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6020/
-----------------------------------------------------------

Review request for Plasma and Marco Martin.


Summary
-------

In some situations (KPackageKit KSNI comes to mind), the KSNI is deleted when the user right clicks it. While the behavior of that application is questionable, it should not crash the workspace.

The change here ensures that the menu is showing asynchronously. It prevents a random crash which I uncovered after fixing the original issue of bug #241562 (see http://reviewboard.kde.org/r/6022/ )


This addresses bug 241562.
    https://bugs.kde.org/show_bug.cgi?id=241562


Diffs
-----

  trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/protocols/dbussystemtray/dbussystemtraywidget.cpp 1201811 

Diff: http://svn.reviewboard.kde.org/r/6020/diff


Testing
-------

Right-clicking KPackageKit KSNI would cause a crash one time out of three before this patch. It does not crash anymore now.


Thanks,

Aurélien

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20101201/6663ffaf/attachment.htm 


More information about the Plasma-devel mailing list