RESOLVED FIXED 29844
[Qt] QWebPage's autotest dependency
https://bugs.webkit.org/show_bug.cgi?id=29844
Summary [Qt] QWebPage's autotest dependency
Jędrzej Nowacki
Reported 2009-09-29 04:07:40 PDT
There is dependency between QWebPage autotest ::databese() and ::testEnablePersistentStorage(). It looks like database connection stay opened after test. ~/dev/webkit/WebKitBuild/Debug/WebKit/qt/tests/qwebpage$ ./tst_qwebpage database ********* Start testing of tst_QWebPage ********* Config: Using QTest library 4.6.0, Qt 4.6.0 PASS : tst_QWebPage::initTestCase() PASS : tst_QWebPage::database() PASS : tst_QWebPage::cleanupTestCase() Totals: 3 passed, 0 failed, 0 skipped ********* Finished testing of tst_QWebPage ********* LEAK: 224 Structure LEAK: 5 WebCoreNode ~/dev/webkit/WebKitBuild/Debug/WebKit/qt/tests/qwebpage$ ./tst_qwebpage testEnablePersistentStorage ********* Start testing of tst_QWebPage ********* Config: Using QTest library 4.6.0, Qt 4.6.0 PASS : tst_QWebPage::initTestCase() PASS : tst_QWebPage::testEnablePersistentStorage() PASS : tst_QWebPage::cleanupTestCase() Totals: 3 passed, 0 failed, 0 skipped ********* Finished testing of tst_QWebPage ********* ~/dev/webkit/WebKitBuild/Debug/WebKit/qt/tests/qwebpage$ ./tst_qwebpage database testEnablePersistentStorage ********* Start testing of tst_QWebPage ********* Config: Using QTest library 4.6.0, Qt 4.6.0 PASS : tst_QWebPage::initTestCase() PASS : tst_QWebPage::database() ASSERTION FAILED: !m_database.isOpen() (../../../WebCore/storage/DatabaseTracker.cpp:78 void WebCore::DatabaseTracker::setDatabaseDirectoryPath(const WebCore::String&)) QFATAL : tst_QWebPage::testEnablePersistentStorage() Received signal 11 FAIL! : tst_QWebPage::testEnablePersistentStorage() Received a fatal error. Loc: [Unknown file(0)] Totals: 2 passed, 1 failed, 0 skipped ********* Finished testing of tst_QWebPage ********* Aborted
Attachments
Fix for autotest (2.06 KB, patch)
2009-09-29 05:17 PDT, Jędrzej Nowacki
hausmann: review+
Tor Arne Vestbø
Comment 1 2009-09-29 05:12:11 PDT
Please follow the QtWebKit bug reporting guidlines: http://trac.webkit.org/wiki/QtWebKitContrib#ReportingBugs In particular: - All bugs related to the Qt port of WebKit should have the keyword 'Qt' - The 'WebKit Qt' component should only be used for the QtWebKit API layer
Jędrzej Nowacki
Comment 2 2009-09-29 05:17:04 PDT
Created attachment 40296 [details] Fix for autotest In patch I assume that some databases might be shared between different instances of QWebPage. If it's not the case, ~QWebPage should implicitly remove all opened databases.
Simon Hausmann
Comment 3 2009-09-29 07:07:58 PDT
Note You need to log in before you can comment on or make changes to this bug.