153649 - Linebreaking inside quoted text is extremely slow, freezes Mozilla.

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect)

Description

[Aleksey Nogin]

Reproducible: always :-(

To reproduce:
0) Set MailNews preferences to use plain text compisition and to always quote
original text.
1) Press "Reply" on one of the messages.
2) Click to position the cursor inside the quoted text
3) Press "Enter".

Expected: Line is broken (or new line is added if cursor was at the end of the
line) without any noticible delay.

Actual: It takes several seconds and Mozilla is totally frozen while it lasts.

BuildID 2002061717 (trunk) on RedHat Linux 7.2 (+all updates) on Dual P-II/200
BuildID 2002062015 (trunk) on RedHat Linux 7.3+ on P-III/450.

I believe this is a recent regression, I do not have this on BuildID 20020610. I
would imagine this is a side-effect of bug 83378.

P.S. I've set severity to "major" for now, by IMHO it's "critical" - this makes
mail composition newrly unusable for me.

Comment 1

[Jean-Francois Ducarroz]

reassign to editor core

Comment 2

[Joe Francis]

You will be mine you will be mine all mine

Comment 3

[Joe Francis]

changing os to all, milestone to moz1.1a, nominating edbase

Comment 4

[Joe Francis]

The trunk is the wave of the future!

Comment 5

[Shotaro Kamio]

Attachment: Proposed patch

In this point, |newOffset| is not initialized variable.

Comment 6

[Andrew Schultz]

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

Comment 7

[Joe Francis]

Fix checked in to trunk.  The patch I had attached to 83378 (which is what I used 
to check in, though I missed a file) was the wrong version (read: unfinished).  
Meanwhiel I had the right version on my main machine and of course didnt see any 
problems.

Sorry for the snafu.  Should be all better now.

Comment 8

[Boris 'pi' Piwinger]

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1a+) Gecko/20020701

WFM

pi

Comment 9

[John Levon]

I have reproduced this problem once on a cvs build from a couple of days
ago, so it is still here under some circumstances. I'm cvs up'ing to
cvs trunk again now, to see if I can make a reproducable case.

Comment 10

[John Levon]

Ignore my last comment, it seems I was still running an earlier mozilla
at the time. I cannot reproduce the problem any more.

Comment 11

[rubydoo123]

per comment #10, marking verified