Changeset 34457 in webkit for trunk/JavaScriptCore/VM/CodeBlock.h

Timestamp:

Jun 8, 2008, 5:57:28 PM (17 years ago)

Author:

[email protected]

Message:

2008-06-08 Cameron Zwarich <​[email protected]>

Reviewed by Darin.

Bug 19346: REGRESSION: Mootools 1.2 Class inheritance broken in post-SquirrelFish merge
<​https://bugs.webkit.org/show_bug.cgi?id=19346>

A check for whether a function's caller is eval code accidentally included
the case where the caller's caller is native code. Add a CodeType field to
CodeBlock and use this for the eval caller test instead.

JavaScriptCore:

• VM/CodeBlock.h: (KJS::CodeBlock::CodeBlock): (KJS::ProgramCodeBlock::ProgramCodeBlock): (KJS::EvalCodeBlock::EvalCodeBlock):
• VM/Machine.cpp: (KJS::getCallerFunctionOffset):
• kjs/nodes.cpp: (KJS::FunctionBodyNode::generateCode): (KJS::ProgramNode::generateCode):

LayoutTests:

• fast/js/function-dot-arguments-and-caller-expected.txt:
• fast/js/function-dot-arguments-and-caller.html:

File:

• trunk/JavaScriptCore/VM/CodeBlock.h (modified) (5 diffs)

trunk/JavaScriptCore/VM/CodeBlock.h

@@ -57 +57 @@
 
     struct CodeBlock {
-        CodeBlock(ScopeNode* ownerNode_)
+        CodeBlock(ScopeNode* ownerNode_, CodeType codeType_)
             : ownerNode(ownerNode_)
             , numTemporaries(0)
@@ -65 +65 @@
             , needsFullScopeChain(ownerNode_->usesEval() || ownerNode_->needsClosure())
             , usesEval(ownerNode_->usesEval())
+            , codeType(codeType_)
         {
         }
@@ -82 +83 @@
         bool needsFullScopeChain;
         bool usesEval;
+        CodeType codeType;
 
         Vector<Instruction> instructions;
@@ -102 +104 @@
 
     struct ProgramCodeBlock : public CodeBlock {
-        ProgramCodeBlock(ScopeNode* ownerNode, JSGlobalObject* globalObject_)
-            : CodeBlock(ownerNode)
+        ProgramCodeBlock(ScopeNode* ownerNode_, CodeType codeType_, JSGlobalObject* globalObject_)
+            : CodeBlock(ownerNode_, codeType_)
             , globalObject(globalObject_)
         {
@@ -119 +121 @@
 
     struct EvalCodeBlock : public ProgramCodeBlock {
-        EvalCodeBlock(ScopeNode* ownerNode, JSGlobalObject* globalObject_)
-            : ProgramCodeBlock(ownerNode, globalObject_)
+        EvalCodeBlock(ScopeNode* ownerNode_, JSGlobalObject* globalObject_)
+            : ProgramCodeBlock(ownerNode_, EvalCode, globalObject_)
         {
         }