[Differential] [Request, 84 lines] D2402: [wayland] Unset focused keyboard surface when handling key event internally

[graesslin (Martin Gräßlin)]

graesslin created this revision.
graesslin added reviewers: KWin, Plasma on Wayland.
Restricted Application added subscribers: kwin, plasma-devel.
Restricted Application added projects: Plasma on Wayland, KWin.

REVISION SUMMARY
  So far when KWin intercepted a key event a leave was not sent to the
  Wayland surface currently having keyboard focus. This could result in
  the Wayland application to start repeating keys. E.g.
  
  1. application gets key press event
  2. This triggers an internal window to show
  3. key release goes to KWin internal window
  4. application starts to repeat key as there is no release
  
  With this change whenever KWin intercepts the key event e.g. due to
  
  - internal window
  - Effects grabbing key event
  - Tabbox
  
  the focused keyboard surface is set to null, thus triggering a leave
  event and the client not starting to repeat the event.

REPOSITORY
  rKWIN KWin

BRANCH
  internal-window-unset-focus-keyboard

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

AFFECTED FILES
  autotests/integration/internal_window.cpp
  autotests/integration/kwin_wayland_test.h
  autotests/integration/test_helpers.cpp
  input.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160811/61642184/attachment.html>