Changeset 32652 in webkit for trunk/JavaScriptCore/kjs/number_object.cpp

Timestamp:

Apr 28, 2008, 11:22:14 AM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Darin.

Fix run-webkit-tests --threading
and provisionally fix <​https://bugs.webkit.org/show_bug.cgi?id=18661>
Proxy server issue in Sunday's Nightly

Changed ClassInfo objects for built-in objects to hold a getter function returning
a per-thread instance. This makes it safe to share these ClassInfo objects between threads -
and these are the only ones that need to be shared.

File:

• trunk/JavaScriptCore/kjs/number_object.cpp (modified) (3 diffs)

trunk/JavaScriptCore/kjs/number_object.cpp

@@ -35 +35 @@
 // ------------------------------ NumberInstance ----------------------------
 
-const ClassInfo NumberInstance::info = { "Number", 0, 0 };
+const ClassInfo NumberInstance::info = { "Number", 0, 0, 0 };
 
 NumberInstance::NumberInstance(JSObject* proto)
@@ -451 +451 @@
 // ------------------------------ NumberObjectImp ------------------------------
 
-const ClassInfo NumberObjectImp::info = { "Function", &InternalFunctionImp::info, &numberTable };
+const ClassInfo NumberObjectImp::info = { "Function", &InternalFunctionImp::info, 0, ExecState::numberTable };
 
 /* Source for number_object.lut.h
@@ -474 +474 @@
 bool NumberObjectImp::getOwnPropertySlot(ExecState* exec, const Identifier& propertyName, PropertySlot& slot)
 {
-    return getStaticValueSlot<NumberObjectImp, InternalFunctionImp>(exec, &numberTable, this, propertyName, slot);
+    return getStaticValueSlot<NumberObjectImp, InternalFunctionImp>(exec, ExecState::numberTable(exec), this, propertyName, slot);
 }