JS callback objects (position, database) should store the global object instead of the frame.
Created attachment 39678 [details] patch This patch changes the callbacks to store a global object instead of the Frame and removes some unnecessary script()->isEnabled() checks. I will do a follow up which changes the global object to use the 'current' one.
Comment on attachment 39678 [details] patch r=me
Landed in r48453 with slight tweaks.
Why exec->dynamicGlobalObject() some times and exec->lexicalGlobalObject() other times? Also, this patch doesn't have tests...
Yeah, this definitely needs tests...