Implemented QWidget's move() vs setGeometry() distinction in QWindow.

QWidget's move() sets the position of the window including the window
frame, unlike setGeometry(). There was no equivalent for this in
QWindow, so several QWidget auto-tests were failing. Now we add
setFramePos() to achieve the same purpose in QWindow.

This fixes tst_QWidget::windowState(), which uses move().

Change-Id: I9a3e558bd615a8f0234cc3dd94fbb2bf5ecbc148
Reviewed-on: http://codereview.qt-project.org/5405
Reviewed-by: Qt Sanity Bot <qt_sanity_bot@ovi.com>
Reviewed-by: Lars Knoll <lars.knoll@nokia.com>

5 files changed, 91 insertions, 8 deletions

diff --git a/src/gui/kernel/qwindow.cpp b/src/gui/kernel/qwindow.cpp
index 759ee8dbc4..118844a0a2 100644
--- a/src/gui/kernel/qwindow.cpp
+++ b/src/gui/kernel/qwindow.cpp
@@ -412,9 +412,13 @@ void QWindow::setSizeIncrement(const QSize &size)
         d->platformWindow->propagateSizeHints();
 }
 
+/*!
+    Sets the geometry of the window excluding its window frame.
+*/
 void QWindow::setGeometry(const QRect &rect)
 {
     Q_D(QWindow);
+    d->positionPolicy = QWindowPrivate::WindowFrameExclusive;
     if (d->platformWindow) {
         d->platformWindow->setGeometry(rect);
     } else {
@@ -422,6 +426,9 @@ void QWindow::setGeometry(const QRect &rect)
     }
 }
 
+/*!
+    Returns the geometry of the window excluding its window frame.
+*/
 QRect QWindow::geometry() const
 {
     Q_D(const QWindow);
@@ -430,6 +437,9 @@ QRect QWindow::geometry() const
     return d->geometry;
 }
 
+/*!
+    Returns the window frame margins surrounding the window.
+*/
 QMargins QWindow::frameMargins() const
 {
     Q_D(const QWindow);
@@ -438,6 +448,56 @@ QMargins QWindow::frameMargins() const
     return QMargins();
 }
 
