Changeset 16117 in webkit for trunk/JavaScriptCore/kjs/function.cpp

Timestamp:

Aug 30, 2006, 11:45:07 AM (19 years ago)

Author:

ggaren

Message:

JavaScriptCore:

Reviewed, tweaked by ggaren.

• Added storage and accessor functions for ExecState as a fix for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=10114

• kjs/ExecState.cpp: (KJS::ExecState::ExecState):
• kjs/ExecState.h:
• kjs/context.h: (KJS::Context::setExecState): (KJS::Context::execState):

LayoutTests:

Reviewed by ggaren.

• Updated test to check for exceptions thrown in a plug-in. This is a test for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=10114.
• plugins/bindings-test-expected.txt:
• plugins/bindings-test.html:

WebCore:

Reviewed, tweaked by ggaren.

• Changed to use ExecState on current context rather than global. Part of the fix for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=10114
• bindings/objc/WebScriptObject.mm: (+[WebScriptObject throwException:]): (-[WebScriptObject setException:]):

File:

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

trunk/JavaScriptCore/kjs/function.cpp

@@ -83 +83 @@
   if (exec->hadException())
     newExec.setException(exec->exception());
+  ctx.setExecState(&newExec);
 
   // assign user supplied arguments to parameters
@@ -819 +820 @@
                        EvalCode,
                        exec->context());
-        
         ExecState newExec(exec->dynamicInterpreter(), &ctx);
         if (exec->hadException())
             newExec.setException(exec->exception());
+        ctx.setExecState(&newExec);
         
         // execute the code