WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
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
Committed
r48872
: <
http://trac.webkit.org/changeset/48872
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug