Changeset 4198 in webkit for trunk/JavaScriptCore/kjs/nodes.cpp

Timestamp:

Apr 25, 2003, 11:05:37 PM (22 years ago)

Author:

darin

Message:

• fixed 3241344 -- REGRESSION: top of page missing on wired.com and cnn.com

Caused by the ResolveNode speedup. Roll it out until I can figure out why.

• kjs/nodes.cpp: (ResolveNode::evaluate): Go back to using evaluateReference.

File:

• trunk/JavaScriptCore/kjs/nodes.cpp (modified) (1 diff)

trunk/JavaScriptCore/kjs/nodes.cpp

@@ -222 +222 @@
 Value ResolveNode::evaluate(ExecState *exec)
 {
-  // This is the same as calling evaluateReference(exec).getValue(exec),
-  // only considerably faster.
-  
-  ScopeChain chain = exec->context().imp()->scopeChain();
-
-  while (!chain.isEmpty()) {
-    ObjectImp *o = chain.top();
-    Value result = o->get(exec, ident);
-    if (result.type() != UndefinedType)
-      return result;
-    chain.pop();
-  }
-
-  return Reference(Null(), ident).getValue(exec);
+  return evaluateReference(exec).getValue(exec);
 }