Closed Bug 153649 Opened 22 years ago Closed 22 years ago

Linebreaking inside quoted text is extremely slow, freezes Mozilla.

Categories

(Core :: DOM: Editor, defect)

x86
All
defect
Not set
major

Tracking

()

VERIFIED FIXED
mozilla1.1beta

People

(Reporter: mozilla-bugs, Assigned: mozeditor)

References

Details

(Keywords: perf, Whiteboard: [EDITORBASE])

Attachments

(1 file)

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.
Depends on: 83378
Keywords: perf
reassign to editor core
Assignee: ducarroz → kin
Component: Composition → Editor: Core
Product: MailNews → Browser
QA Contact: esther → sujay
You will be mine you will be mine all mine
Assignee: kin → jfrancis
changing os to all, milestone to moz1.1a, nominating edbase
Status: NEW → ASSIGNED
OS: Linux → All
Whiteboard: [EDITORBASE]
Target Milestone: --- → mozilla1.1alpha
The trunk is the wave of the future!
Target Milestone: mozilla1.1alpha → mozilla1.1beta
Attached patch Proposed patch (deleted) — Splinter Review
In this point, |newOffset| is not initialized variable.
*** Bug 154742 has been marked as a duplicate of this bug. ***
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.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1a+) Gecko/20020701 WFM pi
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.
Ignore my last comment, it seems I was still running an earlier mozilla at the time. I cannot reproduce the problem any more.
per comment #10, marking verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: