124294 - Slew of linebreak assertions from serializer code

Status: VERIFIED DUPLICATE of bug 94475

(Core :: DOM: Serializers, defect)

Description

[Simon Fraser [no longer active]]

I'm seeing a slew of linebreak assertions:

###!!! ASSERTION: aPos > aLen: 'aPos <= aLen', file nsJISx4501LineBreaker.cpp, 
line 466

which come from the serializer calling the linebreaker code at:

          result = mLineBreaker->Next(mCurrentLine.get(), 
                                      mCurrentLine.Length(), goodSpace,
                                      (PRUint32 *) &goodSpace, &oNeedMoreText);


line 1240 of nsPlainTextSerializer.cpp. We're passing 'goodSpace' values which 
are much longer than mCurrentLine.Length().

To repro this, copy and paste HTML source, for exampe from "HTML source view" in 
composer, into the 'Description' field of a bug entry page. Now activate and 
deactive the window; these assertions happen when we serialize the contents of 
the textarea.

Comment 1

[Simon Fraser [no longer active]]

Attachment: The HTML source with which I can repro the problem (long lines?)

Comment 2

[Tanu Mutreja]

Is this Mac specific bug? I'm unable to reproduce it on WinNT.

Comment 3

[Simon Fraser [no longer active]]

It could be, since Mac's linebreak is \r, not \r\n as on Windows.

Comment 4

[Tanu Mutreja]

As we don't have a Mac machine here, better to be assigned to the right person 
there.

Comment 5

[Simon Fraser [no longer active]]

Giving back. This can be repro'd on Windows; see bug 124984.

Comment 6

[Simon Fraser [no longer active]]

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

Comment 7

[Tanu Mutreja]

Cause of this bug is same as the one for bug# 94475 and patch uploaded there 
fixes this too.

So even though the bug descriptions look different, I'm marking it as a dupe of 
94475.

*** This bug has been marked as a duplicate of 94475 ***

Comment 8

[sujay]

verified.