fix(surface): set maximized/fullscreen geometry on initial surface co…

src/surface/surfacewrapper.cpp

@@ -19,15 +19,17 @@
 #include <woutputrenderwindow.h>
 #include <wsocket.h>
 #include <wxdgpopupsurfaceitem.h>
 #include <wxdgtoplevelsurfaceitem.h>
 #include <wxwaylandsurface.h>
 #include <wxwaylandsurfaceitem.h>
+#include <wxdgtoplevelsurface.h>
 
 #include <qwbuffer.h>
 #include <qwlayershellv1.h>
+#include <qwxdgshell.h>
 
 #include <QColor>
 #include <QVariant>
 
 #define OPEN_ANIMATION 1
 #define CLOSE_ANIMATION 2
@@ -297,6 +299,32 @@
     m_surfaceItem->setShellSurface(m_shellSurface);
 
     if (!m_isProxy) {
+        if (auto toplevelItem = qobject_cast<WXdgToplevelSurfaceItem*>(m_surfaceItem)) {
+            connect(toplevelItem, &WXdgToplevelSurfaceItem::initialCommitReceived, this, [this] {
+                    auto toplevel = qobject_cast<WXdgToplevelSurface *>(m_shellSurface);
+                    auto handle = toplevel->handle()->handle();
+                    if (handle->requested.maximized) {
+                        toplevel->handle()->set_maximized(true);
+                        resize(maximizedGeometry().size());
+                        m_previousSurfaceState.setValueBypassingBindings(State::Maximized);
+                        m_surfaceState.setValueBypassingBindings(State::Maximized);
+                        setVisibleDecoration(false);
+                        setNoCornerRadius(true);
+
+                        return;
+                    }
+                    if (handle->requested.fullscreen) {
+                        toplevel->handle()->set_fullscreen(true);
+                        resize(fullscreenGeometry().size());
+                        m_previousSurfaceState.setValueBypassingBindings(State::Fullscreen);
+                        m_surfaceState.setValueBypassingBindings(State::Fullscreen);
+                        setVisibleDecoration(false);
+                        setNoCornerRadius(true);
+                        return;
+                    }
+
+                }, Qt::SingleShotConnection);
+        }
         m_shellSurface->safeConnect(&WToplevelSurface::requestMinimize, this, [this]() {
             minimize();
         });
@@ -1006,7 +1034,7 @@
         targetGeometry = m_tilingGeometry;
     }
 
-    if (targetGeometry.isValid()) {
+    if (targetGeometry.isValid() && m_previousSurfaceState.value() != newSurfaceState) {
         startStateChangeAnimation(newSurfaceState, targetGeometry);
     } else {
         if (m_geometryAnimation) {
@@ -1414,7 +1442,7 @@
 
 void SurfaceWrapper::onAnimationReady()
 {
-    Q_ASSERT(m_pendingState != m_surfaceState);
+    // Q_ASSERT(m_pendingState != m_surfaceState);
     Q_ASSERT(m_pendingGeometry.isValid());
 
     if (!resize(m_pendingGeometry.size(), true)) {
@@ -1649,6 +1677,7 @@
 
 void SurfaceWrapper::restoreFromMinimized(bool onAnimation)
 {
+    qWarning() << "xyb---state" << m_surfaceState.value();
     if (m_surfaceState != State::Minimized && m_hideByshowDesk)
         return;
     if (!m_hideByshowDesk)
@@ -1670,6 +1699,8 @@
 
 void SurfaceWrapper::unmaximize()
 {
+
+    qWarning() << "xyb---unmaximize state:" << m_surfaceState.value();
     if (m_surfaceState != State::Maximized)
         return;
 

waylib/src/server/qtquick/wxdgtoplevelsurfaceitem.cpp

@@ -78,11 +78,10 @@ void WXdgToplevelSurfaceItem::onSurfaceCommit()
 
     auto xdg_surface = toplevelSurface()->handle()->handle()->base;
     if (xdg_surface->initial_commit) {
-        /* When an xdg_surface performs an initial commit, the compositor must
-         * reply with a configure so the client can map the surface.
-         * configures the xdg_toplevel with 0,0 size to let the client pick the
-         * dimensions itself. */
-        toplevelSurface()->handle()->set_size(0, 0);
+	toplevelSurface()->handle()->set_size(0, 0);
+        if (!xdg_surface->surface->mapped) {
+            Q_EMIT initialCommitReceived();
+        }
     }
 }
 

waylib/src/server/qtquick/wxdgtoplevelsurfaceitem.h

@@ -36,6 +36,7 @@ class WAYLIB_SERVER_EXPORT WXdgToplevelSurfaceItem : public WSurfaceItem
 Q_SIGNALS:
     void minimumSizeChanged();
     void maximumSizeChanged();
+    void initialCommitReceived();
 
 private:
     Q_SLOT void onSurfaceCommit() override;