[Differential] [Request, 20 lines] D2931: Destroy DebugConsole on hide of QWindow

[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
  The quit button in the DebugConsole is connected to deleteLater on the
  DebugConsole. This is to clean up resources like the input event filter
  which is rather expensive to have running.
  
  When closed through the window decoration the DebugConsole window only
  got hidden but not destroyed. Resulting in the input filter to continue
  processing.
  
  This change ensures that the DebugConsole gets properly destroyed once
  the window gets hidden.
  
  BUG: 369858
  FIXED-IN: 5.8.1

REPOSITORY
  rKWIN KWin

BRANCH
  debug-console-destroy-5.8

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

AFFECTED FILES
  autotests/integration/debug_console_test.cpp
  debug_console.cpp
  debug_console.h

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/20161004/51627f88/attachment.html>