mirror of
https://gitlab.winehq.org/wine/wine-gecko.git
synced 2024-09-13 09:24:08 -07:00
f7383de1b3
The old code does two little bits of special sauce that are worth mentioning:
1 - It calls OnWrapper{Created,Destroyed} to maintain the lifetime of the
addref'd context stack pointer. But that whole thing is gone now.
2 - It calls ScriptEvaluated to potentially invoke termination functions in
certain cases. nsCxPusher does this too, but with slightly different logic.
In particular, nsCxPusher checks whether the given JSContext was already
on the stack, whereas AutoCXPusher checked whether there was another cx on
the stack above this one. As far as I can tell from my investigations of
this stuff, this is all total voodoo, and I think it should probably be fine
to just use an nsCxPusher here.
Also, termination functions are going away soon in bug 841312.
|
||
|---|---|---|
| .. | ||
| base | ||
| ipc | ||
| test | ||