Avoid warning when maximum size < minimum size

This only happens in intermediate states, where both *minimumHeight*
and *maximumHeight* is supposed to be updated.
However, since the order of the bindings is not know, we might
have intermediate states where minimumHeight > maximumHeight.

In our case minimumHeight was set first to a smaller size than
maximumHeight, this would trigger the height binding, causing it to try
to set its geometry while its minimumHeight and maximumHeight properties
were not in sync.

This also happened when maximumWidth < minimumWidth.

The output was something like this:

qwindowswindow.cpp(1306):QWindowsWindow::setGeometry: Attempt to set a
size (116x190) violating the constraints(0x200 - 16777215x190) on
window ApplicationWindow_QMLTYPE_12_QML_51/''

Change-Id: Ia4f6e340b608bb4cff5a35bc887adc4eea7efbad
Reviewed-by: J-P Nurmi <jpnurmi@digia.com>

1 files changed, 38 insertions, 0 deletions

diff --git a/tests/auto/controls/data/tst_applicationwindow.qml b/tests/auto/controls/data/tst_applicationwindow.qml
index cfe6e0a3..8e35caff 100644
--- a/tests/auto/controls/data/tst_applicationwindow.qml
+++ b/tests/auto/controls/data/tst_applicationwindow.qml
@@ -168,5 +168,43 @@ TestCase {
         }
     }
 
+    function test_minimumSizeLargerThan_MaximumSize() {
+        var test_control = 'import QtQuick 2.1; \
+        import QtQuick.Controls 1.1;            \
+        import QtQuick.Layouts 1.1;             \
+        ApplicationWindow {                     \
+            minimumWidth: 200;                  \
+            maximumWidth: 200;                  \
+            minimumHeight: 200;                 \
+            maximumHeight: 200;                 \
+            Rectangle {                         \
+                implicitWidth: 1;               \
+                implicitHeight: 20;             \
+            }                                   \
+        }                                       '
+
+        var window = Qt.createQmlObject(test_control, container, '')
+        window.visible = true
+        wait(0)
+        // The following two calls will set the min,max range to be invalid
+        // this should *not* produce a warning
+        compare(window.height, 200)
+        window.maximumHeight -= 10
+        window.minimumHeight += 10
+        // Restore min,max range back to sane values
+        window.maximumHeight += 20
+        compare(window.height, 210)
+
+        // Do the same test for width
+        compare(window.width, 200)
+        window.maximumWidth-= 10
+        window.minimumWidth+= 10
+        // Restore back to sane values
+        window.maximumWidth += 20
+        compare(window.width, 210)
+
+        window.destroy()
+    }
+
 }
 }