Changeset 14896 in webkit for trunk/JavaScriptCore/kjs/debugger.cpp

Timestamp:

Jun 17, 2006, 3:48:33 AM (19 years ago)

Author:

ddkilzer

Message:

JavaScriptCore:

Reviewed by ggaren.

​http://bugzilla.opendarwin.org/show_bug.cgi?id=9476
REGRESSION: Reproducible crash after closing window after viewing
css2.1/t0803-c5501-imrgn-t-00-b-ag.html

• kjs/debugger.cpp: (Debugger::detach): Call setDebugger(0) for all interpreters removed from the 'attached to a debugger' list.

File:

• trunk/JavaScriptCore/kjs/debugger.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/debugger.cpp

@@ -69 +69 @@
 void Debugger::detach(Interpreter* interp)
 {
-  if (interp && interp->debugger() == this)
-    interp->setDebugger(0);
-
   // iterate the addresses where AttachedInterpreter pointers are stored
   // so we can unlink items from the list
@@ -79 +76 @@
     if (!interp || q->interp == interp) {
       *p = q->next;
+      q->interp->setDebugger(0);
       delete q;
-    } else {
+    } else
       p = &q->next;
-    }
   }
 }