jquery/jquery — pull request #1036 by orkel (2012-11-18 23:39:04+00:00)
Title: Follow-up for #8908 fix

@elijahmanor, @dmethvin
Didn't mean to step on any toes here, but unfortunately fix for #8908 is incomplete

First of all, this comment is incorrect, it should 8908 not 8909, simple typo there is.
This assignment is redundant, JSHint is now ok without it.

But what more important, this lines, by my opinion, should be in cloneFixAttribute function, although function name might be confusing.

There is no reason to use getComputedStyle on xml nodes.

You would not have to check for nodeType, cloneFixAttribute already do that.

This fix would not fix issue like that, although, it helps for childrens of disconnected and cloned parent element, but only in IE9.

You might already talked about this, and some perfomance decrease might be an issue, but it felt important to bring this up.