Bug 89278

Summary: [GTK] New http/tests/cache/xhr-vary-header.html fails
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: Page LoadingAssignee: 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    

Description Csaba Osztrogonác 2012-06-16 01:40:27 PDT
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)
Comment 1 Csaba Osztrogonác 2012-06-16 01:42:36 PDT
I skipped it to paint the bot green - r120525. Please unskip it with the proper fix.
Comment 2 Alexey Proskuryakov 2012-06-16 12:44:02 PDT
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.
Comment 3 Sergio Villar Senin 2012-09-14 08:43:11 PDT
Exactly the same outcome in the WK2 GTK+ port