Reviewed by Oliver.

        - fix remaining problems with Window shadowing

        * kjs/nodes.cpp:
        (VarDeclNode::evaluate): Tweak the special case a little.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24534 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/JavaScriptCore/ChangeLog b/JavaScriptCore/ChangeLog
index 03ba9572ec27026d0befbcdb02ba0eea6f431e44..924b43bd168c97ab09e255bbfa739aa9476eafcc 100644 (file)
--- a/JavaScriptCore/ChangeLog
+++ b/JavaScriptCore/ChangeLog
@@ -1,3 +1,12 @@
+2007-07-23  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Oliver.
+        
+        - fix remaining problems with Window shadowing
+
+        * kjs/nodes.cpp:
+        (VarDeclNode::evaluate): Tweak the special case a little.
+
 2007-07-23  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Oliver.

diff --git a/JavaScriptCore/kjs/nodes.cpp b/JavaScriptCore/kjs/nodes.cpp
index 9ffea7bd4ae9bdcef733f2bdb349a95286618623..622f4c3143c11d79a800760c97f1722f6d54a114 100644 (file)
--- a/JavaScriptCore/kjs/nodes.cpp
+++ b/JavaScriptCore/kjs/nodes.cpp
@@ -1623,7 +1623,17 @@ JSValue* VarDeclNode::evaluate(ExecState* exec)
 
     ASSERT(!chain.isEmpty());
 
-    if (init) {
+    bool inGlobalScope = ++chain.begin() == chain.end();
+
+    if (inGlobalScope && (init || !variableObject->getDirect(ident))) {
+        JSValue* val = init ? init->evaluate(exec) : jsUndefined();
+        int flags = Internal;
+        if (exec->context()->codeType() != EvalCode)
+            flags |= DontDelete;
+        if (varType == VarDeclNode::Constant)
+            flags |= ReadOnly;
+        variableObject->putDirect(ident, val, flags);
+    } else if (init) {
         JSValue* val = init->evaluate(exec);
         KJS_CHECKEXCEPTIONVALUE
             
@@ -1638,28 +1648,8 @@ JSValue* VarDeclNode::evaluate(ExecState* exec)
         if (varType == VarDeclNode::Constant)
             flags |= ReadOnly;
         
-        if (++chain.begin() == chain.end()) {
-            int flags = Internal;
-            if (exec->context()->codeType() != EvalCode)
-                flags |= DontDelete;
-            if (varType == VarDeclNode::Constant)
-                flags |= ReadOnly;
-            variableObject->putDirect(ident, val, flags);
-        } else {
-            ASSERT(variableObject->getDirect(ident) || ident == exec->propertyNames().arguments);
-            variableObject->put(exec, ident, val, flags);
-        }
-    } else {
-        if (++chain.begin() == chain.end()) {
-            int flags = Internal;
-            if (exec->context()->codeType() != EvalCode)
-                flags |= DontDelete;
-            if (varType == VarDeclNode::Constant)
-                flags |= ReadOnly;
-            variableObject->putDirect(ident, jsUndefined(), flags);
-        } else {
-
-        }
+        ASSERT(variableObject->getDirect(ident) || ident == exec->propertyNames().arguments);
+        variableObject->put(exec, ident, val, flags);
     }
 
     // no caller of this function actually uses the return value.