[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdelibs/khtml/ecma
From:       David Faure <faure () kde ! org>
Date:       2003-01-15 23:57:43
[Download RAW message or body]

CVS commit by faure: 

WAAAAH - big memory leak, jseventlisteners were never deleted.
Most of the debugging done by Dirk.


  M +3 -0      kjs_window.cpp   1.311


--- kdelibs/khtml/ecma/kjs_window.cpp  #1.310:1.311
@@ -268,4 +268,5 @@ Window::Window(KHTMLPart *p)
   : ObjectImp(/*no proto*/), m_part(p), screen(0), history(0), m_frames(0), loc(0), \
m_evt(0)  {
+  jsEventListeners.setAutoDelete(true);
   winq = new WindowQObject(this);
   //kdDebug(6070) << "Window::Window this=" << this << " part=" << m_part << " " << \
m_part->name() << endl; @@ -1023,4 +1024,6 @@ void Window::clear( ExecState *exec )
   // Get rid of everything, those user vars could hold references to DOM nodes
   deleteAllProperties( exec );
+  jsEventListeners.clear();
+
   // Really delete those properties, so that the DOM nodes get deref'ed
   while(KJS::Interpreter::collect())


[prev in list] [next in list] [prev in thread] [next in thread]