diff options
author | Gunnar Sletta <gunnar.sletta@nokia.com> | 2012-02-01 14:58:24 +0100 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-02-01 16:32:52 +0100 |
commit | 602d48011c5c26a962cdb99de87340c1fcbee4f4 (patch) | |
tree | f21aea3459bfb2a24cd460fc4c170ea75bf3ce82 /tests/auto | |
parent | c4c622f057d5d49cc86ad4a3228e9df592285004 (diff) |
Don't leave windows hanging outside test functions
Change-Id: I2df2167386b0827ae25a14f958ee50d0a15eba0b
Reviewed-by: Sergio Ahumada <sergio.ahumada@nokia.com>
Diffstat (limited to 'tests/auto')
-rw-r--r-- | tests/auto/qtquick2/qquickanimatedimage/tst_qquickanimatedimage.cpp | 32 |
1 files changed, 14 insertions, 18 deletions
diff --git a/tests/auto/qtquick2/qquickanimatedimage/tst_qquickanimatedimage.cpp b/tests/auto/qtquick2/qquickanimatedimage/tst_qquickanimatedimage.cpp index 17721a46df..c124e4f62b 100644 --- a/tests/auto/qtquick2/qquickanimatedimage/tst_qquickanimatedimage.cpp +++ b/tests/auto/qtquick2/qquickanimatedimage/tst_qquickanimatedimage.cpp @@ -141,20 +141,20 @@ void tst_qquickanimatedimage::mirror_running() { // test where mirror is set to true after animation has started - QQuickView *canvas = new QQuickView; - canvas->show(); + QQuickView canvas; + canvas.show(); - canvas->setSource(testFileUrl("hearts.qml")); - QQuickAnimatedImage *anim = qobject_cast<QQuickAnimatedImage *>(canvas->rootObject()); + canvas.setSource(testFileUrl("hearts.qml")); + QQuickAnimatedImage *anim = qobject_cast<QQuickAnimatedImage *>(canvas.rootObject()); QVERIFY(anim); int width = anim->property("width").toInt(); QCOMPARE(anim->currentFrame(), 0); - QPixmap frame0 = QPixmap::fromImage(canvas->grabFrameBuffer()); + QPixmap frame0 = QPixmap::fromImage(canvas.grabFrameBuffer()); anim->setCurrentFrame(1); - QPixmap frame1 = QPixmap::fromImage(canvas->grabFrameBuffer()); + QPixmap frame1 = QPixmap::fromImage(canvas.grabFrameBuffer()); anim->setCurrentFrame(0); @@ -165,11 +165,11 @@ void tst_qquickanimatedimage::mirror_running() anim->setProperty("mirror", true); QCOMPARE(anim->currentFrame(), 1); - QPixmap frame1_flipped = QPixmap::fromImage(canvas->grabFrameBuffer()); + QPixmap frame1_flipped = QPixmap::fromImage(canvas.grabFrameBuffer()); QTRY_VERIFY(spy.count() == 1); spy.clear(); QCOMPARE(anim->currentFrame(), 0); // animation only has 2 frames, should cycle back to first - QPixmap frame0_flipped = QPixmap::fromImage(canvas->grabFrameBuffer()); + QPixmap frame0_flipped = QPixmap::fromImage(canvas.grabFrameBuffer()); QSKIP("Skip while QTBUG-19351 and QTBUG-19252 are not resolved"); @@ -180,23 +180,21 @@ void tst_qquickanimatedimage::mirror_running() QCOMPARE(frame0_flipped, frame0_expected); QCOMPARE(frame1_flipped, frame1_expected); - - delete canvas; } void tst_qquickanimatedimage::mirror_notRunning() { QFETCH(QUrl, fileUrl); - QQuickView *canvas = new QQuickView; - canvas->show(); + QQuickView canvas; + canvas.show(); - canvas->setSource(fileUrl); - QQuickAnimatedImage *anim = qobject_cast<QQuickAnimatedImage *>(canvas->rootObject()); + canvas.setSource(fileUrl); + QQuickAnimatedImage *anim = qobject_cast<QQuickAnimatedImage *>(canvas.rootObject()); QVERIFY(anim); int width = anim->property("width").toInt(); - QPixmap screenshot = QPixmap::fromImage(canvas->grabFrameBuffer()); + QPixmap screenshot = QPixmap::fromImage(canvas.grabFrameBuffer()); QTransform transform; transform.translate(width, 0).scale(-1, 1.0); @@ -207,7 +205,7 @@ void tst_qquickanimatedimage::mirror_notRunning() bool paused = anim->isPlaying(); anim->setProperty("mirror", true); - screenshot = QPixmap::fromImage(canvas->grabFrameBuffer()); + screenshot = QPixmap::fromImage(canvas.grabFrameBuffer()); QSKIP("Skip while QTBUG-19351 and QTBUG-19252 are not resolved"); QCOMPARE(screenshot, expected); @@ -216,8 +214,6 @@ void tst_qquickanimatedimage::mirror_notRunning() QCOMPARE(anim->currentFrame(), frame); QCOMPARE(anim->isPlaying(), playing); QCOMPARE(anim->isPaused(), paused); - - delete canvas; } void tst_qquickanimatedimage::mirror_notRunning_data() |