Changeset 13960 in webkit for trunk/JavaScriptCore/kjs/scope_chain.cpp

Timestamp:

Apr 18, 2006, 3:17:41 PM (19 years ago)

Author:

eseidel

Message:

2006-04-18 Eric Seidel <​[email protected]>

Reviewed by ggaren.

Fix "new Function()" to correctly use lexical scoping.
Add ScopeChain::print() function for debugging.
<rdar://problem/4067864> REGRESSION (125-407): JavaScript failure on PeopleSoft REN Server

• kjs/function_object.cpp: (FunctionObjectImp::construct):
• kjs/scope_chain.cpp: (KJS::ScopeChain::print):
• kjs/scope_chain.h:

File:

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

trunk/JavaScriptCore/kjs/scope_chain.cpp

@@ -22 +22 @@
 #include "config.h"
 #include "scope_chain.h"
+#include "reference_list.h"
 
 namespace KJS {
@@ -35 +36 @@
 }
 
+#ifndef NDEBUG
+
+void ScopeChain::print(ExecState* exec)
+{
+    ScopeChainIterator scopeEnd = end();
+    for (ScopeChainIterator scopeIter = begin(); scopeIter != scopeEnd; ++scopeIter) {
+        JSObject* o = *scopeIter;
+        ReferenceList propList = o->propList(exec, false);
+        ReferenceListIterator propEnd = propList.end();
+
+        fprintf(stderr, "----- [scope %p] -----\n", o);
+        for (ReferenceListIterator propIter = propList.begin(); propIter != propEnd; propIter++) {
+            Identifier name = propIter->getPropertyName(exec);
+            fprintf(stderr, "%s, ", name.ascii());
+        }
+        fprintf(stderr, "\n");
+    }
+}
+
+#endif
+
 } // namespace KJS