Bug 78597
Summary: | [Qt] http/tests/navigation/https-in-page-cache.html fails | ||
---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> |
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED INVALID | ||
Severity: | Normal | CC: | beidson, ossy |
Priority: | P2 | Keywords: | Qt, QtTriaged |
Version: | 528+ (Nightly build) | ||
Hardware: | All | ||
OS: | All | ||
Bug Depends on: | |||
Bug Blocks: | 26777 |
Csaba Osztrogonác
http/tests/navigation/https-in-page-cache.html introduced in
http://trac.webkit.org/changeset/107607 but fails on Qt (wk1 and wk2 too)
wk1 fail:
--- /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/layout-test-results/http/tests/navigation/https-in-page-cache-expected.txt
+++ /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/layout-test-results/http/tests/navigation/https-in-page-cache-actual.txt
@@ -3,5 +3,6361 @@
ALERT: This page is https and has the no-cache cache-control directive. It should NOT go in to the page cache.
ALERT: The page was reloaded on back, not from the page cache. Good job. Running part 3 of the test.
ALERT: This page is https and should go in to the page cache.
-ALERT: The page was restored from the page cache. Good job!
-
+ALERT: This page is https and should go in to the page cache.
+ALERT: This page is https and should go in to the page cache.
+ALERT: This page is https and should go in to the page cache.
...
...
+ALERT: This page is https and should go in to the page cache.
+FAIL: Timed out waiting for notifyDone to be called
+This is just a minimal page that we navigate to as part of testing back/forward.
wk2 fail:
--- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/http/tests/navigation/https-in-page-cache-expected.txt
+++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/http/tests/navigation/https-in-page-cache-actual.txt
@@ -1,7 +1,2 @@
-ALERT: This page is https and has the no-store cache-control directive. It should NOT go in to the page cache.
-ALERT: The page was reloaded on back, not from the page cache. Good job. Running part 2 of the test.
-ALERT: This page is https and has the no-cache cache-control directive. It should NOT go in to the page cache.
-ALERT: The page was reloaded on back, not from the page cache. Good job. Running part 3 of the test.
-ALERT: This page is https and should go in to the page cache.
-ALERT: The page was restored from the page cache. Good job!
+FAIL: Timed out waiting for notifyDone to be called
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Csaba Osztrogonác
I skipped it until proper fix - http://trac.webkit.org/changeset/107702
Brady Eidson
Out of curiosity, does the Qt port even support the Page Cache?
Brady Eidson
(Sorry about adding the failing test, btw)
Jocelyn Turcotte
=== 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.