D15135: [wayland] Syncronise pending geometry with acked configure requests

davidedmundson added inline comments.

INLINE COMMENTS

> romangg wrote in shell_client.cpp:1210
> Instead of taking always the first one and by that looping through ignore (and remove) all configure events before the one associated with `m_lastAckedConfigureRequest` and only add a layer repaint and so on for `m_lastAckedConfigureRequest`.

Do you mean something like

  QPoint position = geom.topLeft();
  it = m_pendingConfigureRequests.begin();
  while (it != m_pendingConfigureRequests.end()) {
      if (it->serialId == m_lastAckedConfigureRequest) {
          if (position != it->positionAfterResize) {
               addLayerRepaint(geometry());
          }
          position = it->positionAfterResize;
  
          m_pendingConfigureRequests.erase(m_pendingConfigureRequests.begin(), it);
          break;
      }
      it++;
  }
  doSetGeometry(QRect(position, m_clientSize + QSize(borderLeft() + borderRight(), borderTop() + borderBottom())));

?

REPOSITORY
  R108 KWin

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

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