D20890: Close screen grabbing effect when screensaver starts

davidedmundson created this revision.
davidedmundson added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
davidedmundson requested review of this revision.

REVISION SUMMARY
  The screenlock fails on X11 if it can't grab the keyboard.
  
  We can't nicely solve the generic case. We can solve the common case of
  a kwin effect being active.
  
  It's not critical, arguably not even desirable to have these effects
  persist after the screen is locked through an external trigger. We can
  just close the effect early.
  
  Key grabs have to be relased early before the close animation completes
  so that the locker doesn't have a race based on animation times.
  
  It's not ideal, but no worse than the current state for not much work.
  
  BUG: 234153

TEST PLAN
  locked screen on a timer
  opened various effects

REPOSITORY
  R108 KWin

BRANCH
  origin-master (branched from master)

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

AFFECTED FILES
  CMakeLists.txt
  effects.cpp
  effects/cube/cube.cpp
  effects/desktopgrid/desktopgrid.cpp
  effects/flipswitch/flipswitch.cpp
  effects/presentwindows/presentwindows.cpp
  libkwineffects/kwineffects.h
  screenlockerwatcher.cpp
  screenlockerwatcher.h

To: davidedmundson, #kwin
Cc: kwin, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart