Changeset 34810 in webkit for trunk/JavaScriptCore/kjs/JSGlobalData.h

Timestamp:

Jun 26, 2008, 11:59:26 AM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Maciej.

Make JSGlobalData per-thread.

No change on SunSpider total.

• wtf/ThreadSpecific.h: Re-enabled the actual implementation.

• kjs/JSGlobalObject.cpp: (KJS::JSGlobalObject::~JSGlobalObject): Re-added a JSLock-related assertion. We'll probably want to preserve these somehow to keep legacy behavior in working condition. (KJS::JSGlobalObject::init): Initialize globalData pointer earlier, so that it is ready when updating JSGlobalObject linked list.

• kjs/JSGlobalObject.h: (KJS::JSGlobalObject::head): Changed head() to be non-static, and to use JSGlobalData associated with the current object.

• kjs/InitializeThreading.cpp: (KJS::initializeThreadingOnce): Removed a no longer needed Heap::registerAsMainThread() call.

• kjs/JSGlobalData.h: Removed a lying lie comment - parserObjectExtraRefCounts is not transient, and while newParserObjects may conceptually be such, there is still some node manipulation going on outside Parser::parse which touches it.

• kjs/JSGlobalData.cpp: (KJS::JSGlobalData::~JSGlobalData): Delete recently added members. (KJS::JSGlobalData::sharedInstance): Actually use a separate instance.

• kjs/collector.cpp: (KJS::Heap::Heap): (KJS::Heap::~Heap): Added a destructor, which unconditionally deletes everything. (KJS::Heap::sweep): Removed code related to "collect on main thread only" logic. (KJS::Heap::collect): Ditto. (KJS::Heap::globalObjectCount): Explicitly use per-thread instance of JSGlobalObject linked list now that JSGlobalObject::head() is not static. Curently, WebCoreStatistics methods only work with the main thread currently anyway. (KJS::Heap::protectedGlobalObjectCount): Ditto.

• kjs/collector.h: Removed code related to "collect on main thread only" logic.

• JavaScriptCore.exp: Removed Heap::collectOnMainThreadOnly.

File:

• trunk/JavaScriptCore/kjs/JSGlobalData.h (modified) (1 diff)

trunk/JavaScriptCore/kjs/JSGlobalData.h

@@ -73 +73 @@
         const ArgList emptyList; // Lists are supposed to be allocated on the stack to have their elements properly marked, which is not the case here - but this list has nothing to mark.
 
-        // Tracked object sets are transient, only needed when parsing.
         HashSet<ParserRefCounted*>* newParserObjects;
         HashCountedSet<ParserRefCounted*>* parserObjectExtraRefCounts;