I suppose in theory this could cause problems for binary addons, but this code is totally untested, so any such users should switch over to Suspect2 and Forget2.
Comment on attachment 655390 [details] [diff] [review] WIP Try run looked okay. https://tbpl.mozilla.org/?tree=Try&rev=9dd920702889
Comment on attachment 655390 [details] [diff] [review] WIP Ask review also from bsmedberg
Comment on attachment 655390 [details] [diff] [review] WIP So hrm, the only part of this that's maybe a problem is the XPCOMFunctions/kFrozenFunctions bit. Nobody is *supposed* to be using that struct across versions any more, but I bet they are. I'm pretty sure nobody is going to be using these specific functions, though. I think it would be better to keep those functions declared and just make them NULL in the struct, which avoids rearranging all the following functions.
Created attachment 655641 [details] [diff] [review] remove them Thanks, that makes sense.