fast/dom/Geolocation/window-close-crash.html was unskipped from wk2 by https://trac.webkit.org/changeset/130325, but it fails with notifydone timeout on Qt WK2: --- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/dom/Geolocation/window-close-crash-expected.txt +++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/fast/dom/Geolocation/window-close-crash-actual.txt @@ -1,14 +1,2 @@ -Tests the assertion that the GeolocationClient should not be updating -when the GeolocationController is destroyed. -See https://bugs.webkit.org/show_bug.cgi?id=52216 +FAIL: Timed out waiting for notifyDone to be called -On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE". - - -Main page opening resources/window-close-popup.html -PASS Received Geoposition. -PASS Success - no crash! -PASS successfullyParsed is true - -TEST COMPLETE -
I skipped it to paint the bot green - r130367 Please unskip it with the proper fix.
https://bugs.webkit.org/show_bug.cgi?id=98212 may fix that for you. If not check your platform.
Nothing changed, the test still fails with notifydone timeout.
(In reply to comment #3) > Nothing changed, the test still fails with notifydone timeout. It is strange, the geolocation tests are now almost completely port independent for WebKit2. If you find out the reason of the timeout, I am interested to hear.
still valid bug
(In reply to comment #4) > If you find out the reason of the timeout, I am interested to hear. I won't find the reason, because I don't work on it, I only reported this bug.
=== Bulk closing of Qt bugs === If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary. If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.