Changeset 13960 in webkit for trunk/JavaScriptCore/kjs

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:

Location:

trunk/JavaScriptCore/kjs

Files:

• function_object.cpp (modified) (1 diff)
• scope_chain.cpp (modified) (2 diffs)
• scope_chain.h (modified) (2 diffs)

trunk/JavaScriptCore/kjs/function_object.cpp

@@ -205 +205 @@
 
   ScopeChain scopeChain;
-  scopeChain.push(exec->dynamicInterpreter()->globalObject());
+  scopeChain.push(exec->lexicalInterpreter()->globalObject());
   FunctionBodyNode *bodyNode = progNode.get();
 

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

trunk/JavaScriptCore/kjs/scope_chain.h

@@ -28 +28 @@
 
     class JSObject;
+    class ExecState;
     
     class ScopeChainNode {
@@ -80 +81 @@
         
         void mark();
+
+#ifndef NDEBUG        
+        void print(ExecState*);
+#endif
         
     private: