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

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