90530 - nsXMLHttpRequest should let you set the content-type

Status: VERIFIED FIXED

(Core :: XML, defect, P2)

Description

[Garth Smedley]

Currently the content-type of the request sent by nsXMLHttpRequest::Send is
hard-coded to be text/xml in nsXMLHttpRequest::GetStreamForWString. The
nsXMLHttpRequest::SetRequestHeader method could be used to set the content type
if it weren't for the hard-coded  value. 

I have a patch which only sets the content type to text/xml if it hasn't already
been set. We are using this patch internally at OEone, but it would be nice to
have this in the main Mozilla source.

Comment 1

[Garth Smedley]

Attachment: Proposed patch to allow the content-type to be set

Comment 2

[Heikki Toivonen (remove -bugzilla when emailing directly)]

r=heikki

I am not sure what would be the correct case for Content-Type, Content-Length
etc. headers; it seems different webservers give different case.

Will try to check this in for 0.9.4.

Comment 3

[Johnny Stenback (:jst)]

sr=jst

Comment 4

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Checked in.

Comment 5

[Chris Petersen]

Since I 'm not exactly sure how to test this, I'm marking verified based on the
last comments