+/*!
+    Returns the geometry of the window including its window frame.
+*/
+QRect QWindow::frameGeometry() const
+{
+    Q_D(const QWindow);
+    if (d->platformWindow) {
+        QMargins m = frameMargins();
+        return d->platformWindow->geometry().adjusted(-m.left(), -m.top(), m.right(), m.bottom());
+    }
+    return d->geometry;
+}
+
+/*!
+    Returns the top left position of the window including its window frame.
+*/
+QPoint QWindow::framePos() const
+{
+    Q_D(const QWindow);
+    if (d->platformWindow) {
+        QMargins margins = frameMargins();
+        return d->platformWindow->geometry().topLeft() - QPoint(margins.left(), margins.top());
+    }
+    return d->geometry.topLeft();
+}
+
+/*!
+    Sets the upper left position of the window including its window frame.
+*/
+void QWindow::setFramePos(const QPoint &point)
+{
+    Q_D(QWindow);
+    d->positionPolicy = QWindowPrivate::WindowFrameInclusive;
+    if (d->platformWindow) {
+        d->platformWindow->setGeometry(QRect(point, size()));
+    } else {
+        d->geometry.setTopLeft(point);
+    }
+}
+
+void QWindow::resize(const QSize &newSize)
+{
+    Q_D(QWindow);
+    if (d->platformWindow) {
+        d->platformWindow->setGeometry(QRect(pos(), newSize));
+    } else {
+        d->geometry.setSize(newSize);
+    }
+}
+
 void QWindow::setWindowIcon(const QImage &icon) const
 {
     Q_UNUSED(icon);
diff --git a/src/gui/kernel/qwindow.h b/src/gui/kernel/qwindow.h
index 4bb1ff77a5..7a075e624d 100644
--- a/src/gui/kernel/qwindow.h
+++ b/src/gui/kernel/qwindow.h
@@ -142,6 +142,12 @@ public:
     void setGeometry(const QRect &rect);
     QRect geometry() const;
 
+    QMargins frameMargins() const;
+    QRect frameGeometry() const;
+
+    QPoint framePos() const;
+    void setFramePos(const QPoint &point);
+
     inline int width() const { return geometry().width(); }
     inline int height() const { return geometry().height(); }
     inline int x() const { return geometry().x(); }
@@ -150,13 +156,14 @@ public:
     inline QSize size() const { return geometry().size(); }
     inline QPoint pos() const { return geometry().topLeft(); }
 
-    inline void move(const QPoint &pt) { setGeometry(QRect(pt, size())); }
-    inline void move(int posx, int posy) { move(QPoint(posx, posy)); }
+    QT_DEPRECATED inline void move(const QPoint &pt) { setPos(pt); }
+    QT_DEPRECATED inline void move(int posx, int posy) { setPos(posx, posy); }
 
-    inline void resize(const QSize &newsize) { setGeometry(QRect(pos(), newsize)); }
-    inline void resize(int w, int h) { setGeometry(QRect(x(), y(), w, h)); }
+    inline void setPos(const QPoint &pt) { setGeometry(QRect(pt, size())); }
+    inline void setPos(int posx, int posy) { setPos(QPoint(posx, posy)); }
 
-    QMargins frameMargins() const;
+    void resize(const QSize &newSize);
+    inline void resize(int w, int h) { setGeometry(QRect(x(), y(), w, h)); }
 
     void setWindowIcon(const QImage &icon) const;
 
diff --git a/src/gui/kernel/qwindow_p.h b/src/gui/kernel/qwindow_p.h
index 8a3bc0d7fe..1ef8f59d33 100644
--- a/src/gui/kernel/qwindow_p.h
+++ b/src/gui/kernel/qwindow_p.h
@@ -43,6 +43,7 @@
 #define QWINDOW_QPA_P_H
 
 #include <QtGui/qwindow.h>
+#include <QtGui/qplatformwindow_qpa.h>
 
 #include <QtCore/private/qobject_p.h>
 
@@ -59,6 +60,12 @@ class Q_GUI_EXPORT QWindowPrivate : public QObjectPrivate
     Q_DECLARE_PUBLIC(QWindow)
 
 public:
+    enum PositionPolicy
+    {
+        WindowFrameInclusive,
+        WindowFrameExclusive
+    };
+
     QWindowPrivate()
         : QObjectPrivate()
         , surfaceType(QWindow::RasterSurface)
@@ -68,6 +75,7 @@ public:
         , visible(false)
         , windowState(Qt::WindowNoState)
         , resizeEventPending(true)
+        , positionPolicy(WindowFrameExclusive)
         , maximumSize(QWINDOWSIZE_MAX, QWINDOWSIZE_MAX)
         , modality(Qt::NonModal)
         , transientParent(0)
@@ -101,6 +109,7 @@ public:
     QRect geometry;
     Qt::WindowState windowState;
     bool resizeEventPending;
+    PositionPolicy positionPolicy;
 
     QSize minimumSize;
     QSize maximumSize;
diff --git a/src/plugins/platforms/xcb/qxcbwindow.cpp b/src/plugins/platforms/xcb/qxcbwindow.cpp
index bf882bdf84..4169ba5e4f 100644
--- a/src/plugins/platforms/xcb/qxcbwindow.cpp
+++ b/src/plugins/platforms/xcb/qxcbwindow.cpp
@@ -246,6 +246,8 @@ void QXcbWindow::create()
 
     Q_XCB_CALL(xcb_change_window_attributes(xcb_connection(), m_window, mask, values));
 
+    propagateSizeHints();
+
     xcb_atom_t properties[4];
     int propertyCount = 0;
     properties[propertyCount++] = atom(QXcbAtom::WM_DELETE_WINDOW);
@@ -335,6 +337,8 @@ void QXcbWindow::setGeometry(const QRect &rect)
 {
     QPlatformWindow::setGeometry(rect);
 
+    propagateSizeHints();
+
     const quint32 mask = XCB_CONFIG_WINDOW_X | XCB_CONFIG_WINDOW_Y | XCB_CONFIG_WINDOW_WIDTH | XCB_CONFIG_WINDOW_HEIGHT;
     const quint32 values[] = { rect.x(),
                                rect.y(),
@@ -1062,11 +1066,12 @@ void QXcbWindow::propagateSizeHints()
 
     QRect rect = geometry();
 
+    QWindow *win = window();
+
     xcb_size_hints_set_position(&hints, true, rect.x(), rect.y());
     xcb_size_hints_set_size(&hints, true, rect.width(), rect.height());
-    xcb_size_hints_set_win_gravity(&hints, XCB_GRAVITY_STATIC);
-
-    QWindow *win = window();
+    xcb_size_hints_set_win_gravity(&hints, qt_window_private(win)->positionPolicy == QWindowPrivate::WindowFrameInclusive
+                                           ? XCB_GRAVITY_NORTH_WEST : XCB_GRAVITY_STATIC);
 
     QSize minimumSize = win->minimumSize();
     QSize maximumSize = win->maximumSize();
diff --git a/src/widgets/kernel/qwidget_qpa.cpp b/src/widgets/kernel/qwidget_qpa.cpp
index 4c65d08b6a..65ba0fcb5a 100644
--- a/src/widgets/kernel/qwidget_qpa.cpp
+++ b/src/widgets/kernel/qwidget_qpa.cpp
@@ -126,6 +126,7 @@ void QWidgetPrivate::create_sys(WId window, bool initializeWindow, bool destroyO
         }
     }
 
+    qt_window_private(win)->positionPolicy = topData()->posFromMove ? QWindowPrivate::WindowFrameInclusive : QWindowPrivate::WindowFrameExclusive;
     win->create();
 
     data.window_flags = win->windowFlags();
@@ -661,6 +662,7 @@ void QWidgetPrivate::setGeometry_sys(int x, int y, int w, int h, bool isMove)
         if (!q->testAttribute(Qt::WA_DontShowOnScreen) && !q->testAttribute(Qt::WA_OutsideWSRange)) {
             if (q->windowHandle()) {
                 if (q->isWindow()) {
+                    qt_window_private(q->windowHandle())->positionPolicy = topData()->posFromMove ? QWindowPrivate::WindowFrameInclusive : QWindowPrivate::WindowFrameExclusive;
                     q->windowHandle()->setGeometry(q->geometry());
                 } else {
                     QPoint posInNativeParent =  q->mapTo(q->nativeParentWidget(),QPoint());