[Differential] [Request, 9 lines] D1088: [krunner] Don't call KWindowSystem::setState on every event
graesslin (Martin Gräßlin)
noreply at phabricator.kde.org
Tue Mar 8 16:49:12 UTC 2016
graesslin created this revision.
graesslin added a reviewer: Plasma.
graesslin added a subscriber: Plasma.
graesslin set the repository for this revision to rPLASMAWORKSPACE Plasma Workspace.
graesslin added a project: Plasma.
Herald added a subscriber: plasma-devel.
REVISION SUMMARY
On Wayland that resulted in an endless recursion and also on X11 it's
a little bit too much to update the state on every event.
Let's only do it on the show event (as the comment states) and on
PlatformSurface created events
TEST PLAN
Doesn't crash on Wayland anymore, on X11 not tested.
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D1088
AFFECTED FILES
krunner/view.cpp
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: graesslin, Plasma
Cc: plasma-devel, Plasma
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160308/a90d8692/attachment.html>
More information about the Plasma-devel
mailing list