Mac charsets: save page converts line-breaks in source-code to 


VERIFIED FIXED

Status

()

VERIFIED FIXED
11 years ago
10 years ago

People

(Reporter: phiw2, Assigned: smontagu)

Tracking

Trunk
PowerPC
Mac OS X
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

11 years ago
STR

1. load page in URL (displays incorrectly, bug 398905)
2. Save page to disc
3. Open in text editor

Actual result: All line-breaks in the source code are converted to 

e.g. <tr class="testcase">&#10;    <td>0x21</td>
Either a bug in the serializer or (more likely?) in the i18n converters that it uses, I'd think.
Assignee: nobody → smontagu
Component: Layout: Fonts and Text → Internationalization
QA Contact: layout.fonts-and-text → i18n
(Assignee)

Comment 3

11 years ago
Created attachment 285122 [details] [diff] [review]
Patch

This is just adding to nsUnicodeEncodeHelper what nsUnicodeDecodeHelper already does: http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/intl/uconv/util/nsUnicodeDecodeHelper.cpp&mark=81-83#62
Attachment #285122 - Flags: review?(jshin1987)
(Assignee)

Comment 4

11 years ago
Also affected are MacCroatian, MacCyrillic, MacGreek, MacIcelandic, MacRoman, MacRomanian, MacTurkish, and MacUkrainian.
Status: NEW → ASSIGNED
Summary: MacGurmukhi: save page converts line-breaks in source-code to &#10; → Mac charsets: save page converts line-breaks in source-code to &#10;

Updated

11 years ago
Attachment #285122 - Flags: review?(jshin1987) → review+
(Assignee)

Comment 5

11 years ago
Checked in to trunk, with unit test.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
(Reporter)

Comment 6

11 years ago
verified with
Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1a1pre) Gecko/2008060902 Minefield/3.1a1pre
Status: RESOLVED → VERIFIED

Updated

10 years ago
Depends on: 457886
You need to log in before you can comment on or make changes to this bug.