Changeset 35807 in webkit for trunk/JavaScriptCore/kjs/InternalFunction.cpp

Timestamp:

Aug 17, 2008, 1:23:49 PM (17 years ago)

Author:

[email protected]

Message:

JavaScriptCore:

2008-08-17 Geoffrey Garen <​[email protected]>

Reviewed by Cameron Zwarich.

Made room for a free word in JSCell.

SunSpider says no change.

I changed JSCallbackObjectData, Arguments, JSArray, and RegExpObject to
store auxiliary data in a secondary structure.

I changed InternalFunction to store the function's name in the property
map.

I changed JSGlobalObjectData to use a virtual destructor, so WebCore's
JSDOMWindowBaseData could inherit from it safely. (It's a strange design
for JSDOMWindowBase to allocate an object that JSGlobalObject deletes,
but that's really our only option, given the size constraint.)

I also added a bunch of compile-time ASSERTs, and removed lots of comments
in JSObject.h because they were often out of date, and they got in the
way of reading what was actually going on.

Also renamed JSArray::getLength to JSArray::length, to match our style
guidelines.

WebCore:

2008-08-17 Geoffrey Garen <​[email protected]>

Reviewed by Cameron Zwarich.

Made room for a free word in JSCell.

Changed JSDOMWindowBase to store its auxiliary data in a subclass of
JSGlobalData, so the two could share a pointer.

Added a bunch of ASSERTs, to help catch over-sized objects.

WebKit/mac:

2008-08-17 Geoffrey Garen <​[email protected]>

Reviewed by Cameron Zwarich.

Made room for a free word in JSCell.

(Updated for JavaScriptCore changes.)

File:

• trunk/JavaScriptCore/kjs/InternalFunction.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/InternalFunction.cpp

@@ -29 +29 @@
 namespace KJS {
 
+ASSERT_CLASS_FITS_IN_CELL(InternalFunction);
+
 const ClassInfo InternalFunction::info = { "Function", 0, 0, 0 };
 
-InternalFunction::InternalFunction()
+InternalFunction::InternalFunction(ExecState* exec)
 {
+    putDirect(exec->propertyNames().name, jsString(exec, exec->propertyNames().nullIdentifier.ustring()), DontDelete | ReadOnly | DontEnum);
 }
 
-InternalFunction::InternalFunction(FunctionPrototype* prototype, const Identifier& name)
+InternalFunction::InternalFunction(ExecState* exec, FunctionPrototype* prototype, const Identifier& name)
     : JSObject(prototype)
-    , m_name(name)
 {
+    putDirect(exec->propertyNames().name, jsString(exec, name.ustring()), DontDelete | ReadOnly | DontEnum);
+}
+
+const UString& InternalFunction::name(ExecState* exec)
+{
+    JSValue* v = getDirect(exec->propertyNames().name);
+    ASSERT(v->isString());
+    return static_cast<JSString*>(v)->value();
 }
 
@@ -46 +56 @@
 }
 
-bool InternalFunction::getOwnPropertySlot(ExecState* exec, const Identifier& propertyName, PropertySlot& slot)
-{
-    if (propertyName == exec->propertyNames().name) {
-        slot.setCustom(this, nameGetter);
-        return true;
-    }
-
-    return JSObject::getOwnPropertySlot(exec, propertyName, slot);
-}
-
-void InternalFunction::put(ExecState* exec, const Identifier& propertyName, JSValue* value)
-{
-    if (propertyName == exec->propertyNames().name)
-        return;
-    JSObject::put(exec, propertyName, value);
-}
-
-bool InternalFunction::deleteProperty(ExecState* exec, const Identifier& propertyName)
-{
-    if (propertyName == exec->propertyNames().name)
-        return false;
-    return JSObject::deleteProperty(exec, propertyName);
-}
-
-JSValue* InternalFunction::nameGetter(ExecState* exec, const Identifier&, const PropertySlot& slot)
-{
-    InternalFunction* thisObj = static_cast<InternalFunction*>(slot.slotBase());
-    return jsString(exec, thisObj->functionName().ustring());
-}
-
 } // namespace KJS