Closed Bug 36633 Opened 25 years ago Closed 25 years ago

Crash in nsEnderEventListener::DispatchMouseEvent (changing style of an element in an onfocus event)

Categories

(Core :: DOM: UI Events & Focus Handling, defect, P3)

x86
All
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: rsalesas, Assigned: akkzilla)

References

Details

(Keywords: crash, testcase, Whiteboard: [nsbeta2+] WORKSFORME, NEEDS RETEST)

Attachments

(3 files)

The following code inside a document cause Mozilla to crash. Enough said. <html> <head> <title>Untitled</title> </head> <style> .noborder {border-width:2px;border-style:solid;border- color:white;overflow:hidden} .border {border-width:2px;border-style:inset} .mouseover {border-width:2px;border-style:solid;border- color:white;overflow:hidden;text-decoration:underline;color:blue} </style> <body> <input value="Click Me!" onfocus="this.className = 'border'" onblur="this.className = 'noborder'" > </body> </html>
updating component
Assignee: asadotzler → pierre
Component: Browser-General → Style System
QA Contact: jelwell → chrisd
Attached file testcase from initial report (deleted) —
Okay, checked this on: Win2000 2000-04-24-08 Mozilla build. No crash. No problem. Marking WORKSFORME.
Status: UNCONFIRMED → RESOLVED
Closed: 25 years ago
Resolution: --- → WORKSFORME
adding self to cc marking VERIFIED WORKSFORME
Status: RESOLVED → VERIFIED
Reopening. To reproduce, follow these steps: 1. load attachment into mozilla 2. click into the input field 3. click outside the input field 4. click into the input field again 5. crash
Status: VERIFIED → UNCONFIRMED
Resolution: WORKSFORME → ---
Confirming bug. Reproduced with M15. Submitted talkback report.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Changing OS to All.
OS: Windows 2000 → All
Adding crash keyword.
Keywords: crash
The above stacktrace is from a 4/28 build. The crash (segv) happens in #0 nsEnderEventListener::DispatchMouseEvent () #1 nsEnderEventListener::MouseDown () Changing summary to include nsEnderEventListener. Adding testcase keyword. Changing component to Event Handling, reassigning to akkana (owner of bug 28401 "Double mouse click events: rewrite nsEnderEventListener").
Assignee: pierre → akkana
Component: Style System → Event Handling
Keywords: testcase
Summary: Changing style of an element in onfocus event crashes mozilla → Crash in nsEnderEventListener::DispatchMouseEvent (changing style of an element in an onfocus event)
Mike is landing code in a few days (if he hasn't already) that completely eliminates this class, as I understand it. Is this blocking anyone right now? I'd rather wait and see if the problem still happens with Mike's code.
Marking dependency on the stuff Mike is working on. When that's in, I'll check this and see if the crash still happens.
Status: NEW → ASSIGNED
Depends on: 34896
Target Milestone: --- → M16
Whiteboard: [nsbeta2+]
Putting on [nsbeta2+] radar for beta2 fix.
Mike has his ender lite stuff (34896) just about ready to go, and several of us are testing it. But I can't reproduce the crash on my linux build even in a standard non-ender-lite build. I've tried clicking in the text widget, then outside it in the content area, back in the text widget, outside it in the urlbar, back in the widget, etc. No crashes. Is anyone still seeing this? Please retest and see if it's still happening, and if so, let me know what I'm doing wrong in trying to reproduce it.
Whiteboard: [nsbeta2+] → [nsbeta2+] WORKSFORME, NEEDS RETEST
Re-tested on PC/Linux. 2000051308 crashes, but 2000051520 works for me. 2000051712 is fine, too. Some noticable changes between 5/13 and 5/15 are that the curser changes to I-beam on mouseover the input text field now, and there are some messages appearing on the console when clicking in the unused content area: no handler found for HTML But the crash seems to be gone. Recommend WORKSFORME. BTW, I really love the new speed :-). Go, mozzy, go!
can't get this to crash on win32 either using the commercial build 2000051808. Akkana -- I'm marking as fixed
Status: ASSIGNED → RESOLVED
Closed: 25 years ago25 years ago
Resolution: --- → FIXED
verified fixt: 2000-05-22-09-M16 WinNT 2000-05-21-20-M16 Mac 2000-05-23-10-M16 Linux
Status: RESOLVED → VERIFIED
Adding keyword to bugs which already show a nsbeta2 triage value in the status whiteboard so the queries don't get screwed up.
Keywords: nsbeta2
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: