D15991: [wayland] Use pending maximize mode in decoration updates

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
  The change to make maximize mode asynchronous featured the comment
  
  > Things are a bit complex with borders. Technically we
  > shouldn't update them till we get a response, but we also need to have
  > the correct geometry of the full size window in our request. For now
  > they behave as before, updating when we request the change.
  
  We call setNoBorder when we request the geometry but decoratedClient
  also checks the maximise mode, in order to follow the scheme above we
  need this to operate on the requested state not current state.
  
  X is unaffected.
  
  This fixes the borders being restored correct after maximize/restore.

TEST PLAN
  Chose a theme with visible borders
  Maximised a window and back
  They restored

REPOSITORY
  R108 KWin

BRANCH
  master

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

AFFECTED FILES
  abstract_client.cpp
  abstract_client.h
  decorations/decoratedclient.cpp
  shell_client.cpp
  shell_client.h

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