D14923: Supress paste in screen locker

[David Edmundson]

davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
davidedmundson requested review of this revision.

REVISION SUMMARY
  KScreenlocker tries to clear the clipboard on load. However, klipper
  also (by default) automatically keeps the last relevant item in the
  clipboard. Whilst both parts work correctly, Plasma as a whole does not.
  
  This became a problem when we added the reveal password button as it is
  a data leak.
  
  MF doesn't want an X specific fix, which is fair enough. My earlier
  patch removing the reveal button was blocked, which is fair enough.
  That doesn't leave many nice places, but we do need to do something.
  
  This patch intercepts the paste shortcut at the window level.
  
  Doing it at a theme level doesn't easily work with QQC1 as the child
  gets key events first. With QQC2 it probably whould, but PC3 doesn't
  have the reveal button.

TEST PLAN
  Reproduced issue
  Added patch
  Couldn't reproduce
  Also can't show the context menu of the screen locker due to kwin behaviour.

REPOSITORY
  R133 KScreenLocker

BRANCH
  master

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

AFFECTED FILES
  greeter/greeterapp.cpp

To: davidedmundson, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180818/c4046e26/attachment.html>