Sorry I forgot to mention, I still kept it in the latest push, I only got rid of the hacky storage events tests.
If you really feel strongly against it and don't want to review it at all (what I mean is: to see whether it changes behavior compared to before, if I missed something, but I hope I didn't since I did it pretty thorough), then please tell me that as last confirmation because, well, waiting for nothing on my end is not really productive, especially with code freeze in less than 2 weeks.
I'll still get rid of document's `outer_window` since it's broken, and I can get rid of the outer window change if you dislike it so strongly, with the risk of introducing more regressions (because it changes existing behavior, unlike my patch is supposed not to). I can't say I'd be too happy about it, given I've spent time on it (and was suggested as solution last time), but I'd like to move it forward too.
That said, it was similar with postMessage being a hook, then rewrote it not to be, and now I had to make it a hook again (for the caller SP) to obtain the source, can't say I'm super thrilled about constantly rewriting what I had originally… I mean if we ever find something depending on the outer window being held, but honestly, it will be a nightmare to figure out if it does, and I'm far more worried about that, but it's pointless if it won't convince you.
Anyway let me know what I should do to get this moving.