RESOLVED WORKSFORME 49609
[Qt] REGRESSION (r72115): tst_qwebpage unit test is broken 
https://bugs.webkit.org/show_bug.cgi?id=49609
Summary [Qt] REGRESSION (r72115): tst_qwebpage unit test is broken
Csaba Osztrogonác
Reported 2010-11-16 12:08:19 PST
FAIL! : tst_QWebPage::testStopScheduledPageRefresh() '::waitForSignal(&page1, SIGNAL(loadFinished(bool)))' returned FALSE. () Loc: [/home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/WebKit/qt/tests/qwebpage/tst_qwebpage.cpp(2173)]
Attachments
Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2010-11-16 14:54:13 PST
Rollout committed in r72137 by Darin
Darin Adler
Comment 2 2010-11-16 14:56:27 PST
It’s likely that I will need to roll that change back in, so I’d like to understand more about this failure. Does this unit test depend on a load that fails the security check?
Csaba Osztrogonác
Comment 3 2010-11-16 15:20:27 PST
(In reply to comment #2) > It’s likely that I will need to roll that change back in, so I’d like to understand more about this failure. Does this unit test depend on a load that fails the security check? I cc-ed Jędrzej and Jocelyn, I think they know this unit test. Guys, any idea what caused this fail?
Jędrzej Nowacki
Comment 4 2010-11-17 01:17:54 PST
I don't know the test, but it fails because this code: "<html><head>" "<meta http-equiv=\"refresh\"content=\"0;URL=qrc:///resources/index.html\">" "</head><body><h1>Page redirects immediately...</h1>" "</body></html>" doesn't redirect anymore, so yes it could be security related.
Csaba Osztrogonác
Comment 5 2010-11-17 03:14:43 PST
reopen to find the correct fix
Csaba Osztrogonác
Comment 6 2011-02-10 09:30:09 PST
The original bug is fixed by http://trac.webkit.org/changeset/73444 (Am I right?) tst_QWebPage::testStopScheduledPageRefresh() works for me now, so we can close this bug.
Note You need to log in before you can comment on or make changes to this bug.