getComputedStyle doesn't use defaultView from Node

[[email protected]…]

dojo.html.getComputedStyle should use the defaultView of the node's owning document instead of the defaultView which dojo is loaded from. This allows getComputedStyle to work on nodes from other documents (e.g., inside iframes).

i.e., dojo uses: document.defaultView.getComputedStyle(node, "").getPropertyValue(cssSelector);

but it should use: node.ownerDocument.defaultView.getComputedStyle(node, "").getPropertyValue(cssSelector);

[nrlz]

testcase (Firefox only)

[stenduncan]

I posted a question to the dojo-interest list that seems related to this (thanks to Robert Coup for pointing me here). My original post is at:

http://dojotoolkit.org/pipermail/dojo-interest/2006-December/022393.html

Basically my problem involves nested iframes and the accessibility function "testAccessible" (a11y.js). There is a call to getComputedStyle(div,"") which actually returns null and therefore causes the script to then fail on the next line.

Hopefully the fix made for this ticket will also fix that problem.

[[email protected]…]

I just ran into the same problem

A quick fix would be to change line 32 to the following:

if (cStyle) bkImg = cStyle.getPropertyValue("background-image");

Keep in mind this is a temporary solution.

[liucougar]

(In [7877]) fixes #2151