Changeset 96465 in webkit for trunk/Source/JavaScriptCore/runtime/JSGlobalData.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/runtime/JSGlobalData.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/runtime/JSGlobalData.h

@@ -296 +296 @@
 #endif
         void dumpRegExpTrace();
-        HandleSlot allocateGlobalHandle() { return heap.allocateGlobalHandle(); }
-        HandleSlot allocateLocalHandle() { return heap.allocateLocalHandle(); }
         void clearBuiltinStructures();
 
@@ -321 +319 @@
     };
 
-    inline HandleSlot allocateGlobalHandle(JSGlobalData& globalData)
-    {
-        return globalData.allocateGlobalHandle();
-    }
-
 #if ENABLE(GC_VALIDATION)
     inline bool JSGlobalData::isInitializingObject() const