Changeset 99312 in webkit for trunk/Source/JavaScriptCore/API/JSValueRef.cpp

Timestamp:

Nov 4, 2011, 1:37:32 PM (14 years ago)

Author:

[email protected]

Message:

De-virtualize JSObject::hasInstance
​https://bugs.webkit.org/show_bug.cgi?id=71430

Reviewed by Darin Adler.

Added hasInstance to the MethodTable, changed all the virtual
implementations of hasInstance to static ones, and replaced
all call sites with corresponding lookups in the MethodTable.

• API/JSCallbackObject.h:
• API/JSCallbackObjectFunctions.h:

(JSC::::hasInstance):

• API/JSValueRef.cpp:

(JSValueIsInstanceOfConstructor):

• JavaScriptCore.exp:
• JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
• interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

• jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

• runtime/ClassInfo.h:
• runtime/JSBoundFunction.cpp:

(JSC::JSBoundFunction::hasInstance):

• runtime/JSBoundFunction.h:
• runtime/JSCell.cpp:

(JSC::JSCell::hasInstance):

• runtime/JSCell.h:
• runtime/JSObject.cpp:

(JSC::JSObject::hasInstance):

• runtime/JSObject.h:

File:

• trunk/Source/JavaScriptCore/API/JSValueRef.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/API/JSValueRef.cpp

@@ -176 +176 @@
     if (!jsConstructor->structure()->typeInfo().implementsHasInstance())
         return false;
-    bool result = jsConstructor->hasInstance(exec, jsValue, jsConstructor->get(exec, exec->propertyNames().prototype)); // false if an exception is thrown
+    bool result = jsConstructor->methodTable()->hasInstance(jsConstructor, exec, jsValue, jsConstructor->get(exec, exec->propertyNames().prototype)); // false if an exception is thrown
     if (exec->hadException()) {
         if (exception)