D18128: Split up fullscreen able check into AbstractClient subclasses

romangg added inline comments.

INLINE COMMENTS

> zzag wrote in client.cpp:590-596
> We lose this path in ShellClient::isFullScreenable. Is there a reason for thta?

Wayland native clients can't dictate the compositor if fullscreen is possible or not. so we don't need to check if their geometry is restricted. At least the call to `AbstractClient::sizeForClientSize` in the case of ShellClient in the current version on master would imply this.

REPOSITORY
  R108 KWin

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

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