Bug 89278
| Summary: | [GTK] New http/tests/cache/xhr-vary-header.html fails | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> |
| Component: | Page Loading | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | ap, bugs-noreply, ddkilzer, mcatanzaro, ossy, svillar |
| Priority: | P2 | Keywords: | Qt, QtTriaged |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 87008, 88925 | ||
Csaba Osztrogonác
New test introduced in http://trac.webkit.org/changeset/120339 fails on Qt WK2:
--- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/http/tests/cache/xhr-vary-header-expected.txt
+++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/http/tests/cache/xhr-vary-header-actual.txt
@@ -3,7 +3,7 @@
Make cross origin request
PASS responseText is 'Cross origin response'
-PASS responseText is 'Same origin response'
+FAIL responseText should be Same origin response. Was Cross origin response.
PASS responseText is 'Cross origin response'
PASS successfullyParsed is true
Maybe we need some change in Source/WebCore/platform/network/qt/ResourceHandleQt.cpp .
(But I'm not sure, because it passes on Qt-WK1, but fails only on Qt-WK2)
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Csaba Osztrogonác
I skipped it to paint the bot green - r120525. Please unskip it with the proper fix.
Alexey Proskuryakov
It's surprising that WK1 and WK2 behaves differently. Perhaps Qt DumpRenderTree has network cache disabled completely somehow?
Anyway, the proper fix would be in networking code outside WebKit. You could consider applying the same workaround I made for Mac in the meanwhile.
Sergio Villar Senin
Exactly the same outcome in the WK2 GTK+ port