Dimensions functions fail exceptionally on parent-frame scope

[nphase]

Forgive me for adding to your pile of work. I'd suggest my own fix but I'm not quite skilled enough with the codebase to do such.

The Dimensions set of functions do not properly handle the parent scope (frames).

parent.$("#overlay,#alert").hide().fadeIn("slow");

^{all of that will run perfectly, yet something like}

parent.$(window).height()

will fail and throw an exeception "NS_ERROR_XPC_BAD_CONVERT_JS" in FF.

parent.window.innerHeight is verified to work properly.

If you need any help testing or replication, feel free to ask

[nphase]

Surprisingly, I got this to work when I tried:

window.parent.$().height(); window.parent.$(document).height() does not work (seems like it should) window.parent.$(window).height() does not work (seems like it shouldn't).

Hopefully this will help someone!

[nphase]

Surprisingly, I got this to work when I tried:

window.parent.$().height();
window.parent.$(document).height() does not work (seems like it should)
window.parent.$(window).height() does not work (seems like it shouldn't).

Hopefully this will help someone!

(formatting)

[flesler]

I think this is quite expected, here why (I think):

parent.$(window) is calling the jQuery function of the parent window, but passing the window from the frame.

That means... The jQuery object from another window, touching a foreign window.

Maybe this could be indeed patched. That would require to never assume that window is the only window. So replace every foo == window for !!foo.setInterval or something like that.

Still, could fail in many cases. This is not a very common use and I'm not sure it's worthy to patch such a thing.

I will keep this open, assigned to Brandon.

[brandon]

This is fixed in latest SVN. Please test it out and let me know if it doesn't work for you.