Changeset 11447 in webkit for trunk/JavaScriptCore/kjs/object.h

Timestamp:

Dec 4, 2005, 3:28:56 PM (19 years ago)

Author:

andersca

Message:

2005-12-04 Anders Carlsson <​[email protected]>

Reviewed by Geoffrey.

• Fixes <​http://bugzilla.opendarwin.org/show_bug.cgi?id=3999>

• kjs/object.cpp: (KJS::ObjectImp::canPut): Refactor to use getPropertyAttributes.

(KJS::ObjectImp::propertyIsEnumerable):
New function which checks if a property is enumerable.

(KJS::ObjectImp::getPropertyAttributes):

• kjs/object.h: Add getPropertyAttributes and propertyIsEnumerable.

• kjs/object_object.cpp: (ObjectPrototypeImp::ObjectPrototypeImp): (ObjectProtoFuncImp::callAsFunction):
• kjs/object_object.h: (KJS::ObjectProtoFuncImp::): Add propertyIsEnumerable to the Object prototype.

File:

• trunk/JavaScriptCore/kjs/object.h (modified) (2 diffs)

trunk/JavaScriptCore/kjs/object.h

@@ -247 +247 @@
 
     /**
+     * Checks if a property is enumerable, that is if it doesn't have the DontEnum
+     * flag set
+     *
+     * See ECMA 15.2.4
+     * @param exec The current execution state
+     * @param propertyName The name of the property
+     * @return true if the property is enumerable, otherwise false
+     */
+    bool propertyIsEnumerable(ExecState *exec, const Identifier &propertyName) const;
+
+    /**
      * Checks to see whether the object (or any object in it's prototype chain)
      * has a property with the specified name.
@@ -455 +466 @@
     ObjectImp *toObject(ExecState *exec) const;
 
+    bool getPropertyAttributes(const Identifier& propertyName, int& attributes) const;
+    
     // This get function only looks at the property map.
     // This is used e.g. by lookupOrCreateFunction (to cache a function, we don't want