464958 - [FIX]set characterSet of XML DOM document created with createDocument() to UTF-8 and not ISO-8859-1

Status: VERIFIED FIXED

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

Description

[Daniel Veditz [:dveditz]]

+++ This bug was initially created as a clone of Bug #431701 +++

regression in Firefox 3.0.4 from bug 431701. See bug 431701 comment 36 and following.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: Fix

A little scary, but seems to be ok on trunk, and the only way to get Fx2-compatible behavior without backing out the whole "serialize in document charset" thing...

Comment 2

[Andrea Lombardoni]

Hey, it is not technical relevant, but THANK YOU (this bug caused most of our applications to fail :)

I am impressed by the speed and I LOVE the transparency of the process.

Comment 3

[Al Billings [:abillings - ex-MoCo]]

QA requests a manual testcase or site to cleanly repro this problem. Please make my life easier. :-)

Comment 4

[Boris Zbarsky [:bzbarsky]]

Al, the unit test (with change) will really catch the issue...  I'll see if I can write up an example, I guess, but basically you want to createDocument() a document, put in some non-ASCII text, then use XHR to send it to a server that echoes the input as UTF-8 and see whether the text shows up right.  Not really that simple to set up, sadly.

Comment 5

[Al Billings [:abillings - ex-MoCo]]

If it is too complex, don't worry about it since there is a unit test for it. I just like to see these fixes for myself when possible. I can always check the logs like anyone else for the unit tests.

Comment 6

[João Nelas]

I'm seeing the same problem with our applications and I'm running English(US) Macintosh version.

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.0.4) Gecko/2008102920 Firefox/3.0.4

Comment 7

[Daniel Veditz [:dveditz]]

Comment on attachment 348233 [details] [diff] [review]
Fix

Approved for 1.9.0.5, a=dveditz for release-drivers

Comment 8

[Boris Zbarsky [:bzbarsky]]

Fixed on 1.9 branch for 1.9.0.5.

Comment 9

[Al Billings [:abillings - ex-MoCo]]

Marking verified for 1.9.0.5 as the unit test has been passing for 1.9.0.