Changeset 96465 in webkit for trunk/Source/JavaScriptCore/heap/Weak.h

Timestamp:

Oct 1, 2011, 5:54:56 PM (14 years ago)

Author:

[email protected]

Message:

Removed redundant helper functions for allocating Strong handles
​https://bugs.webkit.org/show_bug.cgi?id=69218

Reviewed by Sam Weinig.

../JavaScriptCore:

• heap/Heap.h:

(JSC::Heap::handleHeap):

• runtime/JSGlobalData.h: Removed these helper functions, since they

just created indirection.

• heap/StrongInlines.h: Added. Broke out a header for inline functions

to resolve circular dependencies created by inlining. I'm told this is
the future for JavaScriptCore.

• GNUmakefile.list.am:
• JavaScriptCore.gypi:
• JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
• JavaScriptCore.xcodeproj/project.pbxproj: Go forth and build.

• API/JSCallbackObjectFunctions.h:

(JSC::::init):

• runtime/WeakGCMap.h:

(JSC::WeakGCMap::add):
(JSC::WeakGCMap::set):

• runtime/StructureTransitionTable.h:

(JSC::StructureTransitionTable::setSingleTransition):

• heap/Local.h:

(JSC::::Local):

• heap/Strong.h:

(JSC::::Strong):
(JSC::::set):

• heap/Weak.h:

(JSC::Weak::Weak):
(JSC::Weak::set): Allocate handles directly instead of going through a
chain of forwarding functions.

• bytecompiler/BytecodeGenerator.cpp:
• runtime/JSGlobalData.cpp:
• runtime/LiteralParser.cpp:
• runtime/RegExpCache.cpp: Updated for header changes.

../JavaScriptGlue:

• JSRun.cpp:
• JSValueWrapper.cpp:

../WebCore:

• ForwardingHeaders/heap/StrongInlines.h: Added.
• bindings/js/JSCallbackData.h:
• bindings/js/JSDOMWindowShell.cpp:
• bindings/js/ScheduledAction.h:
• bindings/js/ScriptCachedFrameData.cpp:
• bindings/js/ScriptController.cpp:
• bindings/js/ScriptState.cpp:
• bindings/js/ScriptValue.h:
• bindings/js/WorkerScriptController.cpp:
• bridge/runtime_root.cpp:

../WebKit2:

• WebProcess/Plugins/Netscape/NPJSObject.cpp:
• WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp:

File:

• trunk/Source/JavaScriptCore/heap/Weak.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/heap/Weak.h

@@ -48 +48 @@
 
     Weak(JSGlobalData& globalData, ExternalType value = ExternalType(), WeakHandleOwner* weakOwner = 0, void* context = 0)
-        : Handle<T>(globalData.allocateGlobalHandle())
+        : Handle<T>(globalData.heap.handleHeap()->allocate())
     {
         HandleHeap::heapFor(slot())->makeWeak(slot(), weakOwner, context);
@@ -107 +107 @@
     {
         if (!slot()) {
-            setSlot(globalData.allocateGlobalHandle());
+            setSlot(globalData.heap.handleHeap()->allocate());
             HandleHeap::heapFor(slot())->makeWeak(slot(), weakOwner, context);
         }