[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