diff --git a/internal_client.h b/internal_client.h --- a/internal_client.h +++ b/internal_client.h @@ -104,8 +104,6 @@ void updateCaption() override; private: - QRect mapFromClient(const QRect &rect) const; - QRect mapToClient(const QRect &rect) const; void createDecoration(const QRect &rect); void requestGeometry(const QRect &rect); void commitGeometry(const QRect &rect); diff --git a/internal_client.cpp b/internal_client.cpp --- a/internal_client.cpp +++ b/internal_client.cpp @@ -68,7 +68,7 @@ blockGeometryUpdates(true); commitGeometry(m_internalWindow->geometry()); updateDecoration(true); - setFrameGeometry(mapFromClient(m_internalWindow->geometry())); + setFrameGeometry(clientRectToFrameRect(m_internalWindow->geometry())); setGeometryRestore(frameGeometry()); blockGeometryUpdates(false); @@ -356,7 +356,7 @@ return; } - const QRect newClientGeometry = mapToClient(rect); + const QRect newClientGeometry = frameRectToClientRect(rect); if (m_clientSize == newClientGeometry.size()) { commitGeometry(rect); @@ -542,7 +542,7 @@ return; } - const QRect clientGeometry = mapToClient(frameGeometry()); + const QRect clientGeometry = frameRectToClientRect(frameGeometry()); AbstractClient::destroyDecoration(); setFrameGeometry(clientGeometry); } @@ -577,16 +577,6 @@ } } -QRect InternalClient::mapFromClient(const QRect &rect) const -{ - return rect + frameMargins(); -} - -QRect InternalClient::mapToClient(const QRect &rect) const -{ - return rect - frameMargins(); -} - void InternalClient::createDecoration(const QRect &rect) { KDecoration2::Decoration *decoration = Decoration::DecorationBridge::self()->createDecoration(this); @@ -608,15 +598,15 @@ const QRect oldFrameGeometry = frameGeometry(); setDecoration(decoration); - setFrameGeometry(mapFromClient(rect)); + setFrameGeometry(clientRectToFrameRect(rect)); emit geometryShapeChanged(this, oldFrameGeometry); } void InternalClient::requestGeometry(const QRect &rect) { if (m_internalWindow) { - m_internalWindow->setGeometry(mapToClient(rect)); + m_internalWindow->setGeometry(frameRectToClientRect(rect)); } } @@ -628,7 +618,7 @@ m_frameGeometry = rect; - m_clientSize = mapToClient(frameGeometry()).size(); + m_clientSize = frameRectToClientRect(frameGeometry()).size(); addWorkspaceRepaint(visibleRect()); syncGeometryToInternalWindow(); @@ -668,7 +658,7 @@ void InternalClient::syncGeometryToInternalWindow() { - if (m_internalWindow->geometry() == mapToClient(frameGeometry())) { + if (m_internalWindow->geometry() == frameRectToClientRect(frameGeometry())) { return; } @@ -681,7 +671,7 @@ return; } - commitGeometry(mapFromClient(m_internalWindow->geometry())); + commitGeometry(clientRectToFrameRect(m_internalWindow->geometry())); } }