Changeset 28053 in webkit for trunk/JavaScriptCore/kjs/function.cpp

Timestamp:

Nov 26, 2007, 3:48:14 PM (18 years ago)

Author:

[email protected]

Message:

Reviewed by Kevin McCullough.

Fixed <rdar://problem/5597937> REGRESSION (r27126): Drosera does not
show variables (can't enumerate ActivationImp properties)

Implemented a custome ActivationImp::getPropertyNames, since
ActivationImp now uses a custom property storage mechanism for local
variables.

• kjs/function.cpp: (KJS::ActivationImp::getPropertyNames):
• kjs/function.h:

File:

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

trunk/JavaScriptCore/kjs/function.cpp

@@ -28 +28 @@
 
 #include "ExecState.h"
+#include "JSGlobalObject.h"
+#include "PropertyNameArray.h"
 #include "debugger.h"
 #include "dtoa.h"
 #include "function_object.h"
 #include "internal.h"
-#include "JSGlobalObject.h"
 #include "lexer.h"
 #include "nodes.h"
@@ -454 +455 @@
 
     return JSObject::deleteProperty(exec, propertyName);
+}
+
+void ActivationImp::getPropertyNames(ExecState* exec, PropertyNameArray& propertyNames)
+{
+    SymbolTable::const_iterator::Keys end = m_symbolTable->end().keys();
+    for (SymbolTable::const_iterator::Keys it = m_symbolTable->begin().keys(); it != end; ++it)
+        propertyNames.add(Identifier(*it));
+
+    JSObject::getPropertyNames(exec, propertyNames);
 }