Closed Bug 21773 Opened 25 years ago Closed 25 years ago

[MLK] leaking XUL document on keypress

Categories

(Core :: XUL, defect, P3)

Product:
Core
Component:
XUL
Platform:
x86
Windows NT
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: waterson, Assigned: waterson)

Details

Attachments

(1 file)

proposed fix
5.52 KB, patch
Details | Diff | Splinter Review 
As soon as a nsXULKeyListener is created, we leak XUL documents galore.
Attached patch proposed fix — — Splinter Review 

    
  
Status: NEW → ASSIGNED
Target Milestone: M13

    
    

    
  
see attached fix.

1. dont_AddRef() result from nsSupportsHashTable.

2. add variable to condition creation of key listener that creates circular
reference.

r=shaver

    
  
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED

    
    

    
  
oops. the fix for this was checked in a while ago! marking fixed...

    
  
Status: RESOLVED → VERIFIED

    
    

    
  
marking verified

    
    

    
  
BULK MOVE: Changing component from XUL to XP Toolkit/Widgets: XUL.  XUL 
component will be deleted.
Component: XUL → XP Toolkit/Widgets: XUL

    
  
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: paulmac → xptoolkit.widgets

    
    

    
      
  

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: