Closed Bug 5533 Opened 26 years ago Closed 26 years ago

Crasher-Compose, type text, press spacebar apprunner freezes

Categories

(MailNews Core :: Composition, defect, P1)

All
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: nbaca, Assigned: mozeditor)

Details

(Whiteboard: QA BLOCKER)

Build Date & Platform:NT4.0 and Linux, build 042608 Description: In Mail select the New Message button, click onto the message body and type some text and press the space bar. As soon as the space bar is pressed then apprunner freezes. I can reproduce this on my NT4 and Linux system.
Severity: normal → critical
Component: Front End → Composition
cc: sujay.
RE: win32 on win_nt 4.0 and Linux 2.0 (both are dated 1999-04-26-08) The behavior is inconsistent. Three out of five times I was able to type text and send. On Linux, it crashes once out 3 times when type text and hit space bar. Both win_nt 4.0 and Linux showed that: any text typed after the square will not be displayed in the received message (Please refer to bug 5384)
Assignee: phil → kostello
Sounds like an editor issue. Reassigning to kostello.
Target Milestone: M5
Icky. M5 (pretty please?)
Assignee: kostello → buster
Steve, sorry about the last minute change. Could you please look at this bug for M5. Thanks, Greg
Status: NEW → ASSIGNED
Priority: P3 → P1
here is a reproducable test case that does not involve mail. 1. apprunner -editor 2. Ctrl-A 3. Del 4. type one character 5. type space The problem is nsHTMLEditRules::NextNodeInBlock is always returning the node it was passed in. This causes nsHTMLEditRules::IsNextCharWhitespace into an infinite loop. I think this can be fixed with a simple check for node!=aNode: line 989: if (node && IsTextOrElementNode(node) && (node != blockParent) && (node!=aNode)) but I want to talk to Joe about it before I check it in. Also, I think Joe should do a code review of any similar code he's written to make sure this case isn't lurking somewhere else.
QA Contact: 4080 → 4079
Since there is a case of this that does not involve mail, changing QA contact to sujay.
Summary: Compose, type text and apprunner freezes → Crasher - Compose, type text and apprunner freezes
Summary: Crasher - Compose, type text and apprunner freezes → Crasher-Compose, type text, press spacebar apprunner freezes
Whiteboard: QA BLOCKER
Adding to QA Blocker radar.
Assignee: buster → jfrancis
Status: ASSIGNED → NEW
still haven't heard from Joe, so I'm going to check in my change, and let him take it from there. Please code review. If that's it, just mark this fixed. If it's the wrong fix or more work needs to get done, then it's all yours.
accepting bugs
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
buster checked in a fix for this earlier today. i owe him dinner.
Status: RESOLVED → VERIFIED
verified in 4/29 build.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.