D20241: [wayland] Place all toplevels before the first configure

davidedmundson created this revision.
davidedmundson added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
davidedmundson requested review of this revision.

REVISION SUMMARY
  Currently popups get positioned once at the initial configure, to set
  the correct size and again when they are mapped.
  
  Toplevels are currently only positioned when they are mapped. This works
  for all cases where the the toplevel defines its own size, but not if
  the window should have an initial size set by the placement strategy or
  window rules. Most notably the maximised placement strategy used on
  plasma mobile.
  
  Being out of sync and resizing later currently causes a positioning bug
  when plasma mobile is used with XdgShell.
  
  This patch repositions all top levels that don't have a position set
  through the plasma interface.

TEST PLAN
  Relevant unit test

REPOSITORY
  R108 KWin

BRANCH
  master

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

AFFECTED FILES
  autotests/integration/maximize_test.cpp
  shell_client.cpp

To: davidedmundson, #kwin
Cc: bshah, kwin, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart