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