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 1045646 - Part 5: Remove AutoPushJSContext. r=bholley

Browse Source
This commit is contained in:
Bob Owen 2014-07-31 11:06:23 +01:00
parent cac503438b
commit c500510d3a
2 changed files with 0 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
js/xpconnect/src/nsCxPusher.cpp
View File

@ -254,11 +254,4 @@ ThreadsafeAutoSafeJSContext::operator JSContext*() const
}
}
AutoPushJSContext::AutoPushJSContext(JSContext *aCx) : mCx(aCx)
{
if (mCx && mCx != nsXPConnect::XPConnect()->GetCurrentJSContext()) {
mPusher.construct(mCx);
}
}
} // namespace mozilla

23
js/xpconnect/src/nsCxPusher.h
View File

@ -155,29 +155,6 @@ private:
MOZ_DECL_USE_GUARD_OBJECT_NOTIFIER
};
/**
* Use AutoPushJSContext when you want to use a specific JSContext that may or
* may not be already on the stack. This differs from nsCxPusher in that it only
* pushes in the case that the given cx is not the active cx on the JSContext
* stack, which avoids an expensive JS_SaveFrameChain in the common case.
*
* Most consumers of this should probably just use AutoJSContext. But the goal
* here is to preserve the existing behavior while ensure proper cx-stack
* semantics in edge cases where the context being used doesn't match the active
* context.
*
* NB: This will not push a null cx even if aCx is null. Make sure you know what
* you're doing.
*/
class MOZ_STACK_CLASS AutoPushJSContext {
Maybe<AutoCxPusher> mPusher;
JSContext* mCx;
public:
explicit AutoPushJSContext(JSContext* aCx);
operator JSContext*() { return mCx; }
};
} // namespace mozilla
#endif /* nsCxPusher_h */