Changeset 34907 in webkit for trunk/JavaScriptCore/kjs/JSGlobalObject.cpp

Timestamp:

Jul 1, 2008, 12:01:51 AM (17 years ago)

Author:

[email protected]

Message:

2008-06-30 Geoffrey Garen <​[email protected]>

Reviewed by Oliver Hunt.

Fixed a global object leak caused by the switch to one register file.

Don't unconditionally mark the register file, since that logically
makes all global variables GC roots, even when their global object is
no longer reachable.

Instead, make the global object associated with the register file
responsible for marking the register file.

File:

• trunk/JavaScriptCore/kjs/JSGlobalObject.cpp (modified) (1 diff)

trunk/JavaScriptCore/kjs/JSGlobalObject.cpp

@@ -347 +347 @@
         (*it)->mark();
 
+    RegisterFile& registerFile = globalData()->machine->registerFile();
+    if (registerFile.globalObject() == this)
+        registerFile.mark(globalData()->heap);
+
     markIfNeeded(d()->globalExec->exception());