Closed
Bug 142918
Opened 22 years ago
Closed 22 years ago
crash in button_onblur test case within frames
Categories
(Core :: DOM: UI Events & Focus Handling, defect)
Tracking
()
VERIFIED
FIXED
mozilla1.0
People
(Reporter: danm.moz, Assigned: danm.moz)
References
()
Details
(Keywords: topembed+, Whiteboard: [adt3 RTM])
Attachments
(1 file)
1020 bytes,
patch
|
saari
:
review+
jst
:
superreview+
blizzard
:
approval+
|
Details | Diff | Splinter Review |
An embedding client which Shall Not Be Named, and I don't mean mfcEmbed, has been experiencing crashes in nsEventStateManager. Steps to reproduce, taken from bugscape 14292: 1. launch mystery client 2. logon, navigate to http://bubblegum.mcom.com/browser/standards/html-transitional/tests/ 3. scroll down in the left hand frame until you see the test case 'button_onblur.html" click it to load. 4. when page loads, hit 'this is button2. click on me" Result: Client crashes immediately after the results dialog pops up. 100% reproducible, so far only tested on win2k. Not reproducible in mfcembed from 4/19
Severity: normal → blocker
Status: NEW → ASSIGNED
Keywords: topembed+
QA Contact: rakeshmishra → jmkobayashi
Target Milestone: --- → mozilla1.0
I still can't reproduce this crash in a debug build of the Mystery Client from today's source. But other folks continue to see the crash. This patch should stop it.
Comment 2•22 years ago
|
||
Can you attach a screenshot of that mystery client if you can't give us the name ?
Comment 3•22 years ago
|
||
Comment on attachment 82751 [details] [diff] [review] guard gLastFocusedContent against null sr=jst
Attachment #82751 -
Flags: superreview+
Updated•22 years ago
|
QA Contact: jmkobayashi → moied
Comment 4•22 years ago
|
||
Internal Reference: http://bugscape.mcom.com/show_bug.cgi?id=14292
Comment 5•22 years ago
|
||
Comment on attachment 82751 [details] [diff] [review] guard gLastFocusedContent against null r=saari
Attachment #82751 -
Flags: review+
Comment 6•22 years ago
|
||
Comment on attachment 82751 [details] [diff] [review] guard gLastFocusedContent against null a=blizzard on behalf of drivers for the 1.0 branch
Attachment #82751 -
Flags: approval+
Well this is interesting. The prescribed test case on bubblegum.mcom.com has never crashed on me. But this reduced test case: <html><body> <form> <button onBlur="alert('button 1 blurred');">blur alert button</button> </form> </body></html> which is borrowed straight out of the bubblegum test case, does crash the Mystery Client for me *and* it crashes Mozilla. Huh. And handily, the patch fixes both. Fix is checked in on the trunk.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 8•22 years ago
|
||
adt1.0.0+ approval (on ADT's hebalf) for checkin into the 1.0 branch .
Keywords: fixed1.0.0
Tested URL and testcases I don't see any problem and testcases with Mozilla trunk 20020517 on winxp. marking Verified
Status: RESOLVED → VERIFIED
Comment 10•22 years ago
|
||
Verified on the trunk build 2002-05-22-08-trunk on Windows 2000 , with the URL and the testcase.
Comment 11•22 years ago
|
||
Verified on the branch build 2002-05-22-08-1.0.0 on Windows 2000 , with the URL and the testcase. adding verified1.0.0 to the keyword
Keywords: verified1.0.0
Updated•22 years ago
|
QA Contact: moied → rakeshmishra
Updated•5 years ago
|
Component: Event Handling → User events and focus handling
You need to log in
before you can comment on or make changes to this bug.
Description
•