240321 - XHTML Documents should default to UTF-8 and not look at the "default" charset

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Bob Clary [:bc] (inactive)]

When UTF-8 document is served as HTML, multibyte characters are display
correctly if charset autodetection is turned off however if the document is
served as XHTML then the document is incorrectly assumed to be iso-8859-1.

Comment 1

[Bob Clary [:bc] (inactive)]

Attachment: HTML test case

Comment 2

[Bob Clary [:bc] (inactive)]

Attachment: XHTML test case

bz says <quote>It's a bug in nsHTMLDocument::StartDocumentLoad -- for XHTML
documents it needs to do the same thing as nsXMLDocument::StartDocumentLoad
does -- default to UTF-8 and not look at the "default charset".</quote>

Comment 3

[Johnny Stenback (:jst)]

Attachment: Default to UTF-8 if loading XHTML

Comment 4

[Johnny Stenback (:jst)]

Attachment: diff -w of the above for review.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 146033 [details] [diff] [review]
diff -w of the above for review.

r+sr=bzbarsky

Comment 6

[Bob Clary [:bc] (inactive)]

Is this something that is appropriate for 1.7? I guess that XHTML served as
application/xhtml+xml in UTF8 is not that common but ...

Comment 7

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 146033 [details] [diff] [review]
diff -w of the above for review.

I absolutely think we should land this in 1.7 -- it's a trivial patch and could
become a much bigger liability a year out.

Comment 8

[Johnny Stenback (:jst)]

Fix checked in on the trunk.

Comment 9

[Boris Zbarsky [:bzbarsky]]

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