summaryrefslogtreecommitdiffstats
path: root/tests/auto
Commit message (Collapse)AuthorAgeFilesLines
* Fixed crash when setting header data in QSqlQueryModel.Gabriel de Dietrich2009-10-261-4/+14
| | | | | | | | | | | | | | | The crash (Q_ASSERT_X failure) happened when a proxy model was being attached to the QSqlQueryModel, and no query was set yet. The headerDataChanged() signal was being received by the proxy model who wouldn't check its "proxyfied" data bounds. The patch introduces a behaviour change. However, this change makes the usage of QSqlQueryModel::setHeaderData() to be more in accordance with the current documentation, and to behave in the same way as for QStandardItemModel, QTreeModel, and QTableModel. Task-number: QTBUG-4963 Reviewed-by: Olivier
* stabilize test and fix warningOlivier Goffart2009-10-261-3/+3
|
* Drag and drop in QListWidget would not preserve the selectionGabriel de Dietrich2009-10-261-0/+59
| | | | | | | | | | Now, when dropping items, these will remain selected, and in the same visual order as when dragged. Auto-test included for the items moving part. For the rest, it's a drag-and-drop thing. Reviewed-by: Olivier
* Fix buglet in QGraphicsItem::stackBefore().Andreas Aardal Hanssen2009-10-261-28/+28
| | | | | | | | | | | | | | | | | Reported by Alan. The stackBefore() implementation did not alter the insertion order if the two items' current Z values were different. The fix is to ensure it is updated, so that the stackBefore() operation takes effect should the Z values become equal in the future. Example: Current order: A-B-C-D A->setZValue(1); Current order: B-C-D-A (A moves to the end) D->stackBefore(A); Current order: B-C-D-A (unchanged, D is already before A) A->setZValue(0); Current order: D-A-B-C (now A moves back, and D moves in front) Reviewed-by: Aaron Kennedy
* Wrong worldTransform() on the painter in QGraphicsScene::drawForeground.Bjørn Erik Nilsen2009-10-231-0/+49
| | | | | | | | | | | | | | | The painter's worldTransform() is updated for each item we draw, and when the DontSavePainterState optimization flag is set, this change is not protected by save() and restore(). After all the items are drawn, it means the painter is left with the last drawn item's transform. We therefore have to make sure it is reset back to whatever it was before the items were drawn. Auto-test included. Task-number: QTBUG-4973 Reviewed-by: alexis Reviewed-by: andreas
* Increase realiability of pauseanimation autotests on winLeonardo Sobral Cunha2009-10-231-20/+24
| | | | Reviewed-by: thierry
* MAke sure we call setCurrentTime when an animation is startedThierry Bastian2009-10-231-0/+9
| | | | | | This could be prevented by a pause animation currently running. Reviewed-by: Leo
* QGraphicsLineItem leave traces when moving around (reg. against 4.5)Bjørn Erik Nilsen2009-10-231-0/+34
| | | | | | | | | | | | | | | | | | The problem was that QGraphicsLineItem's bounding rect is an empty rect (either width is 0 or height is 0), and when updating the item's old occupied area, we explicitly checked whether the rect was empty() or not. In case of being empty (rect.isEmpty()) we did nothing, which was the root of the problem. We can safely remove the rect.isEmpty() check without any significant loss of performance since the common case is that the rect is non-empty. And in the case of being empty, we'll bail out from QGraphicsViewPrivate::updateRect's highly optimized rect intersection. Auto test included. Task: QTBUG-4877 Reviewed-by: alexis
* Fix crash in QGraphicsView BSP discovered in Amarok.Alexis Menard2009-10-221-0/+58
| | | | | | | | | | | | | Basically some items were not properly remove in the BSP which means that if you delete one of items, the BSP tree may contain dangling pointers. The problem was in removeItemHelper in QGraphicsScene were the child were removed after reparenting to 0 the topmost parent. The sceneBoundingRect for children was invalid which means that we were removing them in the wrong position inside the BSP. Reparenting to 0 means that the sceneBoundingRect will be the boundingRect but wasn't the case before (for the topmost parent). Reviewed-by:bnilsen
* Autotest fix for parallel animation groupThierry Bastian2009-10-221-11/+6
| | | | | On macos (as on symbian), we need to leave some time for the application to become responsive.
* stabilize QListView testOlivier Goffart2009-10-221-3/+6
|
* Merge branch 'origin/4.6' into widgets-team/4.6Olivier Goffart2009-10-227-1/+94
|\
| * QtScript: Compatibility with 4.5Olivier Goffart2009-10-221-0/+24
| | | | | | | | | | | | | | | | We must register the same type as they were registered in Qt 4.5 Reported on qt4-preview-feedback mailing list. Reviewed-by: Kent Hansen
| * qfiledialog2 added to tests/auto/auto.proJoerg Bornemann2009-10-221-0/+1
| | | | | | | | Reviewed-by: alexis
| * consider message id when comparing messagesOswald Buddenhagen2009-10-221-0/+6
| |
| * id-based: use source strings instead of empty translations only for ↵Oswald Buddenhagen2009-10-221-0/+1
| | | | | | | | unfinished messages
| * QDom autotests: make test fail instead of time outPeter Hartmann2009-10-221-0/+1
| | | | | | | | | | | | introduce a QFAIL for now until problem is fixed Reviewed-by: Carlos Duclos
| * Fix a bug in QGraphicsRotation related to 2D projectionsRhys Weatherley2009-10-221-1/+1
| | | | | | | | | | | | | | The projection to 2D needs to be done when the rotation is applied, not after all transformations have been applied. Reviewed-by: trustme
| * Merge branch '4.5' into 4.6Thiago Macieira2009-10-211-0/+45
| |\ | | | | | | | | | | | | | | | | | | Conflicts: src/gui/widgets/qmenu_mac.mm tests/benchmarks/benchmarks.pro tests/benchmarks/qnetworkreply/tst_qnetworkreply.cpp
| | * tst_qsslsocket: new bigChunk testcaseMarkus Goetz2009-10-201-0/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This new test is to find out if the BIO size of OpenSSL is limited or not. The test passes on my Linux, however the OpenSSL docu suggests that the BIO size is limited. From http://www.openssl.org/docs/crypto/BIO_s_bio.html "This is currently 17K". Reviewed-by: Peter Hartmann
| * | Fixed logical error in winIdChanged auto test.Gareth Stockwell2009-10-211-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test (see 02fbfdbd) previously asserted that changing the parent of a native widget caused a WinIdChange event on Symbian, but not on other platforms. The test now asserts that: 1. Changing the parent of a native widget causes a WinIdChange event on all platforms. 2. Changing the grandparent of a native widget causes a WinIdChange event only on Symbian. Reviewed-by: Paul Olav Tvete
* | | Merge branch '4.6' of git@scm.dev.nokia.troll.no:qt/qt into 4.6Marius Bugge Monsen2009-10-215-100/+129
|\ \ \ | |/ /
| * | Fix tst_QHeaderView::length() on Symbian.Liang QI2009-10-201-1/+4
| | | | | | | | | | | | | | | | | | Just the default font size will not work for this case. We set it to smaller one. Notice: we should make sure view->length() is different between view->setStretchLastSection(true) and view->setStretchLastSection(false). RevBy: Aleksandar Babic
| * | Windows CE gets a different paint event count than windowsninerider2009-10-201-2/+4
| | | | | | | | | | | | Reviewed-by: Joerg
| * | Test fixes for Windows Mobile.ninerider2009-10-202-96/+120
| | | | | | | | | | | | | | | | | | | | | | | | Accounting for double resolution devices on Windows Mobile in the test data sets (tst_qgraphicsview_2.cpp). Skipped a test involving mouse tracking (tst_qgraphicsview.cpp). Reviewed-by: Joerg
| * | Use premultiplied alpha pixel format in SymbianShane Kearns2009-10-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Gives better performance in the raster paint engine. For Symbian 9.3 onwards, this can also be used as the native pixmap format. For 9.2, conversion is required. Reviewed-by: Sami Merila Reviewed-by: Jani Hautakangas
* | | Moved private function to test which graphic items is in front of the otherLeonardo Sobral Cunha2009-10-201-0/+43
| | | | | | | | | | | | | | | | | | | | | This function is moved to graphicsitem private because it is needed by multi-touch event handling and is not specific to bsptreeindex. Reviewed-by: bnilsen
* | | Merge branch '4.6' of scm.dev.nokia.troll.no:qt/qt-widget-team into 4.6Jan-Arve Sæther2009-10-201-9/+21
|\ \ \
| * | | Extended commit 6c1388ee for LeftToRight flowGabriel de Dietrich2009-10-201-9/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Auto-test updated. As a bonus, stabilized tst_QListView::task262152_setModelColumnNavigate. Reviewed-by: Thierry
* | | | Merge branch 'qlayout-make_adjustSize_better' into 4.6Jan-Arve Sæther2009-10-201-2/+26
|\ \ \ \ | |/ / / |/| | |
| * | | Make adjustSize work even if there is a pending LayoutRequest event.Jan-Arve Sæther2009-10-201-2/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original report was that the customer did resize(main.sizeHint()) instead of the main.adjustSize(); Note that resize(main.sizeHint() still does not work. However, calling main.adjustSize() should now do what the original reporter wanted. The problem was that the resize did not work, because at the point of the resize the minimumHeight of main was still 22 (8+6+8), and we tried to resize it with a new height of 10. The resize would bound the height up to 22, and the main widget would then get a size of (200x22). The reason why it still had a minimumHeight of 22 was that it was the minimumSize of the previous layout configuration. Unfortunately the new minimumSize of the widget hadn't been updated yet because there was a LayoutRequest event in the queue that hadn't been processed yet. (This LayoutRequest was triggered by that we called invalidate() from hide()). Thus, processing the event queue immediately after the hide() could also have been a workaround for this issue. There is no really good fix for this issue (and it does not seem to be a common problem) without introducing a risk for regressions. Due to that we therefore decided to provide a fix in QWidget::adjustSize(). Reviewed-by: paul
* | | | Fixed QTreeView trying to animate when parent item has no childThierry Bastian2009-10-201-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | We now check that the item has children before animating. Reviewed-by: Alexis
* | | | QPauseAnimation autotests fixedThierry Bastian2009-10-201-1/+2
| | | |
* | | | adding autotestsThierry Bastian2009-10-201-0/+14
| | | |
* | | | Fix bug in embedded dialog demo with tab focus.Alexis Menard2009-10-201-1/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On embedded dialog pressing tab stop changing the focus when the focus was given to QFontComboBox. It's because QFontComboBox embed a QLineEdit in order to allow editing. But this QLineEdit is a focus proxy so we need to special case that. The logic is the same in QApplication. Be careful when changing one of them. Task-number:QTBUG-4818 Reviewed-by:jan-arve Reviewed-by:ogoffart
* | | | Fixed wrong scrolling in QListView with hidden rows in ListModeGabriel de Dietrich2009-10-201-0/+25
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | The flow positions in ScrollPerItem mode did not take the hidden rows into account when configuring the vertical scroll bar. A mapping between the scroll bar value and the flow position has been added. Auto-test included. Task-number: QTBUG-2233 Reviewed-by: Thierry
* | | Merge branch '4.6' of git@scm.dev.nokia.troll.no:qt/qt-widget-team into 4.6Alexis Menard2009-10-209-41/+319
|\ \ \
| * | | add the autotest for the QListView::setModelColumn bugPierre Rossi2009-10-191-0/+24
| |/ / | | | | | | | | | sha1 of the fix: 2633931653757decd93dd3939c09f5e07203da1c
| * | Do not loop forever if we are adding the layout itself to the layout.Jan-Arve Sæther2009-10-192-0/+22
| | | | | | | | | | | | Reviewed-by: alexis
| * | API addition: Make it possible to set the size policy of an anchor.Jan-Arve Sæther2009-10-161-14/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After this commit, when you modify the spacing of an anchor you are effectively modifying the preferred size of the anchor, since all anchors (except internal ones) have their minimumSizeHint to 0 and maximumSizeHint to QWIDGETSIZE_MAX. I also changed the sizeHintsFromItem to be more generic so that I could use it for anchors. (Thus, it was renamed to "internalSizeHints"). It now only takes care of setting the min/pref/exp/maxSize of AnchorData based on the anchor/item size hint and their size policies. As a consequence of all of this, setFixedSize changed behaviour and became setPreferredSize (since setSpacing is basically setPreferredSize). The implementation of that now only sets the prefSize. The patch also has an unrelated fix for IgnoreFlag, where it will (again) return the minimumSize for sizeHint(Qt::PreferredSize) instead of the maximumSize. This was to be more consistent with qgridlayoutengine.cpp. The docs are not very clear on this behaviour unfortunately. This API change has been discussed. Reviewed-by: alexis
| * | Merge branch 'fixes' of ↵Jan-Arve Sæther2009-10-161-22/+63
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | git://gitorious.org/~fleury/qt/fleury-openbossa-clone into 4.6 Conflicts: src/gui/graphicsview/qgraphicsanchorlayout_p.cpp src/gui/graphicsview/qgraphicsanchorlayout_p.h tests/auto/qgraphicsanchorlayout/tst_qgraphicsanchorlayout.cpp
| | * | QGraphicsAnchorLayout: Mark expected failures in tests w/o simplificationEduardo M. Fleury2009-10-151-24/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some tests are expected to fail when run without simplification, ie. by setting the environment var QT_ANCHORLAYOUT_NO_SIMPLIFICATION. This commit adds QEXPECT_FAIL before those tests. In some cases the failures are random therefore we must disable the test instead. Signed-off-by: Eduardo M. Fleury <eduardo.fleury@openbossa.org> Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>
| | * | QGAL: update the sizes of all anchors instead of doing it separatedlyCaio Marcelo de Oliveira Filho2009-10-151-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After calculations, update the size of all anchors in the simplified graph. Those updates were happening locally after each calculation (trunk and semifloats), however some anchors that were not involved in simplex calculation were missing. One concrete consequence of the previous behaviour is that semifloat parts that were simplified into just one anchor, didn't have the chance to set their sizeAt* values. One consequence of the new behaviour is one more test passing. Signed-off-by: Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org> Reviewed-by: Artur Duque de Souza <artur.souza@openbossa.org>
| | * | QGAL: add a way to test whether simplex was used in a calculationCaio Marcelo de Oliveira Filho2009-10-151-1/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is one good way to track whether simplification is doing all its job or not. However it can only track cases where the graph simplify to only one anchor. For more complex cases the graph dumps are the way to go. Signed-off-by: Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org> Reviewed-by: Artur Duque de Souza <artur.souza@openbossa.org>
| | * | QGraphicsAnchorLayout: Add test for 'infinite' max size widgetsEduardo M. Fleury2009-10-151-0/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With four items anchored side by side, each of them being allowed to grow to QWIDGETSIZE_MAX, we have a situation of fair distribution because the layout itself can grow only to QWIDGETSIZE_MAX (and not four times this amount). This test identified an error on the expanding distribution logic. Signed-off-by: Eduardo M. Fleury <eduardo.fleury@openbossa.org> Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>
| * | | Fixed typo in qtableview.cpp.Gabriel de Dietrich2009-10-161-5/+30
| | | | | | | | | | | | | | | | | | | | | | | | Auto-test included. Will timeout on fail. Reviewed-by: Alexis
| * | | Fixed a rare crash in qt_x11_enforce_cursorGabriel de Dietrich2009-10-151-0/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A QPointer was set to point to a QWidget by one of its children, during its deletion. This happens during the child deletion, and after the call to QObject::clearGuards(), which means that the QPointer becomes a dangling one. The fix ensures that qt_x11_enforce_cursor will not be called with a being-deleted QWidget. The included auto-test doesn't test anything, except that it doesn't crash. Reviewed-by: Olivier
| * | | Merge branch '4.6' of scm.dev.nokia.troll.no:qt/qt-widget-team into 4.6Jan-Arve Sæther2009-10-1528-944/+1258
| |\ \ \
| | * | | Compilation of the QGraphicsAnchorLayout test on 64bitOlivier Goffart2009-10-152-2/+3
| | | | | | | | | | | | | | | | | | | | + stabilize the linear layout test
| | * | | Stabilize test on x11Olivier Goffart2009-10-152-7/+8
| | | | |