summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Fix crash in QMenu when using QWidgetActionOlivier Goffart2009-11-041-0/+2
| | | | | | | | | | | If the QWidgetAction is not the first in a menu, it crashed. This is because when the QEvent::ActionRemoved is sent, the event->before() is not set. We need to find another way to clean up the widgetItems list. This basically revert 4b6ab00e6d68c7 Reviewed-by: Thierry
* Merge commit 'origin/4.6' into 4.6Olivier Goffart2009-11-0433-462/+838
|\
| * QVideoFrame API review changes.Andrew den Exter2009-11-041-9/+9
| | | | | | | | | | | | | | | | Rename equivalentImageFormat() to imageFormatFromPixelFormat(). Rename equivalentPixelFormat() to pixelFormatFromImageFormat(). Rename numBytes() to mappedBytes(). Reviewed-by: Justin McPherson
| * QVideoSurfaceFormat API review changes.Andrew den Exter2009-11-041-48/+29
| | | | | | | | | | | | | | | | Rename YuvColorSpace enum and related functions to YCbCrColorSpace. Remove ViewportMode enum and make setFrameSize() always reset the viewport to fill the frame. Reviewed-by: Justin McPherson
| * QAbstractVideoSurface API review changes.Andrew den Exter2009-11-041-10/+28
| | | | | | | | | | | | | | | | | | Rename isStarted() and startedChanged() to is active() and activeChanged(). Remove the the similar format argument from isFormatSupported() and add a new nearestFormat() function which provides the same functionality. Reviewed-by: Justin McPherson
| * Merge branch '4.6' of git@scm.dev.nokia.troll.no:qt/qt into 4.6Gunnar Sletta2009-11-042-36/+47
| |\
| | * Declarative building infrastructure.Warwick Allison2009-11-041-0/+2
| | | | | | | | | | | | Does nothing if no declarative directories installed.
| | * fix test for real - if the reference stream contains doubles, we must read ↵Harald Fernengel2009-11-031-32/+31
| | | | | | | | | | | | doubles, and not qreals
| | * Make the QDataStream autotest much more robustHarald Fernengel2009-11-031-10/+20
| | | | | | | | | | | | | | | | | | | | | In the old version, we'd read all the types, then start comparing. If an operator left the stream in a bogus state, funny things would happen (e.g. crashes). Now, we test the value instantly before reading the next one, thus preventing reading bogus.
| * | Merge branch '4.6' of git@scm.dev.nokia.troll.no:qt/qt-graphics-team into 4.6Gunnar Sletta2009-11-045-191/+87
| |\ \
| | * | Remove QMatrix4x4& return values in response to API review feedbackRhys Weatherley2009-11-041-2/+4
| | | | | | | | | | | | | | | | Reviewed-by: Sarah Smith
| | * | Incorporate API review feedback for math3d classes.Rhys Weatherley2009-11-043-149/+29
| | | | | | | | | | | | | | | | Reviewed-by: Sarah Smith
| | * | Merge branch 'statemachine-api-changes' into 4.6Eskil Abrahamsen Blomfeldt2009-11-031-40/+54
| | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: doc/src/frameworks-technologies/statemachine.qdoc src/corelib/statemachine/qstatemachine.cpp
| | | * | Rename QState::polished() signal to "propertiesAssigned"Eskil Abrahamsen Blomfeldt2009-11-031-17/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "Polished" was never a very descriptive word, and it already has a meaning attached in the QStyle API. Additionally, "propertiesAssigned" has the benefit of giving the relation to the assignProperty() function as part of the name. Reviewed-by: Kent Hansen
| | | * | Remove return type of QState::addTransition(QAbstractTransition*)Eskil Abrahamsen Blomfeldt2009-11-031-14/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Returning the input argument in a function can lead to confusion and serves no purpose here. Instead, we'll mirror the API from QMenu::addAction() overloads, where the overload that takes a preconstructed object has a void return type. Reviewed-by: Kent Hansen
| | | * | Change name of DoNotRestoreProperties enum to DontRestorePropertiesEskil Abrahamsen Blomfeldt2009-11-031-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using the abbreviated "Dont" is consistent with other negated enum names in Qt. Reviewed-by: Kent Hansen
| | | * | Change name of QStateMachine::animationsEnabled property to "animated"Eskil Abrahamsen Blomfeldt2009-11-031-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The name "animated" is consistent with naming in Qt otherwise, as in QTreeView::animated and QMainWindow::animated. Reviewed-by: Kent Hansen
| * | | | Fixed tst_qgraphicsitem::hitTestGraphicsEffectItem, caching makesGunnar Sletta2009-11-031-14/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | testcase invalid... Reviewed-by: Samuel
| * | | Merge branch '4.6' of git@scm.dev.nokia.troll.no:qt/qt into 4.6Gunnar Sletta2009-11-0367-269/+3168
| |\ \ \ | | | |/ | | |/|
| | * | Merge commit 'e4bec39a139363d1ee4cf3fb15a3fe4499215e77' into 4.6Thiago Macieira2009-11-029-28/+472
| | |\ \
| | | * \ Merge remote branch 'origin/4.6' into core-4.6Thiago Macieira2009-10-294-19/+129
| | | |\ \
| | | * | | Autotest: this test is failing, enable debugging to find out why.Thiago Macieira2009-10-291-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It doesn't happen on my machine, so I need to figure out what's different. Reviewed-by: Trust Me
| | | * | | Merge branch '4.6' into core-4.6Thiago Macieira2009-10-297-5/+257
| | | |\ \ \
| | | * | | | QLocalServer: block indefinitely when timeout value is -1Peter Hartmann2009-10-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... as described in the documentation. Furthermore: * use qt_safe_select to timeout correctly * return immediately when timeout value is 0 Reviewed-by: Oswald Buddenhagen
| | | * | | | Fix Qt containers to properly support types with strict alignments.Thiago Macieira2009-10-281-0/+109
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | QContiguousCache is a new type, so there are no binary compatibility issues. QHash and QMap didn't have any public qMalloc / qFree, so the entire logic is contained in .cpp code. However, since old code will not inform us of the alignment requirements, we need to add a bit to the structure to indicate whether strict alignment is in use or not. QList doesn't require any changes. For small, movable types, they're all stored in the pointer array itself, so they're aligned. For larger types, we use new(), so types with stricter requirements should define their own operator new(). QLinkedList cannot be fixed. It uses new() on the QLinkedListNode, which contains a T type. Sorry. QVector did have public qMalloc / qFree. I've moved the calls to the inner function and made it keep the old calls if the alignment requirement is below a certain threshold. The idea is that, if it's above, no one was using QVector anyway. Reviewed-by: Bradley T. Hughes
| | | * | | | Autotest: add unit test for QDBusServiceWatcherThiago Macieira2009-10-282-0/+281
| | | | | | |
| | | * | | | Autotest: add a test that tries to follow a service changing owners.Thiago Macieira2009-10-281-0/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | I'm not sure if this used to work before...
| | | * | | | Autotest: fix improper use of the serviceOwnerChanged signalThiago Macieira2009-10-281-23/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This test was doubly wrong: it first registered a service name, then it connected to signal to watch it. You can't receive a signal if you connect to it after it's emitted... Second, it waited for any serviceOwnerChanged() signal to exit the event loop, not necessarily the one we wanted to receive. This used to work because we'd always connect to the D-Bus signal, but now we don't anymore.
| | | * | | | Merge remote branch 'origin/4.6' into core-4.6Thiago Macieira2009-10-289-32/+179
| | | |\ \ \ \
| | | * | | | | QDom autotests: test is not failing anymorePeter Hartmann2009-10-281-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | removing the temporary QFAIL Reviewed-by: Carlos Duclos
| | | * | | | | Autotest: Add some debugging info as to why the subprocess fails to startThiago Macieira2009-10-271-0/+10
| | | | | | | |
| | | * | | | | QAbstractSocket: wait with closing until all bytes have been writtenPeter Hartmann2009-10-271-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | only disconnect from host when all bytes have been written; i.e. not only check whether the write buffer is empty, but also check whether the socket engine has still bytes to write. This is necessary for HTTP and SOCKS5 socket engine, because they both contain an inner TCP socket which also does buffering. For the native socket engine, there is no difference with this patch. Reviewed-by: Markus Goetz Reviewed-by: Thiago Macieira
| | | * | | | | QDom: make tests fail only on WindowsPeter Hartmann2009-10-261-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reviewed-by: Carlos Duclos
| | * | | | | | Merge remote branch 'mainline/4.6' into 4.6Oswald Buddenhagen2009-11-0248-117/+2567
| | |\ \ \ \ \ \
| | * \ \ \ \ \ \ Merge commit 'mainline/4.6' into 4.6hjk2009-10-3025-61/+670
| | |\ \ \ \ \ \ \
| | * | | | | | | | rename qstringbuilder test classeshjk2009-10-3010-124/+129
| | | |_|/ / / / / | | |/| | | | | |
| * | | | | | | | Adapt testcase to cosmetic pen workarounds in codeGunnar Sletta2009-11-031-2/+2
| | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | Reviewed-by: Samuel
| * | | | | | | Fix some test failures in the qgl test on 16 bit servers.Trond Kjernåsen2009-10-301-0/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These tests are designed to run in 24 or 32 bit mode, and won't work unless they do.. Reviewed-by: Samuel
| * | | | | | | Merge branch '4.6' of git@scm.dev.nokia.troll.no:qt/qt-graphics-team into 4.6Gunnar Sletta2009-10-301-0/+13
| |\ \ \ \ \ \ \ | | |_|/ / / / / | |/| | | | | |
| | * | | | | | Avoid infinite loop when laying out text with unconvertible charsEskil Abrahamsen Blomfeldt2009-10-291-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the stringToCMap() fails, it can be because it did not have enough space in the layout, or it can because of other errors. In order to implement "try-again" processing in a simple way, we had an infinite loop which assumed that stringToCMap() would always succeed in the second run (which would be the case if the only possible error was "not enough space".) Since there are other possible failures not related to the number of glyphs, you could easily get into an infinite loop here, e.g. when laying out text that contains the Byte Order Mark. The fix changes the implementation to explictly try stringToCMap() twice at max, and is also how it's implemented in the default qtextengine.cpp. Task-number: QTBUG-4680 Reviewed-by: Trond
* | | | | | | | Stabilize testsOlivier Goffart2009-11-042-8/+8
| | | | | | | |
* | | | | | | | Pressing return in a QWizard would erase the active password entry.Gabriel de Dietrich2009-11-031-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When echo mode was set to PasswordEchoOnEdit in a QLineEdit, and its text selected, pressing the return key would erase the text and start editing it instead of validating the password. Auto-test included. Reviewed-by: Olivier Task-number: QTBUG-4401
* | | | | | | | Introduced QGraphicsItem::ItemSendsScenePositionChanges andJ-P Nurmi2009-11-031-0/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | QGraphicsItem::ItemScenePositionHasChanged Merge-Request: 1945 Reviewed-By: Andreas
* | | | | | | | Fixed QGraphicsScene::clear() not to crash if related top level itemsJ-P Nurmi2009-11-031-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | happen to delete each others. Merge-Request: 1863 Reviewed-by: Andreas Reviewed-by: Olivier
* | | | | | | | Fixed spacing display in QListView with wrapped text.Gabriel de Dietrich2009-11-021-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The spacing was not being substracted from the viewport width when calculating the available space for items. Auto-test included. Reviewed-by: Olivier Task-number: QTBUG-2678
* | | | | | | | Make the combobox emit activated when it loses focusThierry Bastian2009-11-021-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Task-number: QTBUG-1071 Reviewed-by: ogoffart
* | | | | | | | QCSSParser: Fixes the way spaces are handled in font family.Olivier Goffart2009-11-022-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We cannot simplify spaces because a font may have several spaces in its name. If we only add spaces when needed when merging tokens, we don't need to simplyfy the string later The CSS tokenizer will already make sure that spaces are removed if there is no quotes (btw, the CSS specification says that there must be quotes if there is spaces, but we tollerate if there is no quotes) Reviewed-by: Jocelyn Turcotte Task-number: QTBUG-4344 Task-number: 258586
* | | | | | | | QStyleSheetStyle: ItemViews: Fixes drawing of items and branches.Olivier Goffart2009-11-021-0/+179
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I am not sure why the code was there, but after some testing, it is better to remove it: - While painting the branch, we should not look at the ::item pseudo class at all. - Items specific stuff is drawn in PE_PanelItemViewItem, not PE_PanelItemViewRow - Selection is handled his way by the native style. background-color is not for selection and should not depends on SH_ItemView_ShowDecorationSelected Reviewed-by: Thierry Task-number: QTBUG-5228 Task-number: 258382 Task-number: QTBUG-4338
* | | | | | | | Merge origin/4.6 into team-widgets/4.6Olivier Goffart2009-11-0212-16/+1021
|\ \ \ \ \ \ \ \ | | |_|_|/ / / / | |/| | | | | |
| * | | | | | | Manual glyph shaping test.Alessandro Portale2009-11-023-0/+525
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It tests the glyph shaper in Qt by rendering a couple of glyph combinations that are known to be strongly shaped. For now, this test is manual and simply generates an html report for a visual test. It can, however also be implemented as auto test. The test data contains Vietnamese and Tamil data sets, more will follow. Reviewed-by: trustme