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

trunk/Source/JavaScriptCore/heap/Local.h

@@ -58 +58 @@
 
 template <typename T> inline Local<T>::Local(JSGlobalData& globalData, ExternalType value)
-    : Handle<T>(globalData.allocateLocalHandle())
+    : Handle<T>(globalData.heap.handleStack()->push())
 {
     set(value);
@@ -64 +64 @@
 
 template <typename T> inline Local<T>::Local(JSGlobalData& globalData, Handle<T> other)
-    : Handle<T>(globalData.allocateLocalHandle())
+    : Handle<T>(globalData.heap.handleStack()->push())
 {
     set(other.get());