Bug 1254847 part 3. Make AutoEntryScript always take ownership of error reporting. r=bholley

This commit is contained in:
Boris Zbarsky
2016-03-09 19:02:03 -05:00
parent 2fd4be7bc7
commit 63091385d1
25 changed files with 23 additions and 64 deletions

View File

@@ -279,7 +279,6 @@ WrapperAnswer::RecvGet(const ObjectId& objId, const JSVariant& receiverVar,
// We may run scripted getters.
AutoEntryScript aes(scopeForTargetObjects(),
"Cross-Process Object Wrapper 'get'");
aes.TakeOwnershipOfErrorReporting();
JSContext* cx = aes.cx();
// The outparam will be written to the buffer, so it must be set even if
@@ -317,7 +316,6 @@ WrapperAnswer::RecvSet(const ObjectId& objId, const JSIDVariant& idVar, const JS
// We may run scripted setters.
AutoEntryScript aes(scopeForTargetObjects(),
"Cross-Process Object Wrapper 'set'");
aes.TakeOwnershipOfErrorReporting();
JSContext* cx = aes.cx();
RootedObject obj(cx, findObjectById(cx, objId));
@@ -379,7 +377,6 @@ WrapperAnswer::RecvCallOrConstruct(const ObjectId& objId,
{
AutoEntryScript aes(scopeForTargetObjects(),
"Cross-Process Object Wrapper call/construct");
aes.TakeOwnershipOfErrorReporting();
JSContext* cx = aes.cx();
// The outparam will be written to the buffer, so it must be set even if