broken parsing on CDATA sections

RESOLVED FIXED

Status

--
major
RESOLVED FIXED
13 years ago
9 years ago

People

(Reporter: sayrer, Assigned: sayrer)

Tracking

(4 keywords)

1.8 Branch
x86
All
fixed1.8.0.2, fixed1.8.1, regression, verified1.8.1.3

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Updated

13 years ago
OS: Windows 2000 → All
(Assignee)

Comment 1

13 years ago
Created attachment 206525 [details] [diff] [review]
fix CDATA parsing for AtomIETF

This passes all of the testcases in http://atomtests.philringnalda.com/tests/item/title/titles.opml.

as reported here:
http://www.intertwingly.net/wiki/pie/TitleConformanceTests

We could probably use tests for all formats on this kind of thing, but that's another bug.
Attachment #206525 - Flags: review?(mscott)

Updated

13 years ago
Attachment #206525 - Flags: review?(mscott) → review+

Comment 2

13 years ago
after this has baked on the trunk for a little while, I'll look at moving this onto the appropriate branch(es).
Target Milestone: --- → Thunderbird2.0
(Assignee)

Updated

13 years ago
Whiteboard: [checkin needed]

Updated

13 years ago
Whiteboard: [checkin needed]

Comment 3

13 years ago
fixed on the trunk. leaving open so i can get this into thunderbird 2.0 after it has baked.

Updated

13 years ago
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Keywords: fixed1.8.0.1, fixed1.8.1, regression
Resolution: --- → FIXED

Comment 4

13 years ago
1.8.0.2 is technically the first release with this Thunderbird fix. 
Keywords: fixed1.8.0.1 → fixed1.8.0.2
verified fixed 1.8.1.3 using Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.8.1.3) Gecko/20070326 Thunderbird/2.0.0.0 Mnenhy/0.7.5.0 ID:2007032620 - the testcase from comment#o is working fine for me - i can`t reproduce the bug, so adding the verified keyword
Keywords: verified1.8.1.3
Should I have reopened this bug instead of bug 277243?  I don't _think_ so, but am not confident enough to skip asking here!

Updated

9 years ago
Component: RSS → Feed Reader
Product: Thunderbird → MailNews Core
Target Milestone: Thunderbird2.0 → ---
You need to log in before you can comment on or make changes to this bug.