Bug 76784 - [Qt][WK2] http/tests/history/replacestate-post-to-get-2.html fails with timeout
Summary: [Qt][WK2] http/tests/history/replacestate-post-to-get-2.html fails with timeout
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: jochen
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 76721
  Show dependency treegraph
 
Reported: 2012-01-21 09:07 PST by Csaba Osztrogonác
Modified: 2012-01-31 09:27 PST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2012-01-21 09:07:11 PST
http/tests/history/replacestate-post-to-get-2.html introduced in 
http://trac.webkit.org/changeset/105574 and fails with timeout:

--- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/http/tests/history/replacestate-post-to-get-2-expected.txt 
+++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/http/tests/history/replacestate-post-to-get-2-actual.txt 
@@ -1,7 +1,2 @@
-If the current entry in the session history represents a non-GET request (e.g. it was the result of a POST submission) then update it to instead represent a GET request (or equivalent).
+FAIL: Timed out waiting for notifyDone to be called
 
-This test checks that this works when navigation back and forward again.
-
-This test passes if this page is eventually loaded with a GET request.
-
-This page was loaded with a GET request.
Comment 1 Csaba Osztrogonác 2012-01-21 09:19:50 PST
I skipped it: http://trac.webkit.org/changeset/105577
Comment 2 jochen 2012-01-28 11:24:46 PST
I tried to reproduce this locally, but the test passes for me.

Was it timing out only occasionally?
Comment 3 Csaba Osztrogonác 2012-01-30 05:36:21 PST
It failed always. But it seems it passes now, so I think
we can unskip it: http://trac.webkit.org/changeset/106239

But I'll leave the bug open for a while to see what happens.
Comment 4 jochen 2012-01-30 07:01:25 PST
(In reply to comment #3)
> It failed always. But it seems it passes now, so I think
> we can unskip it: http://trac.webkit.org/changeset/106239
> 
> But I'll leave the bug open for a while to see what happens.

sgtm, thanks
Comment 5 Csaba Osztrogonác 2012-01-31 09:27:12 PST
It seems it works fine, so we can close it.