Remove globalPos() in AbstractWaylandOutput

It's confusing to have globalPos() and geometry() as both can return the
same information.

This change drops globalPos() function as it's not used outside the
AbstractWaylandOutput class and renames setGlobalPos() to moveTo() to
avoid having a setter without matching getter.

src/abstract_wayland_output.cpp

@@ -55,7 +55,7 @@ QUuid AbstractWaylandOutput::uuid() const
 
 QRect AbstractWaylandOutput::geometry() const
 {
-    return QRect(globalPos(), pixelSize() / scale());
+    return QRect(m_position, pixelSize() / scale());
 }
 
 QSize AbstractWaylandOutput::physicalSize() const
@@ -68,12 +68,7 @@ int AbstractWaylandOutput::refreshRate() const
     return m_refreshRate;
 }
 
-QPoint AbstractWaylandOutput::globalPos() const
+void AbstractWaylandOutput::moveTo(const QPoint &pos)
-{
-    return m_position;
-}
-
-void AbstractWaylandOutput::setGlobalPos(const QPoint &pos)
 {
     if (m_position != pos) {
         m_position = pos;
@@ -168,7 +163,7 @@ void AbstractWaylandOutput::applyChanges(const KWaylandServer::OutputChangeSet *
     }
     if (changeSet->positionChanged()) {
         qCDebug(KWIN_CORE) << "Server setting position: " << changeSet->position();
-        setGlobalPos(changeSet->position());
+        moveTo(changeSet->position());
         // may just work already!
         overallSizeCheckNeeded = true;
     }

src/abstract_wayland_output.h

@@ -107,7 +107,7 @@ public:
     QString model() const override;
     QString serialNumber() const override;
 
-    void setGlobalPos(const QPoint &pos);
+    void moveTo(const QPoint &pos);
     void setScale(qreal scale);
 
     void applyChanges(const KWaylandServer::OutputChangeSet *changeSet) override;
@@ -156,8 +156,6 @@ protected:
                     const QSize &physicalSize,
                     const QVector<Mode> &modes, const QByteArray &edid);
 
-    QPoint globalPos() const;
-
     bool internal() const {
         return m_internal;
     }

src/plugins/platforms/drm/drm_backend.cpp

@@ -490,7 +490,7 @@ void DrmBackend::readOutputsConfiguration()
         qCDebug(KWIN_DRM) << "Reading output configuration for " << *it;
         if (!outputInfo.isEmpty()) {
             const QJsonObject pos = outputInfo["pos"].toObject();
-            (*it)->setGlobalPos({pos["x"].toInt(), pos["y"].toInt()});
+            (*it)->moveTo({pos["x"].toInt(), pos["y"].toInt()});
             if (const QJsonValue scale = outputInfo["scale"]; !scale.isUndefined()) {
                 (*it)->setScale(scale.toDouble(1.));
             }
@@ -501,7 +501,7 @@ void DrmBackend::readOutputsConfiguration()
                 (*it)->updateMode(size["width"].toInt(), size["height"].toInt(), mode["refresh"].toDouble() * 1000);
             }
         } else {
-            (*it)->setGlobalPos(pos);
+            (*it)->moveTo(pos);
             (*it)->setTransformInternal(DrmOutput::Transform::Normal);
         }
         pos.setX(pos.x() + (*it)->geometry().width());

src/plugins/platforms/virtual/virtual_output.cpp

@@ -64,7 +64,7 @@ void VirtualOutput::init(const QPoint &logicalPosition, const QSize &pixelSize)
 void VirtualOutput::setGeometry(const QRect &geo)
 {
     // TODO: set mode to have updated pixelSize
-    setGlobalPos(geo.topLeft());
+    moveTo(geo.topLeft());
 }
 
 void VirtualOutput::vblank(std::chrono::nanoseconds timestamp)

src/plugins/platforms/wayland/wayland_output.cpp

@@ -79,7 +79,7 @@ void WaylandOutput::setGeometry(const QPoint &logicalPosition, const QSize &pixe
     // TODO: set mode to have updated pixelSize
     Q_UNUSED(pixelSize)
 
-    setGlobalPos(logicalPosition);
+    moveTo(logicalPosition);
 }
 
 void WaylandOutput::updateEnablement(bool enable)

src/plugins/platforms/x11/windowed/x11windowed_output.cpp

@@ -151,7 +151,7 @@ void X11WindowedOutput::setGeometry(const QPoint &logicalPosition, const QSize &
 {
     // TODO: set mode to have updated pixelSize
     Q_UNUSED(pixelSize);
-    setGlobalPos(logicalPosition);
+    moveTo(logicalPosition);
 }
 
 void X11WindowedOutput::setWindowTitle(const QString &title)