Bug 27143 – [Gtk][REGRESSION] XHR test failures after r45558

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Jan Alonzo]

This is a tracking bug for the three Gtk XHR test failures.

Changeset: http://trac.webkit.org/changeset/45558

Gustavo reported to following libsoup bug: http://bugzilla.gnome.org/show_bug.cgi?id=587907

XHR test failures:
http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache.html
http/tests/xmlhttprequest/cache-override.html
http/tests/xmlhttprequest/small-chunks-response-text.html

[Jan Alonzo]

libsoup bug - http://bugzilla.gnome.org/show_bug.cgi?id=590436

[Gustavo Noronha (kov)]

Created attachment 34696 [details]
disable sniffing when requested

 LayoutTests/ChangeLog                              |   11 +++++++++++
 LayoutTests/platform/gtk/Skipped                   |    1 -
 WebCore/ChangeLog                                  |   13 +++++++++++++
 .../platform/network/soup/ResourceHandleSoup.cpp   |    3 +++
 4 files changed, 27 insertions(+), 1 deletions(-)

[Xan Lopez]

Comment on attachment 34696 [details]
disable sniffing when requested

I endorse this patch or product.

[Xan Lopez]

*** Bug 27981 has been marked as a duplicate of this bug. ***

[Eric Seidel (no email)]

Comment on attachment 34696 [details]
disable sniffing when requested

Clearing flags on attachment: 34696

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	LayoutTests/ChangeLog
	M	LayoutTests/platform/gtk/Skipped
	M	WebCore/ChangeLog
	M	WebCore/platform/network/soup/ResourceHandleSoup.cpp
Committed r47203
	M	WebCore/ChangeLog
	M	WebCore/platform/network/soup/ResourceHandleSoup.cpp
	M	LayoutTests/platform/gtk/Skipped
	M	LayoutTests/ChangeLog
r47203 = 185743137afcd5b2b1ee59ebdea74c5d87cbd485 (trunk)
No changes between current HEAD and refs/remotes/trunk
Resetting to the latest refs/remotes/trunk
http://trac.webkit.org/changeset/47203

[Eric Seidel (no email)]

All reviewed patches have been landed.  Closing bug.

[Jan Alonzo]

Reopening. We still have two failing tests in this bug. The patch only fixed one.

[Jan Alonzo]

(In reply to comment #3)
> (From update of attachment 34696 [details])
> I endorse this patch or product.

I reverted the patch as it made the http tests fail.

[Eric Seidel (no email)]

Thank you.  Sorry that the commit-queue (currently) has no way to test on gtk.

[Jan Alonzo]

(In reply to comment #9)
> Thank you.  Sorry that the commit-queue (currently) has no way to test on gtk.

This is already fixed and landed as http://trac.webkit.org/changeset/47569.