D24542: explicitly place fullscreen windows over fullscreen

[Marco Martin]

mart created this revision.
mart added reviewers: KWin, Plasma.
Herald added a project: KWin.
Herald added a subscriber: kwin.
mart requested review of this revision.

REVISION SUMMARY
  on X11, setFullScreen is always called after Placement::place()
  so they always have the correct geometry.
  on wayland, the window if is shown directly as fullscren,
  is set fullscreen in init() then place() is evecuted, potentially
  moving it to a wrong position and potentially even size (which happens
  with maximiziong placement strategy)
  so instead of place() the client needs to be explicitly set at
  fullscreen geometry

TEST PLAN
  fullscreen windows always appear with the proper geometry
  autotests still pass

REPOSITORY
  R108 KWin

BRANCH
  phab/fullscreen

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

AFFECTED FILES
  xdgshellclient.cpp

To: mart, #kwin, #plasma
Cc: kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20191010/716f043e/attachment.html>