D5746: Move code on wayland surface being created.

[David Edmundson]

davidedmundson created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  We need to setup our attachments when the wl_surface is created.
  This gets recreated when a window is hidden and shown.
  
  QPlatformSurfaceCreated is emitted once, when the backing store is created,
  not when the wl_surface is created, despite it's similar sounding name.
  
  QEvent::Show is too soon, as this is before the platform gets setVisible
  We do however, always get an expose event. QEvent::Hide weirdly comes after
  the platform has done its thing.

TEST PLAN
  See attached test on Qt 5.9
  Still has window decos after hide/show
  This needs testing on Qt 5.7 before merging

REPOSITORY
  R135 Integration for Qt applications in Plasma

BRANCH
  testing

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

AFFECTED FILES
  src/platformtheme/kwaylandintegration.cpp
  src/platformtheme/kwaylandintegration.h
  tests/CMakeLists.txt
  tests/windowdecotest.cpp

To: davidedmundson, #plasma
Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170507/c16edf54/attachment-0001.html>