D21997: [placement] Avoid smart placement strategy with invalid client sizes

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
  To do so can in some situations lock up as the loop goes through
  different positions incrementing by client->width/height.
  
  If this is zero we can get into a stuck state.
  
  This became a more common issue due to my earlier patch that places windows
  in ShellClient::finishInit to allow the maximize placement strategy to
  set the first configure size.
  
  BUG: 408754

REPOSITORY
  R108 KWin

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

AFFECTED FILES
  placement.cpp

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