[webkit-reviews] review denied: [Bug 24851] Failures in http/tests/xhlhttprequest/web-apps : [Attachment 28992] Update to web-apps test result and skip list

[bugzilla-daemon at webkit.org]

Alexey Proskuryakov <ap at webkit.org> has denied Brian Weinstein
<bweinstein at gmail.com>'s request for review:
Bug 24851: Failures in http/tests/xhlhttprequest/web-apps
https://bugs.webkit.org/show_bug.cgi?id=24851

Attachment 28992: Update to web-apps test result and skip list
https://bugs.webkit.org/attachment.cgi?id=28992&action=review

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
The problem here is that XMLHttpRequest.statusText includes the status code
(e.g. it's "400 Good work" instead of "Good Work"). Looking at
ResourceResponse::platformLazyInit(), it has an obvious bug, removing only
"HTTP/1.1" from a response such as "HTTP/1.1 400 Good Work":

	RetainPtr<CFStringRef> statusLine(AdoptCF,
CFHTTPMessageCopyResponseStatusLine(httpResponse));
	String statusText(statusLine.get());
	int spacePos = statusText.find(" ");
	if (spacePos != -1)
	    statusText = statusText.substring(spacePos + 1);
	m_httpStatusText = statusText;

Looks like this is easier to fix than to land failing results.