wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 828244 - Part b: Pass MutableHandleValue to Debugger::resultToCompletion; r=terrence

Browse Source
This commit is contained in:
Ms2ger 2013-01-11 09:42:59 +01:00
parent 1b3f955cca
commit 1371edcb3c
2 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
js/src/vm/Debugger.cpp
View File

@ -517,7 +517,7 @@ Debugger::slowPathOnLeaveFrame(JSContext *cx, bool frameOk)
/* Save the frame's completion value. */
JSTrapStatus status;
RootedValue value(cx);
Debugger::resultToCompletion(cx, frameOk, fp->returnValue(), &status, value.address());
Debugger::resultToCompletion(cx, frameOk, fp->returnValue(), &status, &value);
/* Build a list of the recipients. */
AutoObjectVector frames(cx);
@ -756,20 +756,20 @@ Debugger::handleUncaughtException(Maybe<AutoCompartment> &ac, Value *vp, bool ca
void
Debugger::resultToCompletion(JSContext *cx, bool ok, const Value &rv,
JSTrapStatus *status, Value *value)
JSTrapStatus *status, MutableHandleValue value)
{
JS_ASSERT_IF(ok, !cx->isExceptionPending());
if (ok) {
*status = JSTRAP_RETURN;
*value = rv;
value.set(rv);
} else if (cx->isExceptionPending()) {
*status = JSTRAP_THROW;
*value = cx->getPendingException();
value.set(cx->getPendingException());
cx->clearPendingException();
} else {
*status = JSTRAP_ERROR;
value->setUndefined();
value.setUndefined();
}
}
@ -822,7 +822,7 @@ Debugger::receiveCompletionValue(Maybe<AutoCompartment> &ac, bool ok, Value val,
JSContext *cx = ac.ref().context();
JSTrapStatus status;
Value value;
RootedValue value(cx);
resultToCompletion(cx, ok, val, &status, &value);
ac.destroy();
return newCompletionValue(cx, status, value, vp);

2
js/src/vm/Debugger.h
View File

@ -464,7 +464,7 @@ class Debugger : private mozilla::LinkedListElement<Debugger>
* to be false).
*/
static void resultToCompletion(JSContext *cx, bool ok, const Value &rv,
JSTrapStatus *status, Value *value);
JSTrapStatus *status, MutableHandleValue value);
/*
* Set |*result| to a JavaScript completion value corresponding to |status|