Changeset 94811 in webkit for trunk/Source/JavaScriptGlue/JSRun.cpp

Timestamp:

Sep 8, 2011, 3:38:44 PM (14 years ago)

Author:

[email protected]

Message:

Remove the Completion object from JSC, I have never liked it
​https://bugs.webkit.org/show_bug.cgi?id=67755

Reviewed by Gavin Barraclough.

../JavaScriptCore:

• Removes the Completion object and replaces its use with out parameter exceptions.
• Remove ComplType and virtual exceptionType() function on JSObject. Replace with ClassInfo for InterruptedExecutionError and TerminatedExecutionError.

• API/JSBase.cpp:

(JSEvaluateScript):
(JSCheckScriptSyntax):

• JavaScriptCore.exp:
• JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
• interpreter/Interpreter.cpp:

(JSC::Interpreter::throwException):

• jsc.cpp:

(functionLoad):
(functionCheckSyntax):
(runWithScripts):
(runInteractive):

• runtime/Completion.cpp:

(JSC::checkSyntax):
(JSC::evaluate):

• runtime/Completion.h:
• runtime/ExceptionHelpers.cpp:

(JSC::InterruptedExecutionError::toString):
(JSC::TerminatedExecutionError::toString):
(JSC::createInterruptedExecutionException):

• runtime/ExceptionHelpers.h:

(JSC::InterruptedExecutionError::InterruptedExecutionError):
(JSC::InterruptedExecutionError::create):
(JSC::InterruptedExecutionError::createStructure):
(JSC::TerminatedExecutionError::TerminatedExecutionError):
(JSC::TerminatedExecutionError::create):
(JSC::TerminatedExecutionError::createStructure):

• runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::JSGlobalData):

• runtime/JSObject.h:

../JavaScriptGlue:

• JSRun.cpp:

(JSRun::Evaluate):
(JSRun::CheckSyntax):

• JSRun.h:
• JavaScriptGlue.cpp:

(JSRunEvaluate):

../WebCore:

• bindings/js/JSDOMBinding.cpp:

(WebCore::reportException):

• bindings/js/JSEventListener.cpp:

(WebCore::JSEventListener::handleEvent):

• bindings/js/JSInjectedScriptManager.cpp:

(WebCore::InjectedScriptManager::createInjectedScript):

• bindings/js/JSMainThreadExecState.h:

(WebCore::JSMainThreadExecState::evaluate):

• bindings/js/ScriptController.cpp:

(WebCore::ScriptController::evaluateInWorld):

• bindings/js/WorkerScriptController.cpp:

(WebCore::WorkerScriptController::evaluate):

• bindings/objc/WebScriptObject.mm:

(-[WebScriptObject evaluateWebScript:]):

• bridge/NP_jsobject.cpp:

(_NPN_Evaluate):

• bridge/jni/jni_jsobject.mm:

(JavaJSObject::eval):

../WebKit/mac:

• Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::evaluate):

../WebKit/qt:

• Api/qwebelement.cpp:

(QWebElement::evaluateJavaScript):

../WebKit2:

• WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp:

(WebKit::NPRuntimeObjectMap::evaluate):

File:

• trunk/Source/JavaScriptGlue/JSRun.cpp (modified) (1 diff)

trunk/Source/JavaScriptGlue/JSRun.cpp

@@ -68 +68 @@
 }
 
-Completion JSRun::Evaluate()
+JSValue JSRun::Evaluate(JSValue* expception)
 {
-    return JSC::evaluate(fGlobalObject->globalExec(), fGlobalObject->globalScopeChain(), makeSource(fSource));
+    return JSC::evaluate(fGlobalObject->globalExec(), fGlobalObject->globalScopeChain(), makeSource(fSource), JSValue(), expception);
 }
 
 bool JSRun::CheckSyntax()
 {
-    return JSC::checkSyntax(fGlobalObject->globalExec(), makeSource(fSource)).complType() != Throw;
+    return JSC::checkSyntax(fGlobalObject->globalExec(), makeSource(fSource));
 }