D15893: Generalize constrainedStackingOrder for group transients and ordinary transients

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

REVISION SUMMARY
  We don't really have to have two different code paths for group
  transient and ordinary transients. For now, AbstractClient::hasTransient
  is good enough to check the relationship between potential window and
  the transient.
  
  In long term, we need to "invert" the relationship, instead of checking
  whether given parent window has a transient, we should check whether
  given transient is a transient for a given window so we can keep Deleted
  transients above their old parents.

TEST PLAN
  Manually. Will add unit tests.

REPOSITORY
  R108 KWin

BRANCH
  layers-simplify-logic

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

AFFECTED FILES
  layers.cpp

To: zzag, #kwin
Cc: kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